From f20372cf73f4a9908b4e6810ce4e06e6133928c1 Mon Sep 17 00:00:00 2001
From: Deathamns <deathamns@gmail.com>
Date: Tue, 17 Feb 2015 22:05:23 +0100
Subject: [PATCH] Remove unused code

---
 platform/firefox/install.rdf        |  9 +++++++++
 platform/firefox/vapi-background.js | 27 +++++++--------------------
 2 files changed, 16 insertions(+), 20 deletions(-)

diff --git a/platform/firefox/install.rdf b/platform/firefox/install.rdf
index e186b5b53..38337c2ee 100644
--- a/platform/firefox/install.rdf
+++ b/platform/firefox/install.rdf
@@ -32,6 +32,15 @@
             </r:Description>
         </targetApplication>
 
+        <!-- Firefox for Android
+        <targetApplication>
+            <r:Description>
+                <id>{{aa3c5121-dab2-40e2-81ca-7ea25febc110}}</id>
+                <minVersion>24.0</minVersion>
+                <maxVersion>39.0</maxVersion>
+            </r:Description>
+        </targetApplication> -->
+
         <!-- Thunderbird
         <targetApplication>
             <r:Description>
diff --git a/platform/firefox/vapi-background.js b/platform/firefox/vapi-background.js
index de216d187..9dea03ea2 100644
--- a/platform/firefox/vapi-background.js
+++ b/platform/firefox/vapi-background.js
@@ -792,17 +792,17 @@ vAPI.messaging.broadcast = function(message) {
 
 // This allows to avoid creating a closure for every single message which
 // expects an answer. Having a closure created each time a message is processed
-// has been always bothering me. Another benefit of the implementation here 
+// has been always bothering me. Another benefit of the implementation here
 // is to reuse the callback proxy object, so less memory churning.
 //
 // https://developers.google.com/speed/articles/optimizing-javascript
 // "Creating a closure is significantly slower then creating an inner
-//  function without a closure, and much slower than reusing a static 
+//  function without a closure, and much slower than reusing a static
 //  function"
 //
 // http://hacksoflife.blogspot.ca/2015/01/the-four-horsemen-of-performance.html
-// "the dreaded 'uniformly slow code' case where every function takes 1% 
-//  of CPU and you have to make one hundred separate performance optimizations 
+// "the dreaded 'uniformly slow code' case where every function takes 1%
+//  of CPU and you have to make one hundred separate performance optimizations
 //  to improve performance at all"
 //
 // http://jsperf.com/closure-no-closure/2
@@ -844,9 +844,9 @@ CallbackWrapper.prototype.proxy = function(response) {
     }
 
     // Mark for reuse
-    this.messageManager = 
-    this.channelName = 
-    this.requestId = 
+    this.messageManager =
+    this.channelName =
+    this.requestId =
     this.listenerId = null;
     CallbackWrapper.junkyard.push(this);
 };
@@ -1556,19 +1556,6 @@ vAPI.contextMenu.create = function(details, callback) {
 
 /******************************************************************************/
 
-vAPI.contextMenu.remove = function() {
-    for ( var win of vAPI.tabs.getWindows() ) {
-        this.unregister(win.document);
-    }
-
-    this.menuItemId = null;
-    this.menuLabel = null;
-    this.contexts = null;
-    this.onCommand = null;
-};
-
-/******************************************************************************/
-
 vAPI.lastError = function() {
     return null;
 };