Bug 1287007 - Fix timing issue in browser_ext_tabs_getCurrent.js
browser.test.sendMessage does not have enough time to finish
before tabs.remove since test moved to ChildAPIManager for
extension pages, causing the test to time out.
MozReview-Commit-ID: 1mmGZOi9fzm
--- a/browser/components/extensions/test/browser/browser_ext_tabs_getCurrent.js
+++ b/browser/components/extensions/test/browser/browser_ext_tabs_getCurrent.js
@@ -22,17 +22,16 @@ add_task(function* () {
if (tabId == currentTab.id) {
browser.tabs.onActivated.removeListener(listener);
browser.tabs.getCurrent(currentTab => {
browser.test.assertEq(currentTab.id, tabId, "in active background tab");
browser.test.assertEq(currentTab.url, url, "getCurrent in non-active background tab");
browser.test.sendMessage("tab-finished");
- browser.tabs.remove(tabId);
});
}
});
browser.tabs.update(currentTab.id, {active: true});
});
},
"popup.js": function() {
@@ -61,10 +60,11 @@ add_task(function* () {
yield extension.awaitMessage("background-finished");
yield extension.awaitMessage("tab-finished");
clickBrowserAction(extension);
yield awaitExtensionPanel(extension);
yield extension.awaitMessage("popup-finished");
yield closeBrowserAction(extension);
+ // The extension tab is automatically closed when the extension unloads.
yield extension.unload();
});