https://treeherder.mozilla.org/#/jobs?repo=try&revision=dc35b181a4a55c0bd3cca8a19346ff224545a649&selectedJob=143221643 > FAIL | browser/components/sessionstore/test/browser_crashedTabs.js | Should have closed the tab it's because of the TabClose event is fired and the promise gets resolved before the tab actually gets closed. it should wait for the tab gets closed. then, in bug 1416446, I'm going to make BrowserTestUtils.waitForEvent to do so, and the promiseEvent in browser/components/sessionstore/test/head.js can be replaced with it.

I'll ask review after bug 1416446 (since this patch highly depends on the behavior changed there

Comment on attachment 8929708 [details] [diff] [review] Use BrowserTestUtils.waitForEvent in browser/components/sessionstore/test/. after bug 1193394, the test needs to wait for the next event tick after each event handler gets called, before proceeding to the remaining part. because otherwise the remaining part will be executed too early, compared to current behavior, and UI etc is not yet ready. then, BrowserTestUtils.waitForEvent is going to be modified to do so, in bug 1416153. so, instead of modifying promiseEvent, I changed those consumers to use BrowserTestUtils.waitForEvent.

(In reply to Tooru Fujisawa [:arai] from comment #2) > then, BrowserTestUtils.waitForEvent is going to be modified to do so, in bug > 1416153. > so, instead of modifying promiseEvent, I changed those consumers to use > BrowserTestUtils.waitForEvent. sorry, I pointed wrong bug. it's bug 1416446, not bug 1416153.

arai, wanna land this?

sorry, I forgot to update dependency. it's also blocked by bug 1421113 (the last patch there) I'll land this once those gets fixed.

moved the patch to bug 1424074.

Cool, thanks!

Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/935dcd1ea40c Use BrowserTestUtils.waitForEvent in browser/components/sessionstore/test/. r=felipe

https://hg.mozilla.org/integration/mozilla-inbound/rev/935dcd1ea40cb1b32a1bca067a3e8f422fa8fbef Bug 1418184 - Use BrowserTestUtils.waitForEvent in browser/components/sessionstore/test/. r=felipe

Backout by dluca@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/847ed3d5357c Backed out changeset 935dcd1ea40c for failing browser-chrome's browser/base/content/test/permissions/browser_temporary_permissions_expiry.js on Windows 7 debug without e10s r=backout on a CLOSED TREE

Backed out changeset 56343b0de5c2 (bug 1424083) for failing browser-chrome's browser/base/content/test/permissions/browser_temporary_permissions_expiry.js on Windows 7 debug without e10s r=backout on a CLOSED TREE Failure push: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=1c819b56fe973b077d64737db186b066850e1197 Failure log: https://hg.mozilla.org/integration/mozilla-inbound/rev/a73f5a62f971efb9afa3b2e70b5d01b447ba3ed3 Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/847ed3d5357c6120f39c27b9198550c222b4d2b1

https://hg.mozilla.org/integration/mozilla-inbound/rev/8d1a1a8c9a420279e3b667bef15ebf863ccf556d Bug 1418184 - Use BrowserTestUtils.waitForEvent in browser/components/sessionstore/test/. r=felipe