Closed
Bug 1199985
Opened 9 years ago
Closed 9 years ago
[TV 2.5][Browser] Failed to play a video on Youtube or CNN
Categories
(Firefox OS Graveyard :: AudioChannel, defect, P1)
Tracking
(blocking-b2g:2.5+, b2g-master affected)
RESOLVED
DUPLICATE
of bug 1214148
blocking-b2g | 2.5+ |
Tracking | Status | |
---|---|---|
b2g-master | --- | affected |
People
(Reporter: cynthiatang, Unassigned)
References
Details
(Whiteboard: [ft:conndevices][partner-blocker])
STR:
1. Launch Browser App
2. Go to Youtube or CNN
3. Watch a video on Youtube or CNN
Actual:
- User cannot play a video on Youtube or CNN
Expected:
- User can watch a video on Youtube or CNN
Reporter | ||
Comment 1•9 years ago
|
||
Hi Josh,
This issue can be reproduced in Nexus 5 (TV build). Thank you
Flags: needinfo?(jocheng)
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(jocheng)
Resolution: --- → DUPLICATE
Reporter | ||
Updated•9 years ago
|
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 3•9 years ago
|
||
Hi Cynthia,
Can you provide a build which cause the problem to Blake?
Thanks!
Blocks: TV_FxOS2.5
blocking-b2g: --- → 2.5+
status-b2g-master:
--- → affected
Flags: needinfo?(ctang)
Priority: -- → P1
Whiteboard: [ft:conndevices][partner-blocker]
Reporter | ||
Comment 4•9 years ago
|
||
Hi Blake,
Could you help to find someone who can help check?
Thanks!
Device Nexus5
Build ID 20150917101726
Gaia Revision 46f54ba279858b677793f78b7465b3a82f799eef
Gaia Date 2015-09-17 00:49:40
Gecko Revision n/a
Gecko Version 43.0a1
Device Name hammerhead
Firmware(Release) 5.1
Firmware(Incremental) eng.cltbld.20150721.211608
Firmware Date Tue Jul 21 21:16:23 EDT 2015
Bootloader HHZ12f
Flags: needinfo?(ctang) → needinfo?(bwu)
Comment 5•9 years ago
|
||
Hi Cynthia,
Can this be reproduced by non-TV version?
Flags: needinfo?(bwu) → needinfo?(ctang)
Reporter | ||
Comment 6•9 years ago
|
||
Hi Blake,
This issue cannot be reproduced by non-TV version. Thanks!
(In reply to Blake Wu [:bwu][:blakewu] from comment #5)
> Hi Cynthia,
> Can this be reproduced by non-TV version?
Flags: needinfo?(ctang)
Reporter | ||
Comment 7•9 years ago
|
||
Hi Kilik,
Could you please take a look at this issue if you have time? Thank you.
Flags: needinfo?(kikuo)
Comment 8•9 years ago
|
||
Hi Cynthia, I'm kinda stuck in refactoring video sink stuff, I'll ask James for help.
Hi James, could you help me investigate this issue ? I've got the devices. You can have a look on it.
Thanks
Flags: needinfo?(kikuo) → needinfo?(jacheng)
Comment 9•9 years ago
|
||
OK, I will try it on b2g desktop for TV build first to see if I can reproduce this issue.
Flags: needinfo?(jacheng)
Comment 10•9 years ago
|
||
We can reproduce this issue on b2g desktop TV build.
But if you press the |mute| icon then refresh. The video can be played normally.
If you press |unmute| when playing, the playback will stop immediately.
Comment 11•9 years ago
|
||
Currently we can set pref("dom.audiochannel.mutedByDefault", true); to false to avoid this issue occur both on B2G desktop for TV and Nexus5 for TV build.
The root cause is that the AudioChannel policy has not back-ported yet on TV's system app(2.1).
If finish porting, this issue should be fixed.
Comment 12•9 years ago
|
||
HTMLMediaElement::NotifyAudioChannelAgent(...) {
mAudioChannelAgent->NotifyStartedPlaying(notify, &volume, &muted);
will get the muted value from pref("dom.audiochannel.mutedByDefault", true);
If mutedByDefault == true, it will fire a event to system app and wait for system app to callback for resuming the playback.
If system app did not handle the event and never response the callback, the playback will block forever.
If TV's system app is back-ported, this issue should be fixed.
Updated•9 years ago
|
Comment 13•9 years ago
|
||
So here is the actual problem:
On TV, Browser is an app (different from phone which is part of System app), using mozbrowsr-iframe embedded by System app. Browser app uses mozbrowsr-iframe embeds other web pages. Following the architecture of audio channel control, the audiochannelstatechanged event from web content (CNN website, in this case) and other events should be handled by Browser app. Similar cases like Youtube app embeds youtube site, Marketplace app embeds Marketplaces site, ... etc. It's common in FxOS to have nested (3-layer or more) mozbrowser-iframe to load a video content, but we can't reply on a 3rd party app correctly managing audio channel policy. It looks like we need to tweak the whole mechanism a bit.
We haven't had a concrete and proper solution yet, so I filed bug 1214148 to follow-up.
In this bug, we want a quick hack to address this problem. Since System app is the central control of all audio channel change events, we could bypass the middle layer of mozbroswer-iframes. Therefore the event target will be always an iframe element owned by System app, no matter the requesting video element is embedded in the window of this iframe or its child iframe. System app controls the app and its children, as a whole.
:baku, we had a offline discussion with the Audio team in Taipei, may I have your opinion? Thanks. :)
@Kanru, could you comment here on the implementation details of tracing mozbrowser iframe arch in OOP and non-OOP cases? Thanks!
Updated•9 years ago
|
Flags: needinfo?(kchen)
Flags: needinfo?(amarchesini)
Comment 14•9 years ago
|
||
So it seems that we are going to fix bug 1214148 for a more correct solution instead of workaround here.
@Josh, this bug is equivalent to bug 1214148, we should set it a dup, but I keep it open in case :baku couldn't fixed that bug in 2.5 timeline.
Updated•9 years ago
|
Component: Gaia::TV → AudioChannel
Updated•9 years ago
|
Flags: needinfo?(kchen)
Comment 15•9 years ago
|
||
(In reply to Evelyn Hung [:evelyn] from comment #14)
> So it seems that we are going to fix bug 1214148 for a more correct solution
> instead of workaround here.
>
> @Josh, this bug is equivalent to bug 1214148, we should set it a dup, but I
> keep it open in case :baku couldn't fixed that bug in 2.5 timeline.
Since bug 1214148 is set to 2.5 blocker and it would be better not to have two 2.5 blockers pointing to the same problem, I would like to close this one as a duplicate bug.
Evelyn and Josh,
Please feel free to reopen this one if bug 1214148 cannot be fixed or this bug is not a duplicate of bug 1214148.
Thanks!
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → DUPLICATE
Updated•9 years ago
|
Flags: needinfo?(amarchesini)
You need to log in
before you can comment on or make changes to this bug.
Description
•