Merge pull request #20504 from jimporter/ringtones-hide-share-from-self

Bug 1015513 - [ringtones] An app shouldn't be able to pick itself when l...

apps/ringtones/js/actions_menu.js

@@ -80,6 +80,8 @@ ActionsMenu.prototype = {
         name: 'share',
         data: {
           type: 'audio/*',
+          // Make sure we can't share with ourselves!
+          __bug1015513_hide_from_self__: true,
           number: 1,
           blobs: [blob],
           filenames: [self._tone.name],

apps/ringtones/manifest.webapp

@@ -31,6 +31,7 @@
           "required": true,
           "pattern": "^audio/.*$"
         },
+        "__bug1015513_hide_from_self__": false,
         "number": 1
       },
       "disposition": "inline",

apps/ringtones/test/marionette/lib/ringtones.js

@@ -76,6 +76,7 @@ ActionsMenu.prototype = {
         return list[i];
       }
     }
+    return null;
   }
 };
 createElementGetters(

apps/ringtones/test/marionette/manage_test.js

@@ -140,6 +140,12 @@ marionette('Ringtone management', function() {
         // the right data was sent...
         var actionsMenu = container.soundLists[0].sounds[0].openActions();
         actionsMenu.shareButton.tap();
+
+        // Make sure we can't share with ourselves, since that's weird.
+        assert.equal(actionsMenu.shareWith('Ringtones'), null,
+                     'Ringtones app shouldn\'t be in Share menu');
+
+        // Share with the messages app.
         actionsMenu.shareWith('Messages').tap();
       });
     });