Bug 1301055 - part1 : allow to replay the same video again. r=snorp
authorAlastor Wu <alwu@mozilla.com>
Thu, 24 Nov 2016 10:24:02 +0800
changeset 324039 a66b7c936f76e9ea418855f9c0a6cf477a5e2e1d
parent 324038 e9872fff04e29255714658452af04f58e885747d
child 324040 7794b64cae28ffe697e1844c7dcd7faca86700d1
push id24
push usermaklebus@msu.edu
push dateTue, 20 Dec 2016 03:11:33 +0000
reviewerssnorp
bugs1301055
milestone53.0a1
Bug 1301055 - part1 : allow to replay the same video again. r=snorp MozReview-Commit-ID: LNz6vEeMZOL
mobile/android/chrome/content/browser.js
--- a/mobile/android/chrome/content/browser.js
+++ b/mobile/android/chrome/content/browser.js
@@ -4515,27 +4515,22 @@ var BrowserEventHandler = {
         if (this._inCluster) {
           aEvent.preventDefault();
         }
         break;
       case 'MozMouseHittest':
         this._handleRetargetedTouchStart(aEvent);
         break;
       case 'OpenMediaWithExternalApp': {
-        if (aEvent.target.moz_video_uuid) {
-          return;
-        }
         let mediaSrc = aEvent.target.currentSrc || aEvent.target.src;
-        if (!aEvent.target.moz_video_uuid) {
-          aEvent.target.moz_video_uuid = uuidgen.generateUUID().toString();
-        }
+        let uuid = uuidgen.generateUUID().toString();
         Services.androidBridge.handleGeckoMessage({
           type: "Video:Play",
           uri: mediaSrc,
-          uuid: aEvent.target.moz_video_uuid
+          uuid: uuid
         });
         break;
       }
     }
   },
 
   _handleRetargetedTouchStart: function(aEvent) {
     // we should only get this called just after a new touchstart with a single