When navigating from a page using MediaSession API to a page which is not, the actions persists |
|||||
Issue descriptionA follow up of https://crbug.com/687973 Sadly I didn't realized it in the beginning :(
,
Feb 8 2017
,
Feb 8 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/15ab0ba733d50b9f7567bd77049aafeaa9114b0b commit 15ab0ba733d50b9f7567bd77049aafeaa9114b0b Author: Zhiqiang Zhang <zqzhang@google.com> Date: Wed Feb 08 18:23:56 2017 [Media>UI] Reset media session actions when navigating away There is a bug caused by media session actions being not properly reset when navigating away, so that the notification control buttons stays the same when the new page starts to play media. This CL fixes the issue. BUG= 689664 Review-Url: https://codereview.chromium.org/2680103002 Cr-Commit-Position: refs/heads/master@{#449032} (cherry picked from commit e9c5df7d3be68ce0d8cf656dac0b9a1912ea5a28) Review-Url: https://codereview.chromium.org/2685943002 . Cr-Commit-Position: refs/branch-heads/2987@{#389} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} [modify] https://crrev.com/15ab0ba733d50b9f7567bd77049aafeaa9114b0b/chrome/android/BUILD.gn [modify] https://crrev.com/15ab0ba733d50b9f7567bd77049aafeaa9114b0b/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java [modify] https://crrev.com/15ab0ba733d50b9f7567bd77049aafeaa9114b0b/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java [modify] https://crrev.com/15ab0ba733d50b9f7567bd77049aafeaa9114b0b/chrome/android/java_sources.gni [add] https://crrev.com/15ab0ba733d50b9f7567bd77049aafeaa9114b0b/chrome/android/javatests/src/org/chromium/chrome/browser/media/ui/NotificationActionsUpdatedTest.java
,
Feb 8 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bugdroid1@chromium.org
, Feb 8 2017