New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 607862 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Android MediaRouter only (left Chro...
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

Flicking media notification in MediaFlinging

Project Member Reported by zqzh...@chromium.org, Apr 29 2016

Issue description

1) Start media fling
2) Play/pause several times from the notification

Observed: MediaNotification flicks between MediaStyle and non-MediaStyle

It seems to be happen in the "loading" state, when MediaNotificationInfo.supportsPlayPause() is false.
 
Labels: -Pri-3 Pri-1
Project Member

Comment 2 by bugdroid1@chromium.org, May 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b95089462ef15efc94c745c84a690020ff19c40e

commit b95089462ef15efc94c745c84a690020ff19c40e
Author: zqzhang <zqzhang@chromium.org>
Date: Tue May 10 11:19:05 2016

[MediaFling, UI] Avoid flickering in notification

When the remote playback is in loading state, it does not have play/pause button, then MediaNotification will use non-MediaStyle. This causes the notification flicking between media and non-media style.

This CL fixes the problem by treating the loading state as a transitional state.

BUG= 607862 

Review-Url: https://codereview.chromium.org/1931263002
Cr-Commit-Position: refs/heads/master@{#392578}

[modify] https://crrev.com/b95089462ef15efc94c745c84a690020ff19c40e/chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
[modify] https://crrev.com/b95089462ef15efc94c745c84a690020ff19c40e/chrome/android/java/src/org/chromium/chrome/browser/media/remote/RemoteMediaPlayerController.java

Zhiqiang, is this fixed?
Status: Fixed (was: Assigned)
Yes :)

Sign in to add a comment