Bug 1350842 - part3 : remove logic about handling 'OpenMediaWithExternalApp' event in browser.js. draft
authorAlastor Wu <alwu@mozilla.com>
Tue, 15 Aug 2017 17:48:39 +0800
changeset 647195 c956714771d293ab351e2158c36d2a86b2c68c34
parent 647194 f73a017ee64aa7d15b04de224a3b4686155003bb
child 647196 86a1e57d135c01abbf747734eb8990c9cfd7109e
push id74315
push useralwu@mozilla.com
push dateWed, 16 Aug 2017 02:40:23 +0000
bugs1350842
milestone57.0a1
Bug 1350842 - part3 : remove logic about handling 'OpenMediaWithExternalApp' event in browser.js. MozReview-Commit-ID: Ev9qV0iBUzb
mobile/android/chrome/content/browser.js
--- a/mobile/android/chrome/content/browser.js
+++ b/mobile/android/chrome/content/browser.js
@@ -4698,17 +4698,16 @@ Tab.prototype = {
 };
 
 var BrowserEventHandler = {
   init: function init() {
     BrowserApp.deck.addEventListener("touchend", this, true);
 
     BrowserApp.deck.addEventListener("DOMUpdatePageReport", PopupBlockerObserver.onUpdatePageReport);
     BrowserApp.deck.addEventListener("MozMouseHittest", this, true);
-    BrowserApp.deck.addEventListener("OpenMediaWithExternalApp", this, true);
 
     // ReaderViews support backPress listeners.
     WindowEventDispatcher.registerListener((event, data, callback) => {
       callback.onSuccess(Reader.onBackPress(BrowserApp.selectedTab.id));
     }, "Browser:OnBackPressed");
   },
 
   handleEvent: function(aEvent) {
@@ -4716,26 +4715,16 @@ var BrowserEventHandler = {
       case 'touchend':
         if (this._inCluster) {
           aEvent.preventDefault();
         }
         break;
       case 'MozMouseHittest':
         this._handleRetargetedTouchStart(aEvent);
         break;
-      case 'OpenMediaWithExternalApp': {
-        let mediaSrc = aEvent.target.currentSrc || aEvent.target.src;
-        let uuid = uuidgen.generateUUID().toString();
-        GlobalEventDispatcher.sendRequest({
-          type: "Video:Play",
-          uri: mediaSrc,
-          uuid: uuid
-        });
-        break;
-      }
     }
   },
 
   _handleRetargetedTouchStart: function(aEvent) {
     // we should only get this called just after a new touchstart with a single
     // touch point.
     if (!BrowserApp.isBrowserContentDocumentDisplayed() || aEvent.defaultPrevented) {
       return;