1079761 - add 'stop tab mirroring' one level higher in the menu

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Screencasting, defect)

Description

[Karen Rudnitski [:kar]]

Just like we do with guest browsing, can we:

- keep the function to start tab mirroring in the menu - tools menu, BUT
- bring the function to stop up one level higher in the menu (so you don't need to go back into tools to stop it)

Comment 1

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

Chenxia - Simple bug. Just move the menu.

Comment 2

[Chenxia Liu [:liuche]]

Attachment: Patch: tab mirroring

I can't actually test this because I don't have a roku/chromecast to send things to. Here's an apk:

http://people.mozilla.org/~liuche/bug-1072831/tab-mirror.apk

Comment 3

[Chenxia Liu [:liuche]]

Wes, let me know if you managed to get casting working, and if you can see if this patch/build is working though rbarker mentioned that chromecast might need bug 1080012, bug 1048335, and bug 1080701 to work now, which is not in my test build's patch queue.

Comment 4

[Wesley Johnston (:wesj)]

Comment on attachment 8502754 [details] [diff] [review]
Patch: tab mirroring

Randall sorta owns this now

Comment 5

[Randall Barker [:rbarker]]

Comment on attachment 8502754 [details] [diff] [review]
Patch: tab mirroring

Review of attachment 8502754 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/chrome/content/CastingApps.js
@@ +135,5 @@
>            }
>            NativeWindow.menu.update(this.mirrorStartMenuId, { visible: true });
>            NativeWindow.menu.update(this.mirrorStopMenuId, { visible: false });
>          }.bind(this),
> +        parent: NativeWindow.menu

This didn't work for me, the "stop mirror" option did not get inserted into any menu. :wesj suggested just removing this line and that seems to have the desired effect.

Comment 6

[Randall Barker [:rbarker]]

Attachment: Patch: tab mirroring v2

Updated patch.

Comment 7

[Chenxia Liu [:liuche]]

Thanks, Randall! Landed:

https://hg.mozilla.org/integration/fx-team/rev/b072d6d6d730

Comment 8

[Chenxia Liu [:liuche]]

Comment on attachment 8504203 [details] [diff] [review]
Patch: tab mirroring v2

Approval Request Comment
[Feature/regressing bug #]: Make exiting Guest mode easier
[User impact if declined]: To exit guest mode from menu, user needs to go down two menus, instead of just one
[Describe test coverage new/current, TBPL]: local testing
[Risks and why]: very very low, one-line change in deleting hierarchy
[String/UUID change made/needed]: none

Comment 9

[Chenxia Liu [:liuche]]

Comment on attachment 8504203 [details] [diff] [review]
Patch: tab mirroring v2

Derp, I meant tab casting, not guest session.

Approval Request Comment
[Feature/regressing bug #]: Make exiting Tab Mirroring easier
[User impact if declined]: To stop sharing a tab, user needs to go down two menus, instead of just one
[Describe test coverage new/current, TBPL]: local testing
[Risks and why]: very very low, one-line change in deleting hierarchy
[String/UUID change made/needed]: none

Comment 10

[Ryan VanderMeulen [:RyanVM]]

This landed with the wrong bug number in the commit message. Please double-check that in the future. :)
https://hg.mozilla.org/mozilla-central/rev/b072d6d6d730

Comment 11

[Chenxia Liu [:liuche]]

Comment on attachment 8504203 [details] [diff] [review]
Patch: tab mirroring v2

Approval Request Comment
[Feature/regressing bug #]: Make exiting Tab Mirroring easier
[User impact if declined]: To stop sharing a tab, user needs to go down two menus, instead of just one
[Describe test coverage new/current, TBPL]: local testing, fx-team green
[Risks and why]: very very low, one-line change in deleting hierarchy
[String/UUID change made/needed]: none

Comment 12

[Chenxia Liu [:liuche]]

Attachment: Patch: tab mirroring

Updating patch message because it had the wrong bug number. Carrying over r+.

Approval Request Comment
[Feature/regressing bug #]: Make exiting Tab Mirroring easier
[User impact if declined]: To stop sharing a tab, user needs to go down two menus, instead of just one
[Describe test coverage new/current, TBPL]: local testing, fx-team green
[Risks and why]: very very low, one-line change in deleting hierarchy
[String/UUID change made/needed]: none

Comment 13

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8505696 [details] [diff] [review]
Patch: tab mirroring

Beta+
Aurora+

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/daf0fac79aa0
https://hg.mozilla.org/releases/mozilla-beta/rev/4018d170ab06

Comment 15

[Joni Chan]

I've updated both SUMO articles to reflect the new location: https://support.mozilla.org/kb/send-webpages-your-tv-firefox-android-chromecast and https://support.mozilla.org/kb/view-webpages-on-tv-roku-and-firefox-android

Did the label used to say "Stop mirroring" or am I mistaken? I think it sounds better than "Stop mirror" but the meaning's clear either way.

Comment 16

[Flaviu Cos]

Verified as fixed in builds:
- 36.0a1 2014-11-25;
- 35.0a2 2014-11-25;
- 34 RC1;
Device: Nexus 4 (Android 4.4.4).