1046971 - Add ability to stop mirroring a tab

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Screencasting, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: stop_mirror.patch

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8465679 [details] [diff] [review]
stop_mirror.patch

>diff --git a/mobile/android/chrome/content/CastingApps.js b/mobile/android/chrome/content/CastingApps.js

> var CastingApps = {
>   _castMenuId: -1,
>-  menuItem: -1,
>+  mirrorMenuItem: -1,
>+  stopMenuItem: -1,

Let's just use: startMenuId and stopMenuId


>+      this.stopMenuItem = NativeWindow.menu.add(
>+        Strings.browser.GetStringFromName("casting.mirrorTabStop"),
>+        "drawable://casting",
>+        function() {
>+          if (this.tabMirror) {
>+            this.tabMirror.stop();
>+            this.tabMirror = null;
>+          }
>+          NativeWindow.menu.update(this.mirrorMenuItem, { visible: true });
>+          NativeWindow.menu.update(this.stopMenuItem, { visible: false });
>+        }.bind(this));
>     }

Remember to use the new syntax and place it in the Tools menu

>diff --git a/mobile/android/modules/TabMirror.jsm b/mobile/android/modules/TabMirror.jsm

>     }
>   }
>+  return { stop: function() { let obj = {

* Add a blank line before the "return"
* Add line breaks after the { braces like

    return {
      stop: function() {
        let obj = {

And indent the rest appropriately

>+    if (deviceId) {
>+      Services.androidBridge.handleGeckoMessage(obj);
>+    } } }

Same line breaks here

r+ but make sure this applies over the patch you already have to hide the menu

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

this landed http://hg.mozilla.org/mozilla-central/rev/8d7b53cc415e

Comment 3

[Flaviu Cos]

Verified as fixed in build 34.0a1 (2014-08-19);
Device: Motorola Razr (Android 4.1.2).