Closed
Bug 1365582
Opened 7 years ago
Closed 7 years ago
Audio&Video playback does not pause when Firefox is closed
Categories
(Firefox for Android Graveyard :: Audio/Video, defect)
Tracking
(firefox53 unaffected, firefox54 unaffected, firefox55 verified)
VERIFIED
FIXED
Firefox 55
Tracking | Status | |
---|---|---|
firefox53 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | verified |
People
(Reporter: bsurd, Assigned: jchen)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
ahunt
:
review+
|
Details | Diff | Splinter Review |
Device:
- Nexus 5 (Android 6.0.1);
- Galaxy Note 5 (Android 5.1).
Build:
- Nightly 55.0a1 (2017-05-17);
Steps to reproduce:
1. Open Firefox and go to: https://goo.gl/49FJnb;
2. After video starts playing open task view and close Firefox;
Expected result:
Audio playback should stop since Firefox has been closed.
Actual result:
Audio playback does not stop. The user has to open Firefox and close it again in order to fix the issue.
Notes:
Video: https://goo.gl/aDZe3H
Comment 1•7 years ago
|
||
Bogdan,
Thanks for firing this bug. I can repro this bug on my Pixel wit the latest nightly, but not on the previous nightly. This regression should be introduced recently.
Could you help find out the regression window?
Thanks.
Flags: needinfo?(bogdan.surd)
Keywords: regressionwindow-wanted
Reporter | ||
Comment 2•7 years ago
|
||
Sure thing Blake,
Here is the reggresionwindow:
First known bad build: 2017-05-14
Last known good build: 2017-05-11
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=838652a84b76c273e084d0705f3f4f3be89520a8&tochange=73b3fc64525b6816842c737e104ef2ac5482d217
Notes:
There were no Nightly builds in 12-13.
Flags: needinfo?(bogdan.surd)
Reporter | ||
Comment 3•7 years ago
|
||
Hello again,
My bad there was a Nightly build in 13, I've just missed it in the archive.
Updated window:
First known bad build: 2017-05-14
Last known good build: 2017-05-13
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=838652a84b76c273e084d0705f3f4f3be89520a8&tochange=73b3fc64525b6816842c737e104ef2ac5482d217
Reporter | ||
Updated•7 years ago
|
Keywords: regressionwindow-wanted
Comment 4•7 years ago
|
||
I can reproduce it on Pixel.
The issue is not directly related to media. From what I observed, in good version, SIGKILL(9) was sent and the whole browser is killed, but in bad version there is no signal sent.
Jim, you made some changes to the shutdown code in bug 1362191. Could it be the cause?
Flags: needinfo?(nchen)
Updated•7 years ago
|
Keywords: regression
Assignee | ||
Comment 5•7 years ago
|
||
Before bug 1362191, we always killed our own process when BrowserApp is destroyed. We no longer do that after bug 1362191, and we now rely on Android to kill our process. However, in this case the media control service is blocking Android from killing the Fennec process, so I think the best fix is to have media control service not block us from being killed by Android.
Blocks: 1362191
Flags: needinfo?(nchen)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•7 years ago
|
||
Stop the media control service when our task is removed, so that the
service doesn't block Android from killing our process.
Attachment #8869573 -
Flags: review?(ahunt)
Comment 7•7 years ago
|
||
Comment on attachment 8869573 [details] [diff] [review]
Stop MediaControlService on task removal (v1)
Review of attachment 8869573 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm!
Attachment #8869573 -
Flags: review?(ahunt) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2692d96a53e
Stop MediaControlService on task removal; r=ahunt
Comment 9•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Reporter | ||
Comment 10•7 years ago
|
||
Hello,
I have verified this and can confirm it is properly fixed.
Reporter | ||
Updated•6 years ago
|
Status: RESOLVED → VERIFIED
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•