[tbb-commits] [tor-browser/tor-browser-45.1.1esr-6.0-1] Bug 18884: Add --disable-loop flag

gk at torproject.org gk at torproject.org
Tue May 31 11:27:10 UTC 2016


commit 61a4d57b1cc573b85baef431b277149cbf740c74
Author: Arthur Edelstein <arthuredelstein at gmail.com>
Date:   Tue May 17 16:40:33 2016 -0700

    Bug 18884: Add --disable-loop flag
    
    When a mozconfig file contains
      ac_add_options --disable-loop
    
    then the Loop extension will not be built or bundled with the browser.
---
 browser/base/content/browser-menubar.inc                  |  2 ++
 browser/base/content/browser.css                          |  4 ++++
 browser/base/content/browser.xul                          |  2 ++
 browser/components/about/AboutRedirector.cpp              |  2 ++
 browser/components/customizableui/content/panelUI.inc.xul |  2 ++
 browser/extensions/moz.build                              |  4 +++-
 browser/installer/package-manifest.in                     |  2 ++
 browser/locales/Makefile.in                               |  4 +++-
 browser/locales/jar.mn                                    |  2 ++
 configure.in                                              | 14 ++++++++++++++
 10 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/browser/base/content/browser-menubar.inc b/browser/base/content/browser-menubar.inc
index a987975..2c01c13 100644
--- a/browser/base/content/browser-menubar.inc
+++ b/browser/base/content/browser-menubar.inc
@@ -500,10 +500,12 @@
                         key="key_openAddons"
                         command="Tools:Addons"/>
 
+#ifdef MOZ_LOOP
               <menuitem id="menu_openLoop"
                         label="&loopMenuItem.label;"
                         accesskey = "&loopMenuItem.accesskey;"
                         oncommand="LoopUI.togglePanel();"/>
+#endif
 
               <!-- only one of sync-setup, sync-syncnowitem or sync-reauthitem will be showing at once -->
               <menuitem id="sync-setup"
diff --git a/browser/base/content/browser.css b/browser/base/content/browser.css
index c5dd9b4..f93d13f 100644
--- a/browser/base/content/browser.css
+++ b/browser/base/content/browser.css
@@ -18,6 +18,7 @@
 %endif
 }
 
+%ifdef MOZ_LOOP
 /* These values are chosen to keep the Loop detached chat window from
  * getting too small.  When it's too small, three bad things happen:
  *
@@ -41,6 +42,7 @@
   min-width: 260px;
   min-height: 315px;
 }
+%endif
 
 #main-window[customize-entered] {
   min-width: -moz-fit-content;
@@ -914,6 +916,7 @@ chatbox[customSize] {
   min-width: 350px;
 }
 
+%ifdef MOZ_LOOP
 chatbox[customSize="loopChatEnabled"] {
   /* 430px as defined per UX */
   height: 430px;
@@ -938,6 +941,7 @@ chatbox[customSize="loopChatDisabledMessageAppended"] {
   /* 445px + 30px top bar height. */
   min-height: calc(400px + 30px);
 }
+%endif
 
 chatbox[minimized="true"] {
   width: 160px;
diff --git a/browser/base/content/browser.xul b/browser/base/content/browser.xul
index c63dfed..8749477 100644
--- a/browser/base/content/browser.xul
+++ b/browser/base/content/browser.xul
@@ -290,6 +290,7 @@
            noautofocus="true"
            position="topcenter topright"/>
 
+#ifdef MOZ_LOOP
     <panel id="loop-notification-panel"
            class="loop-panel social-panel"
            type="arrow"
@@ -301,6 +302,7 @@
            type="arrow"
            orient="horizontal"
            hidden="true"/>
+#endif
 
     <menupopup id="toolbar-context-menu"
                onpopupshowing="onViewToolbarsPopupShowing(event, document.getElementById('viewToolbarsMenuSeparator'));">
diff --git a/browser/components/about/AboutRedirector.cpp b/browser/components/about/AboutRedirector.cpp
index ab069e5..2f956b2 100644
--- a/browser/components/about/AboutRedirector.cpp
+++ b/browser/components/about/AboutRedirector.cpp
@@ -108,6 +108,7 @@ static RedirEntry kRedirMap[] = {
   {
     "debugging", "chrome://devtools/content/aboutdebugging/aboutdebugging.xhtml",
     nsIAboutModule::ALLOW_SCRIPT },
+#ifdef MOZ_LOOP
   { "loopconversation", "chrome://loop/content/panels/conversation.html",
     nsIAboutModule::URI_SAFE_FOR_UNTRUSTED_CONTENT |
     nsIAboutModule::ALLOW_SCRIPT |
@@ -120,6 +121,7 @@ static RedirEntry kRedirMap[] = {
     nsIAboutModule::ENABLE_INDEXED_DB,
     // Shares an IndexedDB origin with about:loopconversation.
     "loopconversation" },
+#endif
   { "reader", "chrome://global/content/reader/aboutReader.html",
     nsIAboutModule::URI_SAFE_FOR_UNTRUSTED_CONTENT |
     nsIAboutModule::ALLOW_SCRIPT |
diff --git a/browser/components/customizableui/content/panelUI.inc.xul b/browser/components/customizableui/content/panelUI.inc.xul
index 99bca07..f8975bf 100644
--- a/browser/components/customizableui/content/panelUI.inc.xul
+++ b/browser/components/customizableui/content/panelUI.inc.xul
@@ -243,7 +243,9 @@
 
     <panelview id="PanelUI-socialapi" flex="1"/>
 
+#ifdef MOZ_LOOP
     <panelview id="PanelUI-loopapi" flex="1"/>
+#endif
 
     <panelview id="PanelUI-feeds" flex="1" oncommand="FeedHandler.subscribeToFeed(null, event);">
       <label value="&feedsMenu.label;" class="panel-subview-header"/>
diff --git a/browser/extensions/moz.build b/browser/extensions/moz.build
index b4d84e4..ef6a439 100644
--- a/browser/extensions/moz.build
+++ b/browser/extensions/moz.build
@@ -5,6 +5,8 @@
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 DIRS += [
-    'loop',
     'pdfjs',
 ]
+
+if CONFIG['MOZ_LOOP']:
+    DIRS += ['loop']
diff --git a/browser/installer/package-manifest.in b/browser/installer/package-manifest.in
index c3ce1a5..3d2c507 100644
--- a/browser/installer/package-manifest.in
+++ b/browser/installer/package-manifest.in
@@ -669,7 +669,9 @@
 @RESPATH@/browser/chrome/icons/default/default32.png
 @RESPATH@/browser/chrome/icons/default/default48.png
 #endif
+#ifdef MOZ_LOOP
 @RESPATH@/browser/features/*
+#endif #MOZ_LOOP
 
 ; [Webide Files]
 @RESPATH@/browser/chrome/webide at JAREXT@
diff --git a/browser/locales/Makefile.in b/browser/locales/Makefile.in
index be9454e..ae13de4 100644
--- a/browser/locales/Makefile.in
+++ b/browser/locales/Makefile.in
@@ -133,7 +133,9 @@ ifdef MOZ_WEBAPP_RUNTIME
 	@$(MAKE) -C ../../webapprt/locales AB_CD=$* XPI_NAME=locale-$*
 endif
 	@$(MAKE) -C ../../extensions/spellcheck/locales AB_CD=$* XPI_NAME=locale-$*
-	@$(MAKE) -C ../extensions/loop/chrome/locale AB_CD=$* XPI_NAME=locale-$*
+ifdef MOZ_LOOP
+        @$(MAKE) -C ../extensions/loop/chrome/locale AB_CD=$* XPI_NAME=locale-$*
+endif
 	@$(MAKE) -C ../../intl/locales AB_CD=$* XPI_NAME=locale-$*
 	@$(MAKE) -C ../../devtools/client/locales AB_CD=$* XPI_NAME=locale-$* XPI_ROOT_APPID='$(XPI_ROOT_APPID)'
 	@$(MAKE) -B searchplugins AB_CD=$* XPI_NAME=locale-$*
diff --git a/browser/locales/jar.mn b/browser/locales/jar.mn
index 0f18e06..63c19d4 100644
--- a/browser/locales/jar.mn
+++ b/browser/locales/jar.mn
@@ -30,7 +30,9 @@
     locale/browser/browser-pocket.properties       (%chrome/browser/browser-pocket.properties)
     locale/browser/customizableui/customizableWidgets.properties (%chrome/browser/customizableui/customizableWidgets.properties)
     locale/browser/lightweightThemes.properties    (%chrome/browser/lightweightThemes.properties)
+#ifdef MOZ_LOOP
     locale/browser/loop/loop.properties            (%chrome/browser/loop/loop.properties)
+#endif
     locale/browser/newTab.dtd                      (%chrome/browser/newTab.dtd)
     locale/browser/newTab.properties               (%chrome/browser/newTab.properties)
     locale/browser/pageInfo.dtd                    (%chrome/browser/pageInfo.dtd)
diff --git a/configure.in b/configure.in
index 885ee84..2b0c949 100644
--- a/configure.in
+++ b/configure.in
@@ -3737,6 +3737,7 @@ MOZ_URL_CLASSIFIER=
 MOZ_XUL=1
 MOZ_ZIPWRITER=1
 NS_PRINTING=1
+MOZ_LOOP=1
 if test "$MOZ_IOS"; then
    NS_PRINTING=
 fi
@@ -8456,6 +8457,19 @@ if test "$NECKO_COOKIES"; then
     _NON_GLOBAL_ACDEFINES="$_NON_GLOBAL_ACDEFINES NECKO_COOKIES"
 fi
 
+dnl ========================================================
+dnl disable loop extension
+dnl ========================================================
+MOZ_ARG_DISABLE_BOOL(loop,
+[  --disable-loop
+                          Disable loop extension],
+    MOZ_LOOP=,
+    MOZ_LOOP=1 )
+if test "$MOZ_LOOP"; then
+    AC_DEFINE(MOZ_LOOP)
+fi
+AC_SUBST(MOZ_LOOP)
+
 dnl
 dnl Always build Marionette if not Android or B2G
 dnl





More information about the tbb-commits mailing list