Closed Bug 517744 Opened 10 years ago Closed 10 years ago
layout/reftests/svg/sizing/dynamic--* reftests are too slow
Running the following 6 reftests: layout/reftests/svg/sizing/ dynamic--inline-css-height.xhtml dynamic--inline-css-width.xhtml dynamic--inline-resize-cb-height.xhtml dynamic--inline-resize-cb-width.xhtml dynamic--inline-resize-window-width.xhtml dynamic--object-svg-unloaded.xhtml takes about 35 seconds. The problem seems to be this: // in case we're not gecko setTimeout(notify_test_finished, 5000); can we conditionalize that somehow?
We shouldn't be using the timeout at all, we should be getting a MozReftestInvalidate much sooner than that. Perhaps I should just backout bug 509274
FWIW, this works reliably in my local Linux debug build and two runs on TryServer were green, so it might be worth a try?
Lovely. r=longsonr even though you don't need it. Thanks for sorting this out.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Now that this has landed, we're seeing bug 509274 again: http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1253658060.1253668462.26439.gz
Ouch, well it was worth a try at least. Should we reopen bug 509274 or file a new bug?
Filed bug 518274.
Assignee: nobody → matspal
Target Milestone: --- → mozilla1.9.3a1
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.