Closed
Bug 1290836
Opened 8 years ago
Closed 8 years ago
Re-enable media control for all versions (Remove nightly flag)
Categories
(Firefox for Android Graveyard :: Audio/Video, defect)
Firefox for Android Graveyard
Audio/Video
Tracking
(firefox52 verified)
VERIFIED
FIXED
Firefox 52
Tracking | Status | |
---|---|---|
firefox52 | --- | verified |
People
(Reporter: alwu, Assigned: alwu)
References
Details
Attachments
(1 file)
Fork from bug1290510, we would re-enable media control for all versions when it's more stable and ready to be shipped.
Assignee | ||
Comment 1•8 years ago
|
||
This bug would be landed after fixing bug1290467.
I ask for a review now just because I want to reduce reviewing process time and make sure it can be landed immediately after bug1290467.
Comment hidden (mozreview-request) |
Comment 3•8 years ago
|
||
Comment on attachment 8788123 [details]
Bug 1290836 - Re-enable media control for all versions.
Maybe let's wait after Bug 1290467 lands (I'm just building, testing and reviewing the patch!) - then let's see how it behaves in Nightly and if there are any more issues. With merge day next week let's target 52.
Attachment #8788123 -
Flags: review?(s.kaspari)
Comment 4•8 years ago
|
||
Hi,
Here is the doc sheet with test cases and results: https://docs.google.com/spreadsheets/d/17HbRS8EHayb5G8dwOFcVj7CMjbXq-uMmg0T_H8qbEFU/edit#gid=0. From my side I'm agree with comment 3 because needs more testing after bug 1290467 lands (maybe new test cases).
Thanks!
Comment 5•8 years ago
|
||
Comment on attachment 8788123 [details]
Bug 1290836 - Re-enable media control for all versions.
Snorp, Brad: All linked bugs are fixed now and in today's Nightly. We are thinking about letting the feature ride the trains with 52. You have both been reporting problems in the past. Please let us know if you are seeing any issues in Nightly that should block the feature.
Attachment #8788123 -
Flags: feedback?(snorp)
Attachment #8788123 -
Flags: feedback?(blassey.bugs)
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8788123 [details]
Bug 1290836 - Re-enable media control for all versions.
https://reviewboard.mozilla.org/r/76706/#review76308
Attachment #8788123 -
Flags: review+
Comment 7•8 years ago
|
||
Comment on attachment 8788123 [details]
Bug 1290836 - Re-enable media control for all versions.
The last couple patches landed at the end of last week. I'm all for the plan to ship this in 52, but IMO the right thing to do is to check in mid-nightly cycle and see how the feature is looking and at that point if things look good remove the nightly block.
The other thing I'd note is at this point most nightly testers have it in their heads "this feature is broken and not riding the trains." In order to validate that it is ready to ride the trains you need to reset that mindset. Normally I'd suggest an announcement at the mobile stand up, but we don't have those anymore. Perhaps an email to the nightly-testers list would be the next best thing.
Attachment #8788123 -
Flags: feedback?(blassey.bugs) → feedback-
Updated•8 years ago
|
Flags: needinfo?(s.kaspari)
Updated•8 years ago
|
Flags: needinfo?(s.kaspari)
Summary: Re-enable media control for all versions → Re-enable media control for all versions (Remove nightly flag)
Updated•8 years ago
|
Attachment #8788123 -
Flags: feedback?(snorp)
Comment hidden (mozreview-request) |
Comment 9•8 years ago
|
||
@sorina: All linked bugs are now fixed. Do you recommend doing another round of testing before letting this ride the trains?
Flags: needinfo?(sorina.florean)
Comment 10•8 years ago
|
||
(In reply to Sebastian Kaspari (:sebastian) from comment #9)
> @sorina: All linked bugs are now fixed. Do you recommend doing another round
> of testing before letting this ride the trains?
Hi,
The feature looks stable now and the testing process will continue after this ride the trains.
I will like to test again today, on different devices, and if is no majors issues, this can land.
Flags: needinfo?(sorina.florean)
Comment 11•8 years ago
|
||
mozreview-review |
Comment on attachment 8788123 [details]
Bug 1290836 - Re-enable media control for all versions.
https://reviewboard.mozilla.org/r/76706/#review89712
From the mail:
> Recommendation from QE: SHIP IT.
Let's go! :)
Attachment #8788123 -
Flags: review?(s.kaspari) → review+
Comment 12•8 years ago
|
||
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/be91d38389a5
Re-enable media control for all versions. r=sebastian,snorp
Comment 13•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Comment 14•8 years ago
|
||
Verified as fixed in build 52.0a2 (2016-12-11);
Device: Nexus 5 (Android 6.0.1).
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
•