Closed Bug 1290836 Opened 3 years ago Closed 3 years ago

Re-enable media control for all versions (Remove nightly flag)

Categories

(Firefox for Android :: Audio/Video, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 52
Tracking Status
firefox52 --- verified

People

(Reporter: alwu, Assigned: alwu)

References

(Blocks 1 open bug)

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.
Depends on: 1285739
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 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)
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 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 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-
Depends on: 1302453
Depends on: 1303292
Flags: needinfo?(s.kaspari)
Flags: needinfo?(s.kaspari)
Summary: Re-enable media control for all versions → Re-enable media control for all versions (Remove nightly flag)
Depends on: 1305430
Depends on: 1305869
Depends on: 1303554
Attachment #8788123 - Flags: feedback?(snorp)
No longer depends on: 1306009
Depends on: 1311245
@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)
(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 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+
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/be91d38389a5
Re-enable media control for all versions. r=sebastian,snorp
https://hg.mozilla.org/mozilla-central/rev/be91d38389a5
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Verified as fixed in build 52.0a2 (2016-12-11);
Device: Nexus 5 (Android 6.0.1).
Status: RESOLVED → VERIFIED
Depends on: 1329209
You need to log in before you can comment on or make changes to this bug.