Merge branch 'appid-guards'

1 files changed, 30 insertions, 30 deletions

diff --git a/js/vapi-background.js b/js/vapi-background.js
index cca47c6..08a84bc 100644
--- a/js/vapi-background.js
+++ b/js/vapi-background.js
@@ -48,8 +48,9 @@ const {Services} = Cu.import('resource://gre/modules/Services.jsm', null);
 
 var vAPI = self.vAPI = self.vAPI || {};
 vAPI.firefox = true;
-vAPI.modernFirefox = Services.appinfo.ID === '{ec8030f7-c20a-464f-9b0e-13a3a9e97384}' &&
-                     Services.vc.compare(Services.appinfo.platformVersion, '44') > 0;
+    vAPI.modernFirefox =
+	Services.appinfo.ID === '{ec8030f7-c20a-464f-9b0e-13a3a9e97384}'
+	&& Services.vc.compare(Services.appinfo.version, '44') > 0;
 
 /******************************************************************************/
 
@@ -2362,24 +2363,16 @@ vAPI.toolbarButton = {
 
 /******************************************************************************/
 
-// Firefox 28 and less
-
 (function() {
+    // Add toolbar button for not-Basilisk
+    if (Services.appinfo.ID === "{ec8030f7-c20a-464f-9b0e-13a3a9e97384}") {
+	return;
+    }
+    
     var tbb = vAPI.toolbarButton;
     if ( tbb.init !== null ) {
         return;
     }
-    var CustomizableUI = null;
-    var forceLegacyToolbarButton = vAPI.localStorage.getBool('forceLegacyToolbarButton');
-    if ( !forceLegacyToolbarButton ) {
-        try {
-            CustomizableUI = Cu.import('resource:///modules/CustomizableUI.jsm', null).CustomizableUI;
-        } catch (ex) {
-        }
-    }
-    if ( CustomizableUI !== null ) {
-        return;
-    }
 
     tbb.codePath = 'legacy';
     tbb.viewId = tbb.id + '-panel';
@@ -2604,17 +2597,19 @@ vAPI.toolbarButton = {
 
 /******************************************************************************/
 
-// Firefox Australis < 36.
-
+/*
 (function() {
+    // Add toolbar button for Basilisk
+    if (Services.appinfo.ID !== "{ec8030f7-c20a-464f-9b0e-13a3a9e97384}") {
+	return;
+    }
+    
     var tbb = vAPI.toolbarButton;
     if ( tbb.init !== null ) {
         return;
     }
-    if ( Services.vc.compare(Services.appinfo.platformVersion, '36.0') >= 0 ) {
-        return null;
-    }
-    if ( vAPI.localStorage.getBool('forceLegacyToolbarButton') ) {
+
+    if ( Services.vc.compare(Services.appinfo.version, '36.0') >= 0 ) {
         return null;
     }
     var CustomizableUI = null;
@@ -2675,7 +2670,7 @@ vAPI.toolbarButton = {
                 CustomizableUI.getWidget(tbb.id).areaType === CustomizableUI.TYPE_MENU_PANEL
             );
         } catch (ex) {
-            /* noop */
+            // noop
         }
     };
 
@@ -2731,25 +2726,30 @@ vAPI.toolbarButton = {
         cleanupTasks.push(shutdown);
     };
 })();
+*/
 
 /******************************************************************************/
 
-// Firefox Australis >= 36.
-
 (function() {
     // It appears that this branch actually works on the latest
     // Basilisk.  Maybe we can simply use this one directly instead of
     // making checks like it's done now.
+
+    // It was decided to use this branch unconditionally. It's still
+    // experimental though.
+
+    // Add toolbar button for Basilisk
+    if (Services.appinfo.ID !== "{ec8030f7-c20a-464f-9b0e-13a3a9e97384}") {
+	return;
+    }
+    
     var tbb = vAPI.toolbarButton;
     if ( tbb.init !== null ) {
         return;
     }
-    if ( Services.vc.compare(Services.appinfo.platformVersion, '36.0') < 0 ) {
-        return null;
-    }
-    if ( vAPI.localStorage.getBool('forceLegacyToolbarButton') ) {
-        return null;
-    }
+    // if ( Services.vc.compare(Services.appinfo.version, '36.0') < 0 ) {
+    //     return null;
+    // }
     var CustomizableUI = null;
     try {
         CustomizableUI = Cu.import('resource:///modules/CustomizableUI.jsm', null).CustomizableUI;