1005861 - [Toolkit] Cannot auto-resize built-in media player to fit screen size

Status: RESOLVED WONTFIX

(Toolkit :: Video/Audio Controls, defect)

Description

[William Hsu [:whsu]]

Attachment: 2014-05-05-18-43-52.png

* Description:
  This problem happens on v2.0 build.
  After built-in media player start playing video RTSP streaming, you can see
  (1) the size of built-in media player is small and browser cannot adjust it to fit screen's size.
  (2) the screen of video RTSP is behind the toolbar.
  We may need UX's suggestion to see if we need to adjust the size of built-in media player to fit screen's size.
  Attach the screenshot.(2014-05-05-18-43-52.png)

* Reproduction steps:
  1. Connect to a hotspot
  2. Launch the following page via browser
     - http://goo.gl/FyHFNs
  3. Tap the "Video test page (3GP)"
  4. After FxOS launches the video to play RTSP stream, please check the size of built-in media player

* Expected result:
  Browser resizes built-in media player to fit screen size

* Actual result:
  The size of built-in media player is small

* Reproduction build: V2.0 (Buri)
 - Gaia      f55fc5c507312c7aac51ec9bb73061fd4ed5c5fb
 - Gecko     https://hg.mozilla.org/mozilla-central/rev/3285e030d9c0
 - BuildID   20140504160202
 - Version   32.0a1

Comment 1

[Ethan Tseng [:ethan]]

Does any one know who is the owner or peer of built-in media player?

Comment 2

[howie [:howie]]

Hi Hema, might be media player or browser issue, can you take a look first? Thanks.

Comment 3

[Ken Chang[:kenkai]]

Should we change the component?

Comment 4

[Ethan Tseng [:ethan]]

(In reply to Ken Chang[:ken] from comment #3)
> Should we change the component?
Absolutely. The question is, which component is the right one?
I don't have an answer right now, but had proposed some possible ones.
Please refer to:
https://bugzilla.mozilla.org/show_bug.cgi?id=1005865#c4
https://bugzilla.mozilla.org/show_bug.cgi?id=1005854#c2

Comment 5

[howie [:howie]]

This should be built-in media player's issue. Hi Hema, could you re-assign to the right person to solve it? thank you.

Comment 6

[Hema Koka [:hema]]

Moving this to Toolkit > Video/Audio Controls component for triage and fix. If I got the component wrong, please correct me.

Thanks
Hema

Comment 7

[Candice Serran (:cserran)]

If toolkit is looking into this, I'll wait to see if this is a browser issue.  Please ni me if that is still in question...thanks

Comment 8

[Ken Chang[:kenkai]]

Howie, do you talk with toolkit dev?

Comment 9

[howie [:howie]]

Hi Dave, this issue relates to a major feature: RTSP video streaming for FF OS 2.0. Could you take a look and re-assign to the right person? Thanks.

Comment 10

[Dave Townsend [:mossop]]

Dolske, can you take this or assign to someone on your team?

Comment 11

[Justin Dolske [:Dolske]]

I wasn't aware B2G used toolkit code for this. Doesn't look like the touch controls on Android. Wes?

Comment 12

[Wesley Johnston (:wesj)]

B2G does (although most of their apps override the built in controls. These bugs are making me re-prioritize the rewrite work. I can dig into that again (bug 704229).

Comment 13

[Marvin Khoo [:Marvin_Khoo]]

Hi Howie, who can take this bug? can we assign a owner here, as we don't have much time now. this is 2.0 blocker.

Comment 14

[howie [:howie]]

Hi Wesley, could you take this for now? It's a Firefox OS 2.0 blocker and we kindly need your help here. Thank you.

Comment 15

[howie [:howie]]

ni Mark also

Comment 16

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

Firefox for Android does use the touch-centric video controls, but we really only work on that code to fix bugs in Firefox for Android. We try to avoid doing work that might affect Desktop or Firefox OS.

Each time we need to work on the video controls code, we try to limit the amount of time we spend on the work. We really don't have resources to treat the controls as common toolkit code.

With this in mind, I don't expect to assign someone to this work. But we can help try to figure out the issue and offer some assistance and direction in trying to get it fixed.

* Is this a regression? If it is a regression, what caused it. If not a regression, then...
* Why is this a blocker for 2.0?

Comment 17

[Marvin Khoo [:Marvin_Khoo]]

> * Why is this a blocker for 2.0?

Hi Mark, we have customer request for Audio and Video RTSP, since we are moving playback to system browser, thus, these UI issues has to be fixed.

Comment 18

[William Hsu [:whsu]]

Modify the title because it belongs to toolkit component.

Comment 19

[Marvin Khoo [:Marvin_Khoo]]

per team discussion, since this related to auto-resize,thus, i don't see is is a blocker. sorry for the confusion. 

Hi Howie, kindly remove blocking 2.0? thanks!

Comment 20

[Jared Wein [:jaws] (please needinfo? me)]

Since this is specific to B2G on mobile, can we close this as wontfix?

Comment 21

[Ethan Tseng [:ethan]]

Close this bug because:
1) No one is working on RTSP for b2g right now.
2) The demand is pretty low or even none (rare use cases in using RTSP on mobile devices).