Bug 1010771 - Check for favicon field. r=me

mozilla-b2g · May 23, 2014 · 9729730 · 9729730
1 parent 30e0a8a
commit 9729730
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/apps/system/js/browser_context_menu.js b/apps/system/js/browser_context_menu.js
@@ -241,15 +241,14 @@
   };
 
   BrowserContextMenu.prototype.showDefaultMenu = function() {
+    var config = this.app.config;
+    var icon = ('favicon' in config) ? config.favicon.href : null;
     this.showMenu([{
       label: _('add-to-home-screen'),
-      callback: this.bookmarkUrl.bind(this,
-                                      this.app.config.url,
-                                      this.app.title,
-                                      this.app.config.favicon.href)
+      callback: this.bookmarkUrl.bind(this, config.url, this.app.title, icon)
     }, {
       label: _('share'),
-      callback: this.shareUrl.bind(this, this.app.config.url)
+      callback: this.shareUrl.bind(this, config.url)
     }]);
   };