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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java

Issue 67573003: Android: moves chrome/ to use long for JNI (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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/prerender/ExternalPrerenderRequestTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
index 9c4c3a32d3723fcd97bfbf09af4b095d33de67ec..5f294b0db09fa74a520f66a4ff4e1f07106b8a0f 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/prerender/ExternalPrerenderRequestTest.java
@@ -53,7 +53,7 @@ public class ExternalPrerenderRequestTest extends ChromiumTestShellTestBase {
* Test adding a prerender and canceling that to add a new one.
*/
public void testAddPrerenderAndCancel() throws InterruptedException {
- int webContentsPtr = mHandler.addPrerender(mProfile, GOOGLE_URL, "", 0, 0);
+ long webContentsPtr = mHandler.addPrerender(mProfile, GOOGLE_URL, "", 0, 0);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, GOOGLE_URL, webContentsPtr));
@@ -76,7 +76,7 @@ public class ExternalPrerenderRequestTest extends ChromiumTestShellTestBase {
*/
public void testCancelPrerender() {
mHandler.cancelCurrentPrerender();
- int webContentsPtr = mHandler.addPrerender(mProfile, GOOGLE_URL, "", 0, 0);
+ long webContentsPtr = mHandler.addPrerender(mProfile, GOOGLE_URL, "", 0, 0);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, GOOGLE_URL, webContentsPtr));
}
@@ -88,11 +88,11 @@ public class ExternalPrerenderRequestTest extends ChromiumTestShellTestBase {
* Test adding two prerenders without canceling the first one.
*/
public void testAddingPrerendersInaRow() throws InterruptedException {
- int webContentsPtr = mHandler.addPrerender(mProfile, GOOGLE_URL, "", 0, 0);
+ long webContentsPtr = mHandler.addPrerender(mProfile, GOOGLE_URL, "", 0, 0);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, GOOGLE_URL, webContentsPtr));
Thread.sleep(PRERENDER_DELAY_MS);
- int newWebContentsPtr = mHandler.addPrerender(mProfile, YOUTUBE_URL, "", 0, 0);
+ long newWebContentsPtr = mHandler.addPrerender(mProfile, YOUTUBE_URL, "", 0, 0);
assertTrue(ExternalPrerenderHandler.hasPrerenderedUrl(
mProfile, YOUTUBE_URL, newWebContentsPtr));
}

Powered by Google App Engine
This is Rietveld 408576698