Bug 1391042 - Don't load Sync UI for hidden windows. r?Gijs draft
authorEdouard Oger <eoger@fastmail.com>
Wed, 16 Aug 2017 16:26:53 -0400
changeset 647696 88372411602fa0d463371c6a893e8fdeedb5cb4e
parent 647624 07ce8c96222d533fc89c02802143d35d7c351f9c
child 726608 8abe4043d645ad600c6bdd72d9b31ab2d49915a2
push id74514
push userbmo:eoger@fastmail.com
push dateWed, 16 Aug 2017 20:27:08 +0000
reviewersGijs
bugs1391042
milestone57.0a1
Bug 1391042 - Don't load Sync UI for hidden windows. r?Gijs MozReview-Commit-ID: FUhegHkOoiV
browser/base/content/browser-sync.js
--- a/browser/base/content/browser-sync.js
+++ b/browser/base/content/browser-sync.js
@@ -85,17 +85,18 @@ var gSync = {
       // we open the window. We can avoid a repaint.
       if (state.status != UIState.STATUS_NOT_CONFIGURED) {
         this.updateAllUI(state);
       }
     }
   },
 
   init() {
-    if (this._initialized) {
+    // Bail out if we're already initialized or for hidden windows.
+    if (this._initialized || window.location.href != getBrowserURL()) {
       return;
     }
 
     for (let topic of this._obs) {
       Services.obs.addObserver(this, topic, true);
     }
 
     this._generateNodeGetters();