Closed Bug 1308526 Opened 3 years ago Closed 3 years ago
_bug1307747 .js | Test timed out -
59 bytes, text/x-review-board-request
Pretty sure this is broken because we don't wait for the page to load in the tab we're adding before spawning a ContentTask in it, making the content task race with the load of the page we requested, which then means we talk to dead things and time out. This can be solved by using openNewForegroundTab from BrowserTestUtils and yielding for it (which will wait for the load). I should have a patch soon, need to rebuild after the holidays.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment on attachment 8823256 [details] Bug 1308526 - fix waiting for the test page to load before trying to use it, https://reviewboard.mozilla.org/r/101802/#review102506 Net improvement.
Attachment #8823256 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/266b78fa3d41 fix waiting for the test page to load before trying to use it, r=mikedeboer
I'll look at this end of today / tomorrow and see if we're in the clear here.
No hits on trunk trees yesterday or today, and only 1 on the day this landed (before it landed) - the intermittent is dead, long live the intermittent. No, wait, that other thing.
You need to log in before you can comment on or make changes to this bug.