Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3551)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java

Issue 2626213004: [Media>UI] Set large icon for lockscreen and wearable if it is specified by the page (Closed)
Patch Set: ** ready for review ** expose MediaSession image in MediaNotificationInfo Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
index 7e46c67fa36bac2e56f6a235dc63209ca1237342..396eee14bdb9835ecb102e4a099370e6c74a3aab 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
@@ -83,7 +83,7 @@ public class CastNotificationControl implements MediaRouteController.UiListener,
}
mPosterBitmap = posterBitmap;
if (mNotificationBuilder == null || mMediaRouteController == null) return;
- mNotificationBuilder.setLargeIcon(mMediaRouteController.getPoster());
+ mNotificationBuilder.setNotificationLargeIcon(mMediaRouteController.getPoster());
updateNotification();
}
@@ -106,10 +106,10 @@ public class CastNotificationControl implements MediaRouteController.UiListener,
mNotificationBuilder = new MediaNotificationInfo.Builder()
.setPaused(false)
.setPrivate(false)
- .setIcon(R.drawable.ic_notification_media_route)
+ .setNotificationSmallIcon(R.drawable.ic_notification_media_route)
.setContentIntent(contentIntent)
- .setLargeIcon(mMediaRouteController.getPoster())
- .setDefaultLargeIcon(R.drawable.cast_playing_square)
+ .setNotificationLargeIcon(mMediaRouteController.getPoster())
+ .setDefaultNotificationLargeIcon(R.drawable.cast_playing_square)
.setId(R.id.remote_notification)
.setListener(this);
mState = initialState;
@@ -145,7 +145,7 @@ public class CastNotificationControl implements MediaRouteController.UiListener,
// poster changes.
public void onPosterBitmapChanged() {
if (mNotificationBuilder == null || mMediaRouteController == null) return;
- mNotificationBuilder.setLargeIcon(mMediaRouteController.getPoster());
+ mNotificationBuilder.setNotificationLargeIcon(mMediaRouteController.getPoster());
updateNotification();
}
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698