Title: [129083] trunk/Tools
- Revision
- 129083
- Author
- [email protected]
- Date
- 2012-09-19 18:52:02 -0700 (Wed, 19 Sep 2012)
Log Message
Fix regex groups for bug matching in flakiness dashboard.
https://bugs.webkit.org/show_bug.cgi?id=97152
Unreviewed, build fix.
* TestResultServer/static-dashboards/flakiness_dashboard.js:
(htmlForBugs):
Modified Paths
Diff
Modified: trunk/Tools/ChangeLog (129082 => 129083)
--- trunk/Tools/ChangeLog 2012-09-20 01:41:03 UTC (rev 129082)
+++ trunk/Tools/ChangeLog 2012-09-20 01:52:02 UTC (rev 129083)
@@ -1,5 +1,15 @@
2012-09-19 Dirk Pranke <[email protected]>
+ Fix regex groups for bug matching in flakiness dashboard.
+ https://bugs.webkit.org/show_bug.cgi?id=97152
+
+ Unreviewed, build fix.
+
+ * TestResultServer/static-dashboards/flakiness_dashboard.js:
+ (htmlForBugs):
+
+2012-09-19 Dirk Pranke <[email protected]>
+
nrwt: print unexpected results using new TestExpectations syntax
https://bugs.webkit.org/show_bug.cgi?id=97159
Modified: trunk/Tools/TestResultServer/static-dashboards/flakiness_dashboard.js (129082 => 129083)
--- trunk/Tools/TestResultServer/static-dashboards/flakiness_dashboard.js 2012-09-20 01:41:03 UTC (rev 129082)
+++ trunk/Tools/TestResultServer/static-dashboards/flakiness_dashboard.js 2012-09-20 01:52:02 UTC (rev 129083)
@@ -972,18 +972,16 @@
var BUG_URL_PREFIX = '<a href=""
-var BUG_URL_POSTFIX = '/$2">crbug.com/$2</a> ';
-var WEBKIT_BUG_URL_POSTFIX = '/$2">webkit.org/b/$2</a> ';
+var BUG_URL_POSTFIX = '/$1">crbug.com/$1</a> ';
+var WEBKIT_BUG_URL_POSTFIX = '/$1">webkit.org/b/$1</a> ';
var INTERNAL_BUG_REPLACE_VALUE = BUG_URL_PREFIX + 'b' + BUG_URL_POSTFIX;
var EXTERNAL_BUG_REPLACE_VALUE = BUG_URL_PREFIX + 'crbug.com' + BUG_URL_POSTFIX;
var WEBKIT_BUG_REPLACE_VALUE = BUG_URL_PREFIX + 'webkit.org/b' + WEBKIT_BUG_URL_POSTFIX;
function htmlForBugs(bugs)
{
- // FIXME: do we still need the INTERNAL_BUG replacements? They're
- // broken temporarily.
bugs = bugs.replace(/crbug.com\/(\d+)(\ |$)/g, EXTERNAL_BUG_REPLACE_VALUE);
- bugs = bugs.replace(/webkit.org\/b\/(\d{5}\d*?)(\ |$)/g, WEBKIT_BUG_REPLACE_VALUE);
+ bugs = bugs.replace(/webkit.org\/b\/(\d+)(\ |$)/g, WEBKIT_BUG_REPLACE_VALUE);
return bugs;
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
