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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastTestBase.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/javatests/src/org/chromium/chrome/browser/media/remote/CastTestBase.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastTestBase.java b/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastTestBase.java
index 6f45e8674efdaec1f42170e3068766349877b33b..22663f6e880882450891e0707836c83167ca310e 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastTestBase.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastTestBase.java
@@ -66,12 +66,10 @@ public abstract class CastTestBase extends ChromeActivityTestCaseBase<ChromeActi
}
@Override
- public void onDurationUpdated(int durationMillis) {
- }
+ public void onDurationUpdated(long durationMillis) {}
@Override
- public void onPositionChanged(int positionMillis) {
- }
+ public void onPositionChanged(long positionMillis) {}
@Override
public void onTitleChanged(String title) {
@@ -362,7 +360,7 @@ public abstract class CastTestBase extends ChromeActivityTestCaseBase<ChromeActi
});
}
- protected void sleepNoThrow(int timeout) {
+ protected void sleepNoThrow(long timeout) {
try {
Thread.sleep(timeout);
} catch (InterruptedException e) {
@@ -463,11 +461,11 @@ public abstract class CastTestBase extends ChromeActivityTestCaseBase<ChromeActi
return false;
}
- protected int getRemotePositionMs() {
+ protected long getRemotePositionMs() {
return getMediaRouteController().getPosition();
}
- protected int getRemoteDurationMs() {
+ protected long getRemoteDurationMs() {
return getMediaRouteController().getDuration();
}

Powered by Google App Engine
This is Rietveld 408576698