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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/MediaSessionImpl.java

Issue 2442303002: Adding new media controls to MediaNotification (Closed)
Patch Set: rebased onto MediaSession refactoring Created 4 years, 2 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
Index: content/public/android/java/src/org/chromium/content/browser/MediaSessionImpl.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/MediaSessionImpl.java b/content/public/android/java/src/org/chromium/content/browser/MediaSessionImpl.java
index db04ad734dbe477677bfc6e4143acada88c99541..4fe856305957421a233c6add79502bf32988566f 100644
--- a/content/public/android/java/src/org/chromium/content/browser/MediaSessionImpl.java
+++ b/content/public/android/java/src/org/chromium/content/browser/MediaSessionImpl.java
@@ -52,6 +52,11 @@ public class MediaSessionImpl extends MediaSession {
nativeStop(mNativeMediaSessionAndroid);
}
+ @Override
+ public void didReceiveAction(int action) {
+ nativeDidReceiveAction(mNativeMediaSessionAndroid, action);
+ }
+
@CalledByNative
private boolean hasObservers() {
return !mObservers.isEmpty();
@@ -79,6 +84,20 @@ public class MediaSessionImpl extends MediaSession {
}
@CalledByNative
+ private void mediaSessionEnabledAction(int action) {
+ for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
whywhat 2016/10/28 16:10:23 nit: add a space between ; and ) to highlight ther
Zhiqiang Zhang (Slow) 2016/11/01 15:24:15 Actually `git cl format` don't like the space :/
+ mObserversIterator.next().mediaSessionEnabledAction(action);
+ }
+ }
+
+ @CalledByNative
+ private void mediaSessionDisabledAction(int action) {
+ for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
whywhat 2016/10/28 16:10:23 nit: ditto
Zhiqiang Zhang (Slow) 2016/11/01 15:24:15 See reply above.
+ mObserversIterator.next().mediaSessionDisabledAction(action);
+ }
+ }
+
+ @CalledByNative
private static MediaSessionImpl create(long nativeMediaSession) {
return new MediaSessionImpl(nativeMediaSession);
}
@@ -101,4 +120,5 @@ public class MediaSessionImpl extends MediaSession {
private native void nativeResume(long nativeMediaSessionAndroid);
private native void nativeSuspend(long nativeMediaSessionAndroid);
private native void nativeStop(long nativeMediaSessionAndroid);
+ private native void nativeDidReceiveAction(long nativeMediaSessionAndroid, int action);
}

Powered by Google App Engine
This is Rietveld 408576698