Bug 1261842 - Update browser_bug495058.js to account for the initial browser being remote. r?Gijs
MozReview-Commit-ID: 6Ktl6Dyztg3
--- a/browser/base/content/test/general/browser_bug495058.js
+++ b/browser/base/content/test/general/browser_bug495058.js
@@ -10,30 +10,28 @@ const URIS = [
];
add_task(function*() {
for (let uri of URIS) {
let tab = gBrowser.addTab();
yield BrowserTestUtils.loadURI(tab.linkedBrowser, uri);
let win = gBrowser.replaceTabWithWindow(tab);
- yield BrowserTestUtils.waitForEvent(win, "load");
-
+ yield TestUtils.topicObserved("browser-delayed-startup-finished",
+ subject => subject == win);
tab = win.gBrowser.selectedTab;
- // By default, we'll wait for MozAfterPaint to come up through the
- // browser element. We'll handle the e10s case in the next block.
- let contentPainted = BrowserTestUtils.waitForEvent(tab.linkedBrowser,
- "MozAfterPaint");
-
- let delayedStartup =
- TestUtils.topicObserved("browser-delayed-startup-finished",
- subject => subject == win);
-
- yield Promise.all([delayedStartup, contentPainted]);
+ // BrowserTestUtils doesn't get the add-on shims, which means that
+ // MozAfterPaint won't get shimmed over if we add an event handler
+ // for it in the parent.
+ if (tab.linkedBrowser.isRemoteBrowser) {
+ yield BrowserTestUtils.waitForContentEvent(tab.linkedBrowser, "MozAfterPaint");
+ } else {
+ yield BrowserTestUtils.waitForEvent(tab.linkedBrowser, "MozAfterPaint");
+ }
Assert.equal(win.gBrowser.currentURI.spec, uri, uri + ": uri loaded in detached tab");
Assert.equal(win.document.activeElement, win.gBrowser.selectedBrowser, uri + ": browser is focused");
Assert.equal(win.gURLBar.value, "", uri + ": urlbar is empty");
Assert.ok(win.gURLBar.placeholder, uri + ": placeholder text is present");
yield BrowserTestUtils.closeWindow(win);
}