Merge pull request #31190 from BorisChiou/Bug952456

Bug 952456 - Add "Copy Image" option, r=timdream, kanru
mozilla-b2g · Aug 24, 2015 · 22fba58 · 22fba58
2 parents cddb9f6 + 31fbbc0
commit 22fba58
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 8 deletions.
diff --git a/apps/system/js/browser_context_menu.js b/apps/system/js/browser_context_menu.js
@@ -70,16 +70,29 @@
       var items = [];
 
       // contextmenu.items are specified by the web content via html5
-      // context menu api
+      // context menu api and a specific system command, copy-image.
       if (detail.contextmenu && detail.contextmenu.items.length) {
         detail.contextmenu.items.forEach(function(choice, index) {
-          items.push({
-            label: choice.label,
-            icon: choice.icon,
-            callback: function() {
-              detail.contextMenuItemSelected(choice.id);
-            }
-          });
+          var itemObj = null;
+          switch (choice.id) {
+            case 'copy-image':
+              itemObj = {
+                id: choice.id,
+                label: _(choice.id),
+              };
+              break;
+
+            default:
+              // Customized menu items
+              itemObj = {
+                label: choice.label,
+                icon: choice.icon,
+              };
+          }
+          itemObj.callback = () => {
+            detail.contextMenuItemSelected(choice.id);
+          };
+          items.push(itemObj);
         }, this);
       }
 

diff --git a/apps/system/locales/system.en-US.properties b/apps/system/locales/system.en-US.properties
@@ -600,6 +600,7 @@ save-link=Save linked file
 share-image=Share Image
 share-video=Share Video
 share-audio=Share Audio
+copy-image=Copy Image
 
 # Private Browser
 private-window=Private Window