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

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

Issue 1561663002: Use long for times and timestamps throught Java cast code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix nits Created 4 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
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 255e647d99079eb55d41080383eb2c4f764c2560..7c52ae130c6f78d87d7038d7fa6a83f508006aeb 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
@@ -56,7 +56,7 @@ public interface MediaRouteController extends TransportControl.Listener {
void pauseLocal();
- int getLocalPosition();
+ long getLocalPosition();
/**
* Tells the rest of Chrome that we are starting to cast, so that user inputs control cast
@@ -107,7 +107,7 @@ public interface MediaRouteController extends TransportControl.Listener {
/**
* @return the requested seek location. Only meaningful if isSeekRequested is true.
*/
- int getSeekLocation();
+ long getSeekLocation();
}
/**
@@ -152,14 +152,14 @@ public interface MediaRouteController extends TransportControl.Listener {
* Called when the duration of the currently playing video changes.
* @param durationMillis the new duration in ms.
*/
- void onDurationUpdated(int durationMillis);
+ void onDurationUpdated(long durationMillis);
/**
* Called when the media route controller receives new information about the
* current position in the video.
* @param positionMillis the current position in the video in ms.
*/
- void onPositionChanged(int positionMillis);
+ void onPositionChanged(long positionMillis);
/**
* Called if the title of the video changes
@@ -280,12 +280,12 @@ public interface MediaRouteController extends TransportControl.Listener {
*
* @return The current position of the remote playback in milliseconds.
*/
- int getPosition();
+ long getPosition();
/**
* @return The stream duration in milliseconds.
*/
- int getDuration();
+ long getDuration();
/**
* @return Whether the video is currently being played.
@@ -302,7 +302,7 @@ public interface MediaRouteController extends TransportControl.Listener {
*
* @param msec The position to seek to, in milliseconds.
*/
- void seekTo(int msec);
+ void seekTo(long msec);
/**
* Stop the current remote playback completely and release all resources.

Powered by Google App Engine
This is Rietveld 408576698