The icon on the media control is out of sync

VERIFIED FIXED in Firefox 50

Status

()

P1
normal
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: whsu, Assigned: alwu)

Tracking

50 Branch
Firefox 50
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 verified)

Details

Attachments

(2 attachments)

(Reporter)

Description

2 years ago
Created attachment 8774131 [details]
ICON_ON_MEDIA_CONTROL_0725.png

@ Environment: 
  Device: Nexus 5 (Android 5.1.1);
  Build: Nightly 50.0a1 (2016-07-19);

@ Steps to reproduce:
  1. Open Firefox, and then got to "www.youtube.com" to play a video
  2. Press home button
  3. Open Google Chrome, and then go to "www.youtube.com"
  4. Play a video on Google Chrome, and then use app switch button to switch to Firefox
  5. Press play button to continue to play the same video on Firefox
  6. Swipe down the notification bar to check the status of media control

@ Expected result:
  The video can be played and the icon on the media control is shown as "PAUSE[||]".

@ Actual result:
  The icon on the media control is shown as "PLAY[>]".

@ Note: Attach the screenshot (ICON_ON_MEDIA_CONTROL_0725.png)
Hi Alastor, 

Could you please kindly help us check this ?
Thank you very much !
Flags: needinfo?(alwu)
Blocks: 1264901
Priority: -- → P1
(Assignee)

Updated

2 years ago
Assignee: nobody → alwu
Flags: needinfo?(alwu)
(Assignee)

Updated

2 years ago
Depends on: 1289356
(Assignee)

Updated

2 years ago
Duplicate of this bug: 1290109
(Assignee)

Comment 3

2 years ago
Created attachment 8775884 [details]
Bug 1288971 - notify audio playback changed by media element's play().

Review commit: https://reviewboard.mozilla.org/r/67932/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/67932/
(Assignee)

Comment 4

2 years ago
Comment on attachment 8775884 [details]
Bug 1288971 - notify audio playback changed by media element's play().

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/67932/diff/1-2/
Attachment #8775884 - Flags: review?(amarchesini)
Attachment #8775884 - Flags: review?(amarchesini) → review+
Comment on attachment 8775884 [details]
Bug 1288971 - notify audio playback changed by media element's play().

https://reviewboard.mozilla.org/r/67932/#review65072

Comment 7

2 years ago
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cf3d0b086060
notify audio playback changed by media element's play(). r=baku

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/cf3d0b086060
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
(Assignee)

Updated

2 years ago
Blocks: 1290836
(Assignee)

Comment 9

2 years ago
Comment on attachment 8775884 [details]
Bug 1288971 - notify audio playback changed by media element's play().

Approval Request Comment
[Feature/regressing bug #]: Let the media control interface can be consistent with the media element's playing state.
[User impact if declined]: Media control would be inconsistent.
[Describe test coverage new/current, TreeHerder]: No UI test
[Risks and why]: Low, this patch would only affect the control interface.
[String/UUID change made/needed]: no
Attachment #8775884 - Flags: approval-mozilla-beta?
Now that the feature is behind a Nightly flag you do not necessarily need to request uplift everywhere. As soon as everything is fixed we just remove the flag for the current version and let the changes ride the trains. This will give us some additional testing time too.
(Assignee)

Updated

2 years ago
Attachment #8775884 - Flags: approval-mozilla-beta?
Verified as fixed in build 51.0a1 2016-08-05;
Device: LG G4 (Android 5.1).
status-firefox50: fixed → verified

Comment 12

2 years ago
I'm changing the status to verified fixed considering that the bug is verified.
Status: RESOLVED → VERIFIED

Updated

2 years ago
Depends on: 1328057
You need to log in before you can comment on or make changes to this bug.