|Product:||WebKit||Reporter:||Dima Gorbik <dgorbik>|
|Component:||New Bugs||Assignee:||Dima Gorbik <dgorbik>|
|Severity:||Normal||CC:||barraclough, benjamin, dgorbik, fpizlo, ggaren, mark.lam, mhahnenberg, msaboff, oliver, webkit.review.bot|
|Version:||528+ (Nightly build)|
Description Dima Gorbik 2013-02-12 21:22:07 PST
Skipping the flacky test fast/js/dfg-poison-fuzz.html
Comment 2 WebKit Review Bot 2013-02-12 21:34:52 PST
Attachment 188004 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/platform/mac/TestExpectations']" exit_code: 1 LayoutTests/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber]  Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Benjamin Poulain 2013-02-12 22:01:05 PST
Comment on attachment 188004 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188004&action=review > LayoutTests/platform/mac/TestExpectations:1339 > +# Timing out or crashing. > +webkit.org/b/109657 fast/js/dfg-poison-fuzz.html [ Crash Timeout Pass ] I think this needs [ Debug ]. It looks like the test works fine webkit.org/b/109657 [ Debug ] fast/js/dfg-poison-fuzz.html [ Timeout Pass ] The test seems to Pass fairly reliably in release.
Comment 4 Filip Pizlo 2013-02-12 22:17:28 PST
Actually, I haven't seen this test fail in *ages*. And I've seen a lot of DFG bugs! This is a super expensive test for one bug that used to be in the DFG but that has since been thoroughly eradicated. Likely the plethora of other DFG tests cover that bug. Heck, I'm not even sure if the code that this covers is even in the repository anymore. I will remove it.
Comment 5 Filip Pizlo 2013-02-12 22:18:05 PST
(In reply to comment #4) > Actually, I haven't seen this test fail in *ages*. And I've seen a lot of DFG bugs! > > This is a super expensive test for one bug that used to be in the DFG but that has since been thoroughly eradicated. Likely the plethora of other DFG tests cover that bug. Heck, I'm not even sure if the code that this covers is even in the repository anymore. > > I will remove it. (If anyone disagrees feel free to roll out my eradication patch.)