Closed
Bug 1111490
Opened 10 years ago
Closed 10 years ago
[FFOS2.0][Woodduck][Camera]The videos still play after lock the screen.
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect, P2)
Firefox OS Graveyard
Gaia::Camera
Tracking
(blocking-b2g:-, b2g-v2.0 wontfix, b2g-v2.0M wontfix, b2g-v2.1 unaffected, b2g-v2.2 unaffected)
RESOLVED
WONTFIX
blocking-b2g | - |
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | wontfix |
b2g-v2.0M | --- | wontfix |
b2g-v2.1 | --- | unaffected |
b2g-v2.2 | --- | unaffected |
People
(Reporter: sync-1, Unassigned)
References
Details
Attachments
(3 files)
DEFECT DESCRIPTION:
The videos still play after lock the screen.
REPRODUCING PROCEDURES:
Enter the camera, record one video, play it, then press the power key to lock screen. the video is still playing.
EXPECTED BEHAVIOUR:
The videos should stop after lock the screen.
Comment 1•10 years ago
|
||
Hi Norry,
qawanted for Woodduck 2.0M and Flame 2.0/2.1/2.2. Thanks!
Blocks: Woodduck
Flags: needinfo?(fan.luo)
Comment 2•10 years ago
|
||
The bug can be repro on Woodduck 2.0 and Flame 2.0, it cannot be repro on Flame 2.1/2.2.
See attachments: video.MP4 & Woodduck2.0_logcat_1650.txt & Flame2.0_logcat_1416.txt
Repro time: Woodduck2.0: 16:50, Flame2.0: 14:16
Reproducing rate: 0/5
Woodduck 2.0 build:
Gaia-Rev 1f082dae21f60735e20669b4dcfddc39383b4ccf
Gecko-Rev e3dc9e5cba07c956e34f03fc0b8fcd2342d57234
Build-ID 20141216050313
Version 32.0
Flame 2.0 build:
Gaia-Rev f3b9806f687fbbd7eba6b0e1f6ebb8bde09840ea
Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/28bacbc71efb
Build-ID 20141215000201
Version 32.0
Flags: needinfo?(fan.luo)
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
Updated•10 years ago
|
status-b2g-v2.0:
--- → affected
status-b2g-v2.0M:
--- → affected
status-b2g-v2.1:
--- → unaffected
status-b2g-v2.2:
--- → unaffected
Hi
it weired,when the video is playing ,and lock the screen , the visibilitychange event can't be triggered. but stop the playing video ,and lock the screen, can trigger the visibilitychange event normally.
Comment 6•10 years ago
|
||
It is by design. It seems dup of Bug 906612.
Hi
about Bug 906612,we can't control the websites to use firefoxos APIs. but we can control the camera app here. are you sure it's by design ?
Comment 8•10 years ago
|
||
Hi Gary,
Could you please help to check the problem? Thanks!
Flags: needinfo?(gchen)
Comment 9•10 years ago
|
||
Hi Sotaro,
In my inspect, just like partner mentioned on comment #5, camera doesn't receive `visibilitychange` while preview video is playing.
But we play the same video file which was recording by camera app in gallery app, while user click `sleep` button, the gallery app will get `visibilitychange` and stop playing.
I just wonder if this is by design, why we have difference scenarios between these 2 apps? Could you help to give me some hints about this issue?
Thanks.
Flags: needinfo?(gchen) → needinfo?(sotaro.ikeda.g)
Comment 10•10 years ago
|
||
Its difference seems to come from mozAudioChannelType value. The following explains about mozAudioChannelType.
https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement.mozAudioChannelType
Gallery application seems to use /shared/js/media/video_player.js for video playback, it set 'content' to mozAudioChannelType. `mozaudiochannel="content"` allows to receive the "visibilitychange" event when the "Lock" button is pressed.
In camera app's case, it seems to have a element that block to sent visibilitychange" event when the "Lock" button is pressed.
Flags: needinfo?(sotaro.ikeda.g)
Reporter | ||
Comment 11•10 years ago
|
||
Hi
the Camera application also use /shared/js/media/video_player.js for video playback.
it's so weired.could you find what block the 'visibilitychange' event?
Updated•10 years ago
|
Flags: needinfo?(gchen)
Comment 12•10 years ago
|
||
Audio channel check that is related to lock screen is done in the following. It seems better to ask gaia camera app's engineer about it.
https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/visibility_manager.js#L107
---------------------------------------------------------
case 'lockscreen-appopened':
// If the audio is active, the app should not set non-visible
// otherwise it will be muted.
// TODO: Remove this hack.
this.debug('locking, hide the whole windows',
this._normalAudioChannelActive);
if (!this._normalAudioChannelActive) {
this.publish('hidewindow',
{ screenshoting: false, type: evt.type });
}
this._resetDeviceLockedTimer();
break;
Comment 13•10 years ago
|
||
Hi Wilson,
Could you help to clarify why the behavior isn't consistent between v2.0 and v2.1?
Thanks.
Flags: needinfo?(gchen) → needinfo?(wilsonpage)
Comment 14•10 years ago
|
||
IIRC we fixed an issue to do with `mozAudioChannel` value preventing the app from receiving `visibilitychange` events. justindarc worked quite closely with seom platform devs to fix the bug. I'm guessing the patch wasn't uplifted to v2.0.
NU justin to see if he remembers anything about this.
Flags: needinfo?(wilsonpage) → needinfo?(jdarcangelo)
Reporter | ||
Comment 15•10 years ago
|
||
Hi
is there any news about this bug ?
Comment 16•10 years ago
|
||
Communicate with partner and agree accept 2.0M behavior.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Updated•10 years ago
|
Updated•10 years ago
|
Blocks: Woodduck_P2
You need to log in
before you can comment on or make changes to this bug.
Description
•