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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastPositionTransferTest.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/CastPositionTransferTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastPositionTransferTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastPositionTransferTest.java
index ff44b17c761026ddcf00f348fb534973278ab6a6..64eeceb1693da6e99ecfc5e90a2a730aed19a9ef 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastPositionTransferTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/media/remote/CastPositionTransferTest.java
@@ -99,10 +99,10 @@ public class CastPositionTransferTest extends CastTestBase {
castVideoAndWaitUntilPlaying(CAST_TEST_ROUTE, tab, videoRect);
tapPlayPauseButton(tab, videoRect);
- int pausePosition = getRemotePositionMs();
+ long pausePosition = getRemotePositionMs();
for (int time = 0; time < MAX_VIEW_TIME_MS; time += VIEW_RETRY_MS) {
Thread.sleep(VIEW_RETRY_MS);
- int newPosition = getRemotePositionMs();
+ long newPosition = getRemotePositionMs();
if (newPosition == pausePosition) {
break;
}
@@ -114,7 +114,7 @@ public class CastPositionTransferTest extends CastTestBase {
&& getRemotePositionMs() != pausePosition; time += VIEW_RETRY_MS) {
Thread.sleep(VIEW_RETRY_MS);
}
- int remotePositionMs = getRemotePositionMs();
+ long remotePositionMs = getRemotePositionMs();
assertEquals("The remote player did not seek",
CAST_START_TIME_MS, remotePositionMs, SEEK_EPSILON_MS);

Powered by Google App Engine
This is Rietveld 408576698