Closed Bug 1385298 Opened 2 years ago Closed 2 years ago
.html | load failed: timed out waiting for reftest-wait to be removed
Filed by: cbook [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=118917299&repo=autoland https://queue.taskcluster.net/v1/task/b3VypVtwS2SQE7chKSCJww/runs/0/artifacts/public/logs/live_backing.log https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/b3VypVtwS2SQE7chKSCJww/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1
Hi Bevis, I know this is not at your typical wheelhouse, but could you please help take a look first and suggest the next step? Thanks!
Sure! Keep NI on me to follow-up!
The window.frames.location.reload() happens too soon - it just reloads the iframe on and on w/o a chance for it to load and get to the point when unload event can trigger, which can happen on any slower machine and specially on android and specially when we remove cache read/write concurrency of top level loads in bug 1340197. This test takes enormous amount of time even when passing, on most platforms - I've seem 2 to 3 minutes! I rewrote it to make more sense and be stable and also pass way more quickly. I also verified using  that it still crashes Firefox before bug 504224 fix. Ben, you were on the list of suggested reviewers, feel free to bounce or forward at will, thanks. https://treeherder.mozilla.org/#/jobs?repo=try&revision=f0411e42c5151025163b73ac1d6ce53b18a41c16  http://ftp.mozilla.org/pub/firefox/nightly/2009/07/2009-07-25-04-mozilla-central/
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8934325 - Flags: review?(bkelly)
Whiteboard: [stockwell needswork:owner] → [stockwell]
Thanks for looking into this, Honza!
Comment on attachment 8934325 [details] [diff] [review] v1 Review of attachment 8934325 [details] [diff] [review]: ----------------------------------------------------------------- Seems reasonable. Thank you for verifying that it still crashes before bug 504224 was fixed!
Attachment #8934325 - Flags: review?(bkelly) → review+
Thanks for a quick r+!
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/bff73e215867 Fix test for bug 504224 to actually do something. r=bkelly
Whiteboard: [stockwell] → [stockwell fixed:other]
You need to log in before you can comment on or make changes to this bug.