Builder should not be a valid parameter for stats dashboard
authorjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 12 Jan 2013 00:41:48 +0000 (00:41 +0000)
committerjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 12 Jan 2013 00:41:48 +0000 (00:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=106698

Reviewed by Dirk Pranke.

Since builder is a dashboard specific parameter, dashboard_base
should not be handling it in handleValidHashParameterWrapper, rather,
each specific dashboard should in its handleValidHashParameter function.

Note that the FIXME in dashboard_base about this was actually wrong -
we don't want to move it to g_crossDashboardState, because it isn't
actually cross dashboard state.

In fact, flakiness and treemap have the code in
handleValidHashParameter, but it is currently dead code because
dashboard_base gets to it first.

Added the proper handling to timeline_explorer, since builder is
valid there.

* TestResultServer/static-dashboards/dashboard_base.js:
* TestResultServer/static-dashboards/timeline_explorer.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@139522 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/TestResultServer/static-dashboards/dashboard_base.js
Tools/TestResultServer/static-dashboards/timeline_explorer.html

index 2fe4aee..a1088ba 100644 (file)
@@ -1,3 +1,28 @@
+2013-01-11  Julie Parent  <jparent@chromium.org>
+
+        Builder should not be a valid parameter for stats dashboard
+        https://bugs.webkit.org/show_bug.cgi?id=106698
+
+        Reviewed by Dirk Pranke.
+        
+        Since builder is a dashboard specific parameter, dashboard_base
+        should not be handling it in handleValidHashParameterWrapper, rather,
+        each specific dashboard should in its handleValidHashParameter function.
+    
+        Note that the FIXME in dashboard_base about this was actually wrong -
+        we don't want to move it to g_crossDashboardState, because it isn't
+        actually cross dashboard state.
+        
+        In fact, flakiness and treemap have the code in 
+        handleValidHashParameter, but it is currently dead code because 
+        dashboard_base gets to it first.
+        
+        Added the proper handling to timeline_explorer, since builder is
+        valid there.
+
+        * TestResultServer/static-dashboards/dashboard_base.js:
+        * TestResultServer/static-dashboards/timeline_explorer.html:
+
 2013-01-11  Ryosuke Niwa  <rniwa@webkit.org>
 
         REGRESSION: buildbot mangles error messages with timestamps run-webkit-tests outputs
index 35093e2..5e56d9d 100644 (file)
@@ -185,12 +185,6 @@ function handleValidHashParameterWrapper(key, value)
             });
         return true;
 
-    // FIXME: This should probably be stored on g_crossDashboardState like everything else in this function.
-    case 'builder':
-        validateParameter(g_currentState, key, value,
-            function() { return value in currentBuilders(); });
-        return true;
-
     case 'useTestData':
     case 'showAllRuns':
         g_crossDashboardState[key] = value == 'true';
index d4aa8e8..ac5891a 100644 (file)
@@ -152,6 +152,10 @@ function initCurrentBuilderTestResults()
 function handleValidHashParameter(key, value)
 {
     switch(key) {
+    case 'builder':
+        validateParameter(g_currentState, key, value,
+            function() { return value in currentBuilders(); });
+        return true;
     case 'buildTimestamp':
         g_currentState.buildTimestamp = parseInt(value, 10);
         return true;