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

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

Issue 1839013002: Distingush between displayed and remote playback state. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2661
Patch Set: Created 4 years, 9 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: chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaRouteController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaRouteController.java b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaRouteController.java
index 05be768804304708749a9d0cadc2dabdb19a0a65..d09aa05db9c9b88e5bde8f1d8be5e9c5a45addd2 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaRouteController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/MediaRouteController.java
@@ -139,10 +139,9 @@ public interface MediaRouteController {
/**
* Called when the Playback state has changed (e.g. from playing to paused)
- * @param oldState the old state
* @param newState the new state
*/
- void onPlaybackStateChanged(PlayerState oldState, PlayerState newState);
+ void onPlaybackStateChanged(PlayerState newState);
/**
* Called when the duration of the currently playing video changes.
@@ -303,12 +302,8 @@ public interface MediaRouteController {
*/
MediaStateListener getMediaStateListener();
- /**
- * @return true if the video is new
- */
- boolean shouldResetState(MediaStateListener newListener);
-
- @VisibleForTesting PlayerState getPlayerState();
+ @VisibleForTesting
+ PlayerState getDisplayedPlayerState();
/**
* Remove an existing media state listener

Powered by Google App Engine
This is Rietveld 408576698