Bug 1385630 - Backed out changeset d159686d279c for regressing bug 1374048, rs=mixedpuppy
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Fri, 01 Sep 2017 17:22:40 +0100
changeset 378277 1470c550ab6d543cbeaeac953fba00df9cb69626
parent 378276 d2305bd57a979d78015fb7421a2d7be6918a17e8
child 378278 a85f03aacb81ef7ca26e9942c41832a9a2044116
push id32427
push userkwierso@gmail.com
push dateFri, 01 Sep 2017 23:29:24 +0000
treeherdermozilla-central@5278dfcf5eb9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmixedpuppy
bugs1385630, 1374048
milestone57.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1385630 - Backed out changeset d159686d279c for regressing bug 1374048, rs=mixedpuppy MozReview-Commit-ID: BkV6C3m7LWx
browser/components/extensions/ext-sidebarAction.js
--- a/browser/components/extensions/ext-sidebarAction.js
+++ b/browser/components/extensions/ext-sidebarAction.js
@@ -60,19 +60,16 @@ this.sidebarAction = class extends Exten
       icon: IconDetails.normalize({path: options.default_icon}, extension),
       panel: options.default_panel || "",
     };
 
     this.tabContext = new TabContext(tab => Object.create(this.defaults),
                                      extension);
 
     // We need to ensure our elements are available before session restore.
-    for (let window of windowTracker.browserWindows()) {
-      this.createMenuItem(window, this.defaults);
-    }
     this.windowOpenListener = (window) => {
       this.createMenuItem(window, this.defaults);
     };
     windowTracker.addOpenListener(this.windowOpenListener);
 
     this.updateHeader = (event) => {
       let window = event.target.ownerGlobal;
       let details = this.tabContext.get(window.gBrowser.selectedTab);