Title: [225230] trunk/Tools
Revision
225230
Author
[email protected]
Date
2017-11-28 11:17:29 -0800 (Tue, 28 Nov 2017)

Log Message

Bugs filed from the flakiness dashboard should include a link to the dashboard
https://bugs.webkit.org/show_bug.cgi?id=180090

Reviewed by Aakash Jain.

* TestResultServer/static-dashboards/flakiness_dashboard.js:
(createBugHTML):
Open the bug compose page in a new tab and include a link
to the flakiness dashboard results for the selected test.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (225229 => 225230)


--- trunk/Tools/ChangeLog	2017-11-28 19:10:44 UTC (rev 225229)
+++ trunk/Tools/ChangeLog	2017-11-28 19:17:29 UTC (rev 225230)
@@ -1,3 +1,15 @@
+2017-11-28  Ryan Haddad  <[email protected]>
+
+        Bugs filed from the flakiness dashboard should include a link to the dashboard
+        https://bugs.webkit.org/show_bug.cgi?id=180090
+
+        Reviewed by Aakash Jain.
+
+        * TestResultServer/static-dashboards/flakiness_dashboard.js:
+        (createBugHTML):
+        Open the bug compose page in a new tab and include a link
+        to the flakiness dashboard results for the selected test.
+
 2017-11-28  Jonathan Bedard  <[email protected]>
 
         webkitpy: Standardize Version representation

Modified: trunk/Tools/TestResultServer/static-dashboards/flakiness_dashboard.js (225229 => 225230)


--- trunk/Tools/TestResultServer/static-dashboards/flakiness_dashboard.js	2017-11-28 19:10:44 UTC (rev 225229)
+++ trunk/Tools/TestResultServer/static-dashboards/flakiness_dashboard.js	2017-11-28 19:17:29 UTC (rev 225230)
@@ -1319,13 +1319,14 @@
 {
     var symptom = test.isFlaky ? 'flaky' : 'failing';
     var title = encodeURIComponent('Layout Test ' + test.test + ' is ' + symptom);
+    var dashboardURL = 'https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=' + encodeURIComponent(test.test);
     var description = encodeURIComponent('The following layout test is ' + symptom + ' on ' +
-        '[insert platform]\n\n' + test.test + '\n\nProbable cause:\n\n' +
-        '[insert probable cause]');
+        '[insert platform]\n\n' + test.test + '\n\nProbable cause:\n\n' + '[insert probable cause]' +
+        '\n\nFlakiness Dashboard:\n\n' + dashboardURL);
     
     var component = encodeURIComponent('Tools / Tests');
     url = '' + component + '&short_desc=' + title + '&comment=' + description;
-    return '<a href="" + url + '" class="file-bug">File</a>';
+    return '<a href="" + url + '" class="file-bug" target="_blank">File</a>';
 }
 
 function isCrossBuilderView()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to