Commit 43ab0827 authored by Itiel's avatar Itiel
Browse files

Bug 1634247 - Fix separators in the Web Developer menu. r=dao, a=pascalc 

Differential Revision: https://phabricator.services.mozilla.com/D73906
parent 999212fb

browser/components/customizableui/CustomizableUI.jsm

+2 −2
Original line number Diff line number Diff line
@@ -4367,11 +4367,11 @@ var CustomizableUI = { 
        // menus (which we don't copy) above the separator.

        if (

          !fragment.lastElementChild ||

          fragment.lastElementChild.localName == "menuseparator"

          fragment.lastElementChild.localName == "toolbarseparator"

        ) {

          continue;

        }

        subviewItem = doc.createXULElement("menuseparator");

        subviewItem = doc.createXULElement("toolbarseparator");

      } else if (menuChild.localName == "menuitem") {

        subviewItem = doc.createXULElement("toolbarbutton");

        CustomizableUI.addShortcut(menuChild, subviewItem);

devtools/startup/DevToolsStartup.jsm

+1 −1
Original line number Diff line number Diff line
@@ -534,7 +534,7 @@ DevToolsStartup.prototype = { 
   * Also, this menu duplicates its own entries from the "Web Developer"

   * menu in the system menu, under "Tools" main menu item. The system

   * menu is being hooked by "hookWebDeveloperMenu" which ends up calling

   * devtools/client/framework/browser-menu to create the items for real,

   * devtools/client/framework/browser-menus to create the items for real,

   * initDevTools, from onViewShowing is also calling browser-menu.

   */

  hookDeveloperToggle() {