RESOLVED FIXED 36402
replace all occurrences of crbug.com and BUGXXXXX in Chromium test_expectations 
https://bugs.webkit.org/show_bug.cgi?id=36402
Summary replace all occurrences of crbug.com and BUGXXXXX in Chromium test_expectations
Dirk Pranke
Reported 2010-03-19 18:26:05 PDT
We need to do a pass over LayoutTests/platform/chromium/test_expectations.txt to update bug references to distinguish between webkit.org bug numberse and chromium.org bug numbers. Ojan suggested something like WKXXXX and CRXXXX, or we could change BUGXXXXX to refer to webkit bugs and use CRXXXX for Chromium bugs. Note that the actual text is meaningless to everything except for (as far as I know) some of the dashboards, like the Chromium LTTF dashboard and maybe the flakiness dashboards. Ojan - do you use the BUGXXXX syntax for anything in the flakiness dashboards?
Attachments
Add attachment proposed patch, testcase, etc.
Ojan Vafai
Comment 1 2010-03-19 18:32:34 PDT
Yes, but only in line 790 of trunk/tools/dashboards/flakiness_dashboard.html. The fix is just to change BUG to CR in that regexp. We'll also need a little bit of extra logic to also consider WKXXX as a bug instead of another kind of modifier.
Dirk Pranke
Comment 2 2011-02-17 12:34:33 PST
This was fixed a while ago (not sure when). We actually print out warnings now if you try to use the old BUGXXXX syntax.
Note You need to log in before you can comment on or make changes to this bug.