Bug 1260304 - Remove mediasource string from context menu on Android. r?margaret draft
authorRay Lin <ralin@mozilla.com>
Mon, 18 Apr 2016 15:26:21 +0800
changeset 352966 b1f4440877ec335e2255858f214c53453587ca96
parent 352861 67ac40fb8f680ea5e03805552187ba1b5e8392a1
child 518797 a94231b235489c958d33f969f4d7f98bdd02ff57
push id15854
push userbmo:ralin@mozilla.com
push dateTue, 19 Apr 2016 02:51:25 +0000
reviewersmargaret
bugs1260304
milestone48.0a1
Bug 1260304 - Remove mediasource string from context menu on Android. r?margaret MozReview-Commit-ID: 5BdIUzgkTV
mobile/android/chrome/content/browser.js
--- a/mobile/android/chrome/content/browser.js
+++ b/mobile/android/chrome/content/browser.js
@@ -2647,17 +2647,19 @@ var NativeWindow = {
       } else if (node instanceof Ci.nsIImageLoadingContent && node.currentURI) {
         // The image is blocked by Tap-to-load Images
         let originalURL = node.getAttribute("data-ctv-src");
         if (originalURL) {
           return originalURL;
         }
         return node.currentURI.spec;
       } else if (node instanceof Ci.nsIDOMHTMLMediaElement) {
-        return (node.currentSrc || node.src);
+        let srcUrl = node.currentSrc || node.src;
+        // If URL prepended with blob or mediasource, we'll remove it.
+        return srcUrl.replace(/^(?:blob|mediasource):/, '');
       }
 
       return "";
     },
 
     // Adds an array of menuitems to the current list of items to show, in the correct context
     _addMenuItems: function(items, context) {
         if (!this.menus[context]) {