Bug 1394207 - Export a function to show the sidebar without firing fo…

…cus for session restore;r=Gijs

MozReview-Commit-ID: uHrShpl2xJ

--HG--
extra : rebase_source : b9a79d541171982c6860edec366722821844f3e1

browser/base/content/browser-sidebar.js

@@ -194,7 +194,7 @@ var SidebarUI = {
     }
 
     this._box.setAttribute("width", sourceUI._box.boxObject.width);
-    this._show(commandID);
+    this.showInitially(commandID);
 
     return true;
   },
@@ -229,7 +229,7 @@ var SidebarUI = {
     }
 
     if (document.getElementById(commandID)) {
-      this._show(commandID);
+      this.showInitially(commandID);
     } else {
       // Remove the |sidebarcommand| attribute, because the element it
       // refers to no longer exists, so we should assume this sidebar
@@ -335,6 +335,17 @@ var SidebarUI = {
     });
   },
 
+  /**
+   * Show the sidebar, without firing the focused event or logging telemetry.
+   * This is intended to be used when the sidebar is opened automatically
+   * when a window opens (not triggered by user interaction).
+   *
+   * @param {string} commandID ID of the xul:broadcaster element to use.
+   */
+   showInitially(commandID) {
+     return this._show(commandID);
+   },
+
   /**
    * Implementation for show. Also used internally for sidebars that are shown
    * when a window is opened and we don't want to ping telemetry.

browser/components/sessionstore/SessionStore.jsm

@@ -4087,7 +4087,7 @@ var SessionStoreInternal = {
       }
       var sidebar = aWindow.document.getElementById("sidebar-box");
       if (sidebar.getAttribute("sidebarcommand") != aSidebar) {
-        aWindow.SidebarUI.show(aSidebar);
+        aWindow.SidebarUI.showInitially(aSidebar);
       }
       // since resizing/moving a window brings it to the foreground,
       // we might want to re-focus the last focused window