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

Unified Diff: content/public/android/javatests/src/org/chromium/content/browser/PostMessageTest.java

Issue 572013002: Removing ContentViewCore dependencies from direct WebContents functions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased the patch and addressed review comments. Created 6 years, 2 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: content/public/android/javatests/src/org/chromium/content/browser/PostMessageTest.java
diff --git a/content/public/android/javatests/src/org/chromium/content/browser/PostMessageTest.java b/content/public/android/javatests/src/org/chromium/content/browser/PostMessageTest.java
index 65c56791b3e1ee51d60ad390e6195bfa912573de..92dd860e3b879429e76dca198d5e10675c11caea 100644
--- a/content/public/android/javatests/src/org/chromium/content/browser/PostMessageTest.java
+++ b/content/public/android/javatests/src/org/chromium/content/browser/PostMessageTest.java
@@ -72,7 +72,8 @@ public class PostMessageTest extends ContentViewTestBase {
@Feature({"AndroidWebView", "Android-PostMessage"})
public void testPostMessageToMainFrame() throws Throwable {
ContentViewCore contentViewCore = getContentViewCore();
- loadDataSync(contentViewCore, URL1, "text/html", false);
+ loadDataSync(contentViewCore.getWebContents().getNavigationController(), URL1, "text/html",
+ false);
contentViewCore.postMessageToFrame(null, MESSAGE, SOURCE_ORIGIN, "*");
mMessageObject.waitForMessage();
assertEquals(MESSAGE, mMessageObject.getData());

Powered by Google App Engine
This is Rietveld 408576698