[Dashboard] Clean up the timeline updating code, removing Chromium-specific cases
authorzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Apr 2013 20:39:22 +0000 (20:39 +0000)
committerzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Apr 2013 20:39:22 +0000 (20:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114756

Patch by Zan Dobersek <zandobersek@gmail.com> on 2013-04-17
Reviewed by Ryosuke Niwa.

Clean up the updating of the timeline in the timeline explorer, defaulting to the webkit.org build master when
constructing the results URL and removing the possible row addition containing the Chromium commit range that
only applied to Chromium builders in the first place. Also removes the now-redundant shouldShowWebKitRevisionsOnly
method.

* TestResultServer/static-dashboards/timeline_explorer.js:

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

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

index 72f779c..7878d46 100644 (file)
@@ -1,3 +1,17 @@
+2013-04-17  Zan Dobersek  <zandobersek@gmail.com>
+
+        [Dashboard] Clean up the timeline updating code, removing Chromium-specific cases
+        https://bugs.webkit.org/show_bug.cgi?id=114756
+
+        Reviewed by Ryosuke Niwa.
+
+        Clean up the updating of the timeline in the timeline explorer, defaulting to the webkit.org build master when
+        constructing the results URL and removing the possible row addition containing the Chromium commit range that
+        only applied to Chromium builders in the first place. Also removes the now-redundant shouldShowWebKitRevisionsOnly
+        method.
+
+        * TestResultServer/static-dashboards/timeline_explorer.js:
+
 2013-04-17  Alberto Garcia  <agarcia@igalia.com>
 
         media-stream and xslt are no longer configurable options in the GTK+ port
index b885d63..b4c7977 100644 (file)
@@ -111,11 +111,6 @@ function initCurrentBuilderTestResults()
     console.log( 'Time to get test results by build: ' + (Date.now() - startTime));
 }
 
-function shouldShowWebKitRevisionsOnly()
-{
-    return isTipOfTreeWebKitBuilder();
-}
-
 function updateTimelineForBuilder()
 {
     var builder = g_history.dashboardSpecificState.builder || currentBuilderGroup().defaultBuilder();
@@ -229,19 +224,12 @@ function updateBuildInspector(results, builder, dygraph, index)
     addRow('', '');
     var master = builderMaster(builder);
     var buildUrl = master.logPath(builder, results[BUILD_NUMBERS_KEY][index]);
-    if (master.name == WEBKIT_BUILDER_MASTER) {
-        var resultsUrl = 'http://build.webkit.org/results/' + builder + '/r' + results[WEBKIT_REVISIONS_KEY][index] +
-            ' (' + results[BUILD_NUMBERS_KEY][index] + ')';
-    } else {
-        var resultsUrl = 'http://build.chromium.org/f/chromium/layout_test_results/' +
-            currentBuilders()[builder] + '/' + results[CHROME_REVISIONS_KEY][index];
-    }
+    var resultsUrl = 'http://build.webkit.org/results/' + builder + '/r' + results[WEBKIT_REVISIONS_KEY][index] +
+        ' (' + results[BUILD_NUMBERS_KEY][index] + ')';
 
     addRow('Build:', '<a href="' + buildUrl + '" target="_blank">' + buildNumber + '</a> (<a href="' + resultsUrl + '" target="_blank">results</a>)');
 
-    // Revision link(s)
-    if (!shouldShowWebKitRevisionsOnly())
-        addRow('Chromium change:', ui.html.chromiumRevisionLink(results, index));
+    // Revision link
     addRow('WebKit change:', ui.html.webKitRevisionLink(results, index));
 
     // Test status/counts