diff options
author | mkosiba@chromium.org <mkosiba@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-09 22:41:48 +0000 |
---|---|---|
committer | mkosiba@chromium.org <mkosiba@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-09 22:41:48 +0000 |
commit | a7c6433995e9f2ebe99cd55e692d53044731c731 (patch) | |
tree | 2dd039e5362d078e5f6c0162afbe53df21778697 /android_webview | |
parent | efcc84fca4ac288401d21f556892b3e949f945d6 (diff) | |
download | chromium_src-a7c6433995e9f2ebe99cd55e692d53044731c731.zip chromium_src-a7c6433995e9f2ebe99cd55e692d53044731c731.tar.gz chromium_src-a7c6433995e9f2ebe99cd55e692d53044731c731.tar.bz2 |
[android_webview] Use AwContents loadUrl method instead of ContentViewCore.
This introduces a loadUrl method on AwContents and modifies all of
our test code to use that method instead of the on ContentViewCore.
BUG=154558
Review URL: https://chromiumcodereview.appspot.com/11028094
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@160959 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'android_webview')
19 files changed, 482 insertions, 417 deletions
diff --git a/android_webview/java/src/org/chromium/android_webview/AwContents.java b/android_webview/java/src/org/chromium/android_webview/AwContents.java index 377e568..a5a9617 100644 --- a/android_webview/java/src/org/chromium/android_webview/AwContents.java +++ b/android_webview/java/src/org/chromium/android_webview/AwContents.java @@ -18,6 +18,7 @@ import org.chromium.base.JNINamespace; import org.chromium.base.ThreadUtils; import org.chromium.chrome.browser.component.navigation_interception.InterceptNavigationDelegate; import org.chromium.content.browser.ContentViewCore; +import org.chromium.content.browser.LoadUrlParams; import org.chromium.content.browser.NavigationHistory; import org.chromium.content.common.CleanupReference; import org.chromium.net.X509Util; @@ -173,6 +174,17 @@ public class AwContents { return null; } + /** + * Load url without fixing up the url string. Consumers of ContentView are responsible for + * ensuring the URL passed in is properly formatted (i.e. the scheme has been added if left + * off during user input). + * + * @param pararms Parameters for this load. + */ + public void loadUrl(LoadUrlParams params) { + mContentViewCore.loadUrl(params); + } + //-------------------------------------------------------------------------------------------- // WebView[Provider] method implementations (where not provided by ContentViewCore) //-------------------------------------------------------------------------------------------- diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewLoadUrlTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewLoadUrlTest.java index ea2c6c7..466d714 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewLoadUrlTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewLoadUrlTest.java @@ -9,8 +9,8 @@ import android.test.suitebuilder.annotation.SmallTest; import org.apache.http.Header; import org.apache.http.HttpRequest; +import org.chromium.android_webview.AwContents; import org.chromium.base.test.util.Feature; -import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.LoadUrlParams; import org.chromium.content.browser.test.util.CallbackHelper; import org.chromium.android_webview.test.util.TestWebServer; @@ -31,11 +31,12 @@ public class AndroidWebViewLoadUrlTest extends AndroidWebViewTestBase { "<html><head><title>" + expectedTitle + "</title></head><body>foo</body></html>"; final TestAwContentsClient contentsClient = new TestAwContentsClient(); - final ContentViewCore contentViewCore = - createAwTestContainerViewOnMainSync(contentsClient).getContentViewCore(); - loadDataSync(contentViewCore, contentsClient.getOnPageFinishedHelper(), data, + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentsClient); + final AwContents awContents = testContainerView.getAwContents(); + loadDataSync(awContents, contentsClient.getOnPageFinishedHelper(), data, "text/html", false); - assertEquals(expectedTitle, getTitleOnUiThread(contentViewCore)); + assertEquals(expectedTitle, getTitleOnUiThread(awContents)); } @SmallTest @@ -46,18 +47,19 @@ public class AndroidWebViewLoadUrlTest extends AndroidWebViewTestBase { "L2hlYWQ+PC9odG1sPg=="; final TestAwContentsClient contentsClient = new TestAwContentsClient(); - final ContentViewCore contentViewCore = - createAwTestContainerViewOnMainSync(contentsClient).getContentViewCore(); - loadDataSync(contentViewCore, contentsClient.getOnPageFinishedHelper(), data, + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentsClient); + final AwContents awContents = testContainerView.getAwContents(); + loadDataSync(awContents, contentsClient.getOnPageFinishedHelper(), data, "text/html", true); - assertEquals(expectedTitle, getTitleOnUiThread(contentViewCore)); + assertEquals(expectedTitle, getTitleOnUiThread(awContents)); } /** * Loads url on the UI thread and blocks until onPageFinished is called. */ protected void loadUrlWithExtraHeadersSync( - final ContentViewCore contentViewCore, + final AwContents awContents, CallbackHelper onPageFinishedHelper, final String url, final Map<String, String> extraHeaders) throws Throwable { @@ -67,7 +69,7 @@ public class AndroidWebViewLoadUrlTest extends AndroidWebViewTestBase { public void run() { LoadUrlParams params = new LoadUrlParams(url); params.setExtraHeaders(extraHeaders); - contentViewCore.loadUrl(params); + awContents.loadUrl(params); } }); onPageFinishedHelper.waitForCallback(currentCallCount, 1, WAIT_TIMEOUT_SECONDS, @@ -78,8 +80,9 @@ public class AndroidWebViewLoadUrlTest extends AndroidWebViewTestBase { @Feature({"Android-WebView"}) public void testLoadUrlWithExtraHeaders() throws Throwable { final TestAwContentsClient contentsClient = new TestAwContentsClient(); - final ContentViewCore contentViewCore = - createAwTestContainerViewOnMainSync(contentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentsClient); + final AwContents awContents = testContainerView.getAwContents(); TestWebServer webServer = null; try { @@ -93,7 +96,7 @@ public class AndroidWebViewLoadUrlTest extends AndroidWebViewTestBase { for (int i = 0; i < headerNames.length; ++i) extraHeaders.put(headerNames[i], headerValues[i]); - loadUrlWithExtraHeadersSync(contentViewCore, + loadUrlWithExtraHeadersSync(awContents, contentsClient.getOnPageFinishedHelper(), url, extraHeaders); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewTestBase.java b/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewTestBase.java index 211f6dd..f51847e 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewTestBase.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/AndroidWebViewTestBase.java @@ -74,11 +74,11 @@ public class AndroidWebViewTestBase } } - protected void enableJavaScriptOnUiThread(final ContentViewCore contentViewCore) { + protected void enableJavaScriptOnUiThread(final AwContents awContents) { getInstrumentation().runOnMainSync(new Runnable() { @Override public void run() { - contentViewCore.getContentSettings().setJavaScriptEnabled(true); + awContents.getContentViewCore().getContentSettings().setJavaScriptEnabled(true); } }); } @@ -86,11 +86,11 @@ public class AndroidWebViewTestBase /** * Loads url on the UI thread and blocks until onPageFinished is called. */ - protected void loadUrlSync(final ContentViewCore contentViewCore, + protected void loadUrlSync(final AwContents awContents, CallbackHelper onPageFinishedHelper, final String url) throws Throwable { int currentCallCount = onPageFinishedHelper.getCallCount(); - loadUrlAsync(contentViewCore, url); + loadUrlAsync(awContents, url); onPageFinishedHelper.waitForCallback(currentCallCount, 1, WAIT_TIMEOUT_SECONDS, TimeUnit.SECONDS); } @@ -98,12 +98,12 @@ public class AndroidWebViewTestBase /** * Loads url on the UI thread but does not block. */ - protected void loadUrlAsync(final ContentViewCore contentViewCore, + protected void loadUrlAsync(final AwContents awContents, final String url) throws Throwable { runTestOnUiThread(new Runnable() { @Override public void run() { - contentViewCore.loadUrl(new LoadUrlParams(url)); + awContents.getContentViewCore().loadUrl(new LoadUrlParams(url)); } }); } @@ -111,12 +111,12 @@ public class AndroidWebViewTestBase /** * Loads data on the UI thread and blocks until onPageFinished is called. */ - protected void loadDataSync(final ContentViewCore contentViewCore, + protected void loadDataSync(final AwContents awContents, CallbackHelper onPageFinishedHelper, final String data, final String mimeType, final boolean isBase64Encoded) throws Throwable { int currentCallCount = onPageFinishedHelper.getCallCount(); - loadDataAsync(contentViewCore, data, mimeType, isBase64Encoded); + loadDataAsync(awContents, data, mimeType, isBase64Encoded); onPageFinishedHelper.waitForCallback(currentCallCount, 1, WAIT_TIMEOUT_SECONDS, TimeUnit.SECONDS); } @@ -124,32 +124,32 @@ public class AndroidWebViewTestBase /** * Loads data on the UI thread but does not block. */ - protected void loadDataAsync(final ContentViewCore contentViewCore, final String data, + protected void loadDataAsync(final AwContents awContents, final String data, final String mimeType, final boolean isBase64Encoded) throws Throwable { runTestOnUiThread(new Runnable() { @Override public void run() { - contentViewCore.loadUrl(LoadUrlParams.createLoadDataParams( + awContents.loadUrl(LoadUrlParams.createLoadDataParams( data, mimeType, isBase64Encoded)); } }); } protected AwTestContainerView createAwTestContainerView(final boolean incognito, - final AwContentsClient contentsClient) { + final AwContentsClient awContentsClient) { return createAwTestContainerView(incognito, new AwTestContainerView(getActivity()), - contentsClient); + awContentsClient); } protected AwTestContainerView createAwTestContainerView(final boolean incognito, final AwTestContainerView testContainerView, - final AwContentsClient contentsClient) { + final AwContentsClient awContentsClient) { ContentViewCore contentViewCore = new ContentViewCore( getActivity(), ContentViewCore.PERSONALITY_VIEW); testContainerView.initialize(contentViewCore, new AwContents(testContainerView, testContainerView.getInternalAccessDelegate(), - contentViewCore, contentsClient, new ActivityNativeWindow(getActivity()), + contentViewCore, awContentsClient, new ActivityNativeWindow(getActivity()), incognito, false)); getActivity().addView(testContainerView); return testContainerView; @@ -175,31 +175,31 @@ public class AndroidWebViewTestBase return testContainerView.get(); } - protected void destroyAwContentsOnMainSync(final AwContents contents) { - if (contents == null) return; + protected void destroyAwContentsOnMainSync(final AwContents awContents) { + if (awContents == null) return; getInstrumentation().runOnMainSync(new Runnable() { @Override public void run() { - contents.destroy(); + awContents.destroy(); } }); } - protected String getTitleOnUiThread(final ContentViewCore contentViewCore) throws Throwable { + protected String getTitleOnUiThread(final AwContents awContents) throws Throwable { return runTestOnUiThreadAndGetResult(new Callable<String>() { @Override public String call() throws Exception { - return contentViewCore.getTitle(); + return awContents.getContentViewCore().getTitle(); } }); } protected ContentSettings getContentSettingsOnUiThread( - final ContentViewCore contentViewCore) throws Throwable { + final AwContents awContents) throws Throwable { return runTestOnUiThreadAndGetResult(new Callable<ContentSettings>() { @Override public ContentSettings call() throws Exception { - return contentViewCore.getContentSettings(); + return awContents.getContentViewCore().getContentSettings(); } }); } @@ -218,7 +218,7 @@ public class AndroidWebViewTestBase * Executes the given snippet of JavaScript code within the given ContentView. Returns the * result of its execution in JSON format. */ - protected String executeJavaScriptAndWaitForResult(final ContentViewCore core, + protected String executeJavaScriptAndWaitForResult(final AwContents awContents, TestAwContentsClient viewClient, final String code) throws Throwable { final AtomicInteger requestId = new AtomicInteger(); TestCallbackHelperContainer.OnEvaluateJavaScriptResultHelper @@ -227,7 +227,7 @@ public class AndroidWebViewTestBase runTestOnUiThread(new Runnable() { @Override public void run() { - requestId.set(core.evaluateJavaScript(code)); + requestId.set(awContents.getContentViewCore().evaluateJavaScript(code)); } }); onEvaluateJavaScriptResultHelper.waitForCallback(currentCallCount); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ArchiveTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ArchiveTest.java index 05a4074..14b4a55 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ArchiveTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ArchiveTest.java @@ -80,7 +80,7 @@ public class ArchiveTest extends AndroidWebViewTestBase { file.delete(); assertFalse(file.exists()); - loadUrlSync(mTestContainerView.getContentViewCore(), + loadUrlSync(mTestContainerView.getAwContents(), mContentsClient.getOnPageFinishedHelper(), TEST_PAGE); doArchiveTest(mTestContainerView.getAwContents(), path, false, path); @@ -91,7 +91,7 @@ public class ArchiveTest extends AndroidWebViewTestBase { public void testAutoGoodPath() throws Throwable { final String path = getActivity().getFilesDir().getAbsolutePath() + "/"; - loadUrlSync(mTestContainerView.getContentViewCore(), + loadUrlSync(mTestContainerView.getAwContents(), mContentsClient.getOnPageFinishedHelper(), TEST_PAGE); // Create the first archive @@ -115,7 +115,7 @@ public class ArchiveTest extends AndroidWebViewTestBase { file.delete(); assertFalse(file.exists()); - loadUrlSync(mTestContainerView.getContentViewCore(), + loadUrlSync(mTestContainerView.getAwContents(), mContentsClient.getOnPageFinishedHelper(), TEST_PAGE); doArchiveTest(mTestContainerView.getAwContents(), path, false, null); @@ -129,7 +129,7 @@ public class ArchiveTest extends AndroidWebViewTestBase { file.delete(); assertFalse(file.exists()); - loadUrlSync(mTestContainerView.getContentViewCore(), + loadUrlSync(mTestContainerView.getAwContents(), mContentsClient.getOnPageFinishedHelper(), TEST_PAGE); doArchiveTest(mTestContainerView.getAwContents(), path, true, null); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsTest.java index bfcfce6..17065cf1 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsTest.java @@ -19,7 +19,6 @@ import org.chromium.android_webview.test.util.TestWebServer; import org.chromium.base.test.util.DisabledTest; import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.UrlUtils; -import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.test.util.CallbackHelper; import org.chromium.content.browser.test.util.Criteria; import org.chromium.content.browser.test.util.CriteriaHelper; @@ -83,12 +82,12 @@ public class AwContentsTest extends AndroidWebViewTestBase { final String imageDoc = "<head/><body><img/><img/></body>"; // Make sure a document that does not have images returns 0 - loadDataSync(awContents.getContentViewCore(), loadHelper, emptyDoc, mime, false); + loadDataSync(awContents, loadHelper, emptyDoc, mime, false); int result = callDocumentHasImagesSync(awContents); assertEquals(0, result); // Make sure a document that does have images returns 1 - loadDataSync(awContents.getContentViewCore(), loadHelper, imageDoc, mime, false); + loadDataSync(awContents, loadHelper, imageDoc, mime, false); result = callDocumentHasImagesSync(awContents); assertEquals(1, result); } @@ -109,7 +108,6 @@ public class AwContentsTest extends AndroidWebViewTestBase { final TestAwContentsClient contentClient = new TestAwContentsClient(); final AwTestContainerView testContainer = createAwTestContainerViewOnMainSync(false, contentClient); - final ContentViewCore contentView = testContainer.getContentViewCore(); final AwContents awContents = testContainer.getAwContents(); TestWebServer webServer = null; @@ -125,31 +123,31 @@ public class AwContentsTest extends AndroidWebViewTestBase { // First load to populate cache. clearCacheOnUiThread(awContents, true); - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), pageUrl); assertEquals(1, webServer.getRequestCount(pagePath)); // Load about:blank so next load is not treated as reload by webkit and force // revalidate with the server. - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), "about:blank"); // No clearCache call, so should be loaded from cache. - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), pageUrl); assertEquals(1, webServer.getRequestCount(pagePath)); // Same as above. - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), "about:blank"); // Clear cache, so should hit server again. clearCacheOnUiThread(awContents, true); - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), pageUrl); assertEquals(2, webServer.getRequestCount(pagePath)); @@ -187,7 +185,6 @@ public class AwContentsTest extends AndroidWebViewTestBase { public void testGetFavicon() throws Throwable { final AwTestContainerView testView = createAwTestContainerViewOnMainSync(mContentsClient); final AwContents awContents = testView.getAwContents(); - final ContentViewCore contentViewCore = testView.getContentViewCore(); TestWebServer webServer = null; try { @@ -203,8 +200,8 @@ public class AwContentsTest extends AndroidWebViewTestBase { // the page load completes which makes it slightly hard to test. final Bitmap defaultFavicon = awContents.getFavicon(); - getContentSettingsOnUiThread(contentViewCore).setImagesEnabled(true); - loadUrlSync(contentViewCore, mContentsClient.getOnPageFinishedHelper(), pageUrl); + getContentSettingsOnUiThread(awContents).setImagesEnabled(true); + loadUrlSync(awContents, mContentsClient.getOnPageFinishedHelper(), pageUrl); assertTrue(CriteriaHelper.pollForCriteria(new Criteria() { @Override diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java index 436f34a..62a395c0 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java @@ -51,16 +51,16 @@ public class AwSettingsTest extends AndroidWebViewTestBase { * In this case, requiresJsEnabled constructor argument must be set to true. */ abstract class AwSettingsTestHelper<T> { - protected final ContentViewCore mContentViewCore; + protected final AwContents mAwContents; protected final TestAwContentsClient mContentViewClient; protected final ContentSettings mContentSettings; - AwSettingsTestHelper(ContentViewCore contentViewCore, + AwSettingsTestHelper(AwContents awContents, TestAwContentsClient contentViewClient, boolean requiresJsEnabled) throws Throwable { - mContentViewCore = contentViewCore; + mAwContents = awContents; mContentViewClient = contentViewClient; - mContentSettings = getContentSettingsOnUiThread(mContentViewCore); + mContentSettings = getContentSettingsOnUiThread(mAwContents); if (requiresJsEnabled) { mContentSettings.setJavaScriptEnabled(true); } @@ -93,12 +93,12 @@ public class AwSettingsTest extends AndroidWebViewTestBase { protected abstract void doEnsureSettingHasValue(T value) throws Throwable; protected String getTitleOnUiThread() throws Throwable { - return AwSettingsTest.this.getTitleOnUiThread(mContentViewCore); + return AwSettingsTest.this.getTitleOnUiThread(mAwContents); } protected void loadDataSync(String data) throws Throwable { AwSettingsTest.this.loadDataSync( - mContentViewCore, + mAwContents, mContentViewClient.getOnPageFinishedHelper(), data, "text/html", @@ -107,7 +107,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { protected void loadUrlSync(String url) throws Throwable { AwSettingsTest.this.loadUrlSync( - mContentViewCore, + mAwContents, mContentViewClient.getOnPageFinishedHelper(), url); } @@ -122,9 +122,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String JS_ENABLED_STRING = "JS Enabled"; private static final String JS_DISABLED_STRING = "JS Disabled"; - AwSettingsJavaScriptTestHelper(ContentViewCore contentViewCore, + AwSettingsJavaScriptTestHelper(AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, false); + super(awContents, contentViewClient, false); } @Override @@ -166,9 +166,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { // JavaScript state. class AwSettingsJavaScriptDynamicTestHelper extends AwSettingsJavaScriptTestHelper { AwSettingsJavaScriptDynamicTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient); + super(awContents, contentViewClient); // Load the page. super.doEnsureSettingHasValue(getInitialValue()); } @@ -178,7 +178,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { String oldTitle = getTitleOnUiThread(); String newTitle = oldTitle + "_modified"; executeJavaScriptAndWaitForResult( - mContentViewCore, mContentViewClient, getScript(newTitle)); + mAwContents, mContentViewClient, getScript(newTitle)); assertEquals(value == ENABLED ? newTitle : oldTitle, getTitleOnUiThread()); } @@ -191,9 +191,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String PLUGINS_ENABLED_STRING = "Embed"; private static final String PLUGINS_DISABLED_STRING = "NoEmbed"; - AwSettingsPluginsTestHelper(ContentViewCore contentViewCore, + AwSettingsPluginsTestHelper(AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); } @Override @@ -232,9 +232,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { class AwSettingsStandardFontFamilyTestHelper extends AwSettingsTestHelper<String> { AwSettingsStandardFontFamilyTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); } @Override @@ -272,9 +272,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { class AwSettingsDefaultFontSizeTestHelper extends AwSettingsTestHelper<Integer> { AwSettingsDefaultFontSizeTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); } @Override @@ -314,10 +314,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private ImagePageGenerator mGenerator; AwSettingsLoadImagesAutomaticallyTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient, ImagePageGenerator generator) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mGenerator = generator; } @@ -353,9 +353,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { class AwSettingsDefaultTextEncodingTestHelper extends AwSettingsTestHelper<String> { AwSettingsDefaultTextEncodingTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); } @Override @@ -395,9 +395,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String CUSTOM_UA = "ChromeViewTest"; AwSettingsUserAgentStringTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mDefaultUa = mContentSettings.getUserAgentString(); } @@ -445,9 +445,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String HAS_LOCAL_STORAGE = "Has localStorage"; AwSettingsDomStorageEnabledTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); } @Override @@ -487,9 +487,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String ACCESS_DENIED_TITLE = "undefined"; AwSettingsUniversalAccessFromFilesTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mIframeContainerUrl = UrlUtils.getTestFileUrl("webview/iframe_access.html"); mIframeUrl = UrlUtils.getTestFileUrl("webview/hello_world.html"); // The value of the setting depends on the SDK version. @@ -538,9 +538,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String ACCESS_DENIED_TITLE = "undefined"; AwSettingsFileAccessFromFilesIframeTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mIframeContainerUrl = UrlUtils.getTestFileUrl("webview/iframe_access.html"); mIframeUrl = UrlUtils.getTestFileUrl("webview/hello_world.html"); mContentSettings.setAllowUniversalAccessFromFileURLs(false); @@ -585,9 +585,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String ACCESS_DENIED_TITLE = "Exception"; AwSettingsFileAccessFromFilesXhrTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mXhrContainerUrl = UrlUtils.getTestFileUrl("webview/xhr_access.html"); mContentSettings.setAllowUniversalAccessFromFileURLs(false); // The value of the setting depends on the SDK version. @@ -630,10 +630,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private static final String ACCESS_DENIED_TITLE = "about:blank"; AwSettingsFileUrlAccessTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient, int startIndex) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mIndex = startIndex; } @@ -675,10 +675,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private final String mTarget; AwSettingsContentUrlAccessTestHelper( - ContentViewCore contentViewCore, + AwContents awContents, TestAwContentsClient contentViewClient, int index) throws Throwable { - super(contentViewCore, contentViewClient, true); + super(awContents, contentViewClient, true); mTarget = "content_access_" + index; } @@ -706,7 +706,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { protected void doEnsureSettingHasValue(Boolean value) throws Throwable { AwSettingsTest.this.resetResourceRequestCountInContentProvider(mTarget); AwSettingsTest.this.loadUrlSync( - mContentViewCore, + mAwContents, mContentViewClient.getOnPageFinishedHelper(), AwSettingsTest.this.createContentUrl(mTarget)); if (value == ENABLED) { @@ -733,15 +733,16 @@ public class AwSettingsTest extends AndroidWebViewTestBase { + "</head><body onload=\"document.title='" + JS_ENABLED_STRING + "';\"></body></html>"; final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); loadDataSync( - contentView, + awContents, contentClient.getOnPageFinishedHelper(), TEST_PAGE_HTML, "text/html", false); - assertEquals(JS_DISABLED_STRING, getTitleOnUiThread(contentView)); + assertEquals(JS_DISABLED_STRING, getTitleOnUiThread(awContents)); } @SmallTest @@ -749,8 +750,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testJavaScriptEnabledNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsJavaScriptTestHelper(views.getView0(), views.getClient0()), - new AwSettingsJavaScriptTestHelper(views.getView1(), views.getClient1())); + new AwSettingsJavaScriptTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsJavaScriptTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -758,8 +759,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testJavaScriptEnabledIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsJavaScriptTestHelper(views.getView0(), views.getClient0()), - new AwSettingsJavaScriptTestHelper(views.getView1(), views.getClient1())); + new AwSettingsJavaScriptTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsJavaScriptTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -767,8 +768,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testJavaScriptEnabledBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsJavaScriptTestHelper(views.getView0(), views.getClient0()), - new AwSettingsJavaScriptTestHelper(views.getView1(), views.getClient1())); + new AwSettingsJavaScriptTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsJavaScriptTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -776,8 +777,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testJavaScriptEnabledDynamicNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsJavaScriptDynamicTestHelper(views.getView0(), views.getClient0()), - new AwSettingsJavaScriptDynamicTestHelper(views.getView1(), views.getClient1())); + new AwSettingsJavaScriptDynamicTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsJavaScriptDynamicTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -785,8 +786,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testJavaScriptEnabledDynamicIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsJavaScriptDynamicTestHelper(views.getView0(), views.getClient0()), - new AwSettingsJavaScriptDynamicTestHelper(views.getView1(), views.getClient1())); + new AwSettingsJavaScriptDynamicTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsJavaScriptDynamicTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -794,8 +795,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testJavaScriptEnabledDynamicBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsJavaScriptDynamicTestHelper(views.getView0(), views.getClient0()), - new AwSettingsJavaScriptDynamicTestHelper(views.getView1(), views.getClient1())); + new AwSettingsJavaScriptDynamicTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsJavaScriptDynamicTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -803,8 +804,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testPluginsEnabledNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsPluginsTestHelper(views.getView0(), views.getClient0()), - new AwSettingsPluginsTestHelper(views.getView1(), views.getClient1())); + new AwSettingsPluginsTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsPluginsTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -812,8 +813,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testPluginsEnabledIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsPluginsTestHelper(views.getView0(), views.getClient0()), - new AwSettingsPluginsTestHelper(views.getView1(), views.getClient1())); + new AwSettingsPluginsTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsPluginsTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -821,8 +822,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testPluginsEnabledBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsPluginsTestHelper(views.getView0(), views.getClient0()), - new AwSettingsPluginsTestHelper(views.getView1(), views.getClient1())); + new AwSettingsPluginsTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsPluginsTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -830,8 +831,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testStandardFontFamilyNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsStandardFontFamilyTestHelper(views.getView0(), views.getClient0()), - new AwSettingsStandardFontFamilyTestHelper(views.getView1(), views.getClient1())); + new AwSettingsStandardFontFamilyTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsStandardFontFamilyTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -839,8 +840,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testStandardFontFamilyIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsStandardFontFamilyTestHelper(views.getView0(), views.getClient0()), - new AwSettingsStandardFontFamilyTestHelper(views.getView1(), views.getClient1())); + new AwSettingsStandardFontFamilyTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsStandardFontFamilyTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -848,8 +849,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testStandardFontFamilyBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsStandardFontFamilyTestHelper(views.getView0(), views.getClient0()), - new AwSettingsStandardFontFamilyTestHelper(views.getView1(), views.getClient1())); + new AwSettingsStandardFontFamilyTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsStandardFontFamilyTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -857,8 +858,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDefaultFontSizeNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsDefaultFontSizeTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDefaultFontSizeTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDefaultFontSizeTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDefaultFontSizeTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -866,8 +867,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDefaultFontSizeIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsDefaultFontSizeTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDefaultFontSizeTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDefaultFontSizeTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDefaultFontSizeTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -875,8 +876,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDefaultFontSizeBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsDefaultFontSizeTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDefaultFontSizeTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDefaultFontSizeTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDefaultFontSizeTestHelper(views.getContents1(), views.getClient1())); } // The test verifies that disabling images loading by setting @@ -887,63 +888,67 @@ public class AwSettingsTest extends AndroidWebViewTestBase { @SmallTest public void testLoadsImagesAutomaticallyWithCachedImage() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); - ContentSettings settings0 = getContentSettingsOnUiThread(views.getView0()); + ContentSettings settings0 = getContentSettingsOnUiThread(views.getContents0()); settings0.setJavaScriptEnabled(true); - ContentSettings settings1 = getContentSettingsOnUiThread(views.getView1()); + ContentSettings settings1 = getContentSettingsOnUiThread(views.getContents1()); settings1.setJavaScriptEnabled(true); ImagePageGenerator generator = new ImagePageGenerator(0, false); // First disable images loading and verify it. settings0.setLoadsImagesAutomatically(false); settings1.setLoadsImagesAutomatically(false); - loadDataSync(views.getView0(), + loadDataSync(views.getContents0(), views.getClient0().getOnPageFinishedHelper(), generator.getPageSource(), "text/html", false); - loadDataSync(views.getView1(), + loadDataSync(views.getContents1(), views.getClient1().getOnPageFinishedHelper(), generator.getPageSource(), "text/html", false); assertEquals(ImagePageGenerator.IMAGE_NOT_LOADED_STRING, - getTitleOnUiThread(views.getView0())); + getTitleOnUiThread(views.getContents0())); assertEquals(ImagePageGenerator.IMAGE_NOT_LOADED_STRING, - getTitleOnUiThread(views.getView1())); + getTitleOnUiThread(views.getContents1())); // Now enable images loading only for view 0. settings0.setLoadsImagesAutomatically(true); - loadDataSync(views.getView0(), + loadDataSync(views.getContents0(), views.getClient0().getOnPageFinishedHelper(), generator.getPageSource(), "text/html", false); - loadDataSync(views.getView1(), + loadDataSync(views.getContents1(), views.getClient1().getOnPageFinishedHelper(), generator.getPageSource(), "text/html", false); // Once the image has been cached by contentView0, it is available to contentView1. - assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(views.getView0())); - assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(views.getView1())); + assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, + getTitleOnUiThread(views.getContents0())); + assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, + getTitleOnUiThread(views.getContents1())); ImagePageGenerator generator1 = new ImagePageGenerator(1, false); // This is a new image. view 1 can't load it. - loadDataSync(views.getView1(), + loadDataSync(views.getContents1(), views.getClient1().getOnPageFinishedHelper(), generator1.getPageSource(), "text/html", false); assertEquals(ImagePageGenerator.IMAGE_NOT_LOADED_STRING, - getTitleOnUiThread(views.getView1())); - loadDataSync(views.getView0(), + getTitleOnUiThread(views.getContents1())); + loadDataSync(views.getContents0(), views.getClient0().getOnPageFinishedHelper(), generator1.getPageSource(), "text/html", false); - loadDataSync(views.getView1(), + loadDataSync(views.getContents1(), views.getClient1().getOnPageFinishedHelper(), generator1.getPageSource(), "text/html", false); // Once the image has been cached by contentViewCore0, it is available to contentViewCore1. - assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(views.getView0())); - assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(views.getView1())); + assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, + getTitleOnUiThread(views.getContents0())); + assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, + getTitleOnUiThread(views.getContents1())); } // The test verifies that after changing the LoadsImagesAutomatically @@ -953,26 +958,27 @@ public class AwSettingsTest extends AndroidWebViewTestBase { @Feature({"Android-WebView", "Preferences"}) public void testLoadsImagesAutomaticallyNoPageReload() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper(); - ContentSettings settings = getContentSettingsOnUiThread(contentView); + ContentSettings settings = getContentSettingsOnUiThread(awContents); settings.setJavaScriptEnabled(true); ImagePageGenerator generator = new ImagePageGenerator(0, false); settings.setLoadsImagesAutomatically(false); - loadDataSync(contentView, + loadDataSync(awContents, contentClient.getOnPageFinishedHelper(), generator.getPageSource(), "text/html", false); assertEquals(ImagePageGenerator.IMAGE_NOT_LOADED_STRING, - getTitleOnUiThread(contentView)); + getTitleOnUiThread(awContents)); settings.setLoadsImagesAutomatically(true); assertTrue(CriteriaHelper.pollForCriteria(new Criteria() { @Override public boolean isSatisfied() { try { return !ImagePageGenerator.IMAGE_NOT_LOADED_STRING.equals( - getTitleOnUiThread(contentView)); + getTitleOnUiThread(awContents)); } catch (Throwable t) { t.printStackTrace(); fail("Failed to getTitleOnUiThread: " + t.toString()); @@ -980,7 +986,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { } } }, TEST_TIMEOUT, CHECK_INTERVAL)); - assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(contentView)); + assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(awContents)); } @SmallTest @@ -989,9 +995,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( new AwSettingsLoadImagesAutomaticallyTestHelper( - views.getView0(), views.getClient0(), new ImagePageGenerator(0, true)), + views.getContents0(), views.getClient0(), new ImagePageGenerator(0, true)), new AwSettingsLoadImagesAutomaticallyTestHelper( - views.getView1(), views.getClient1(), new ImagePageGenerator(1, true))); + views.getContents1(), views.getClient1(), new ImagePageGenerator(1, true))); } @SmallTest @@ -1000,9 +1006,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( new AwSettingsLoadImagesAutomaticallyTestHelper( - views.getView0(), views.getClient0(), new ImagePageGenerator(0, true)), + views.getContents0(), views.getClient0(), new ImagePageGenerator(0, true)), new AwSettingsLoadImagesAutomaticallyTestHelper( - views.getView1(), views.getClient1(), new ImagePageGenerator(1, true))); + views.getContents1(), views.getClient1(), new ImagePageGenerator(1, true))); } @SmallTest @@ -1011,9 +1017,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( new AwSettingsLoadImagesAutomaticallyTestHelper( - views.getView0(), views.getClient0(), new ImagePageGenerator(0, true)), + views.getContents0(), views.getClient0(), new ImagePageGenerator(0, true)), new AwSettingsLoadImagesAutomaticallyTestHelper( - views.getView1(), views.getClient1(), new ImagePageGenerator(1, true))); + views.getContents1(), views.getClient1(), new ImagePageGenerator(1, true))); } @SmallTest @@ -1021,8 +1027,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDefaultTextEncodingNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsDefaultTextEncodingTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDefaultTextEncodingTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDefaultTextEncodingTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDefaultTextEncodingTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1030,8 +1036,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDefaultTextEncodingIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsDefaultTextEncodingTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDefaultTextEncodingTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDefaultTextEncodingTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDefaultTextEncodingTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1039,8 +1045,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDefaultTextEncodingBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsDefaultTextEncodingTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDefaultTextEncodingTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDefaultTextEncodingTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDefaultTextEncodingTestHelper(views.getContents1(), views.getClient1())); } // The test verifies that the default user agent string follows the format @@ -1053,9 +1059,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { @Feature({"Android-WebView", "Preferences"}) public void testUserAgentStringDefault() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + ContentSettings settings = getContentSettingsOnUiThread(awContents); final String actualUserAgentString = settings.getUserAgentString(); final String patternString = "Mozilla/5\\.0 \\(Linux;( U;)? Android ([^;]+);( (\\w+)-(\\w+);)?" + @@ -1081,9 +1088,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { @Feature({"Android-WebView", "Preferences"}) public void testUserAgentStringOverride() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + ContentSettings settings = getContentSettingsOnUiThread(awContents); final String defaultUserAgentString = settings.getUserAgentString(); // Check that an attempt to reset the default UA string has no effect. @@ -1110,10 +1118,12 @@ public class AwSettingsTest extends AndroidWebViewTestBase { @Feature({"Android-WebView", "Preferences"}) public void testUserAgentStringOverrideForHistory() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + final ContentViewCore contentView = testContainerView.getContentViewCore(); CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper(); - ContentSettings settings = getContentSettingsOnUiThread(contentView); + ContentSettings settings = getContentSettingsOnUiThread(awContents); settings.setJavaScriptEnabled(true); final String defaultUserAgentString = settings.getUserAgentString(); final String customUserAgentString = "ContentSettingsTest"; @@ -1129,19 +1139,19 @@ public class AwSettingsTest extends AndroidWebViewTestBase { final String page2 = String.format(pageTemplate, page2Title); settings.setUserAgentString(customUserAgentString); loadDataSync( - contentView, contentClient.getOnPageFinishedHelper(), page1, "text/html", false); - assertEquals(page1Title + customUserAgentString, getTitleOnUiThread(contentView)); + awContents, contentClient.getOnPageFinishedHelper(), page1, "text/html", false); + assertEquals(page1Title + customUserAgentString, getTitleOnUiThread(awContents)); loadDataSync( - contentView, contentClient.getOnPageFinishedHelper(), page2, "text/html", false); - assertEquals(page2Title + customUserAgentString, getTitleOnUiThread(contentView)); + awContents, contentClient.getOnPageFinishedHelper(), page2, "text/html", false); + assertEquals(page2Title + customUserAgentString, getTitleOnUiThread(awContents)); settings.setUserAgentString(null); // Must not cause any changes until the next page loading. - assertEquals(page2Title + customUserAgentString, getTitleOnUiThread(contentView)); + assertEquals(page2Title + customUserAgentString, getTitleOnUiThread(awContents)); HistoryUtils.goBackSync(getInstrumentation(), contentView, onPageFinishedHelper); - assertEquals(page1Title + defaultUserAgentString, getTitleOnUiThread(contentView)); + assertEquals(page1Title + defaultUserAgentString, getTitleOnUiThread(awContents)); HistoryUtils.goForwardSync(getInstrumentation(), contentView, onPageFinishedHelper); - assertEquals(page2Title + defaultUserAgentString, getTitleOnUiThread(contentView)); + assertEquals(page2Title + defaultUserAgentString, getTitleOnUiThread(awContents)); } @SmallTest @@ -1149,8 +1159,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testUserAgentStringNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsUserAgentStringTestHelper(views.getView0(), views.getClient0()), - new AwSettingsUserAgentStringTestHelper(views.getView1(), views.getClient1())); + new AwSettingsUserAgentStringTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsUserAgentStringTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1158,8 +1168,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testUserAgentStringIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsUserAgentStringTestHelper(views.getView0(), views.getClient0()), - new AwSettingsUserAgentStringTestHelper(views.getView1(), views.getClient1())); + new AwSettingsUserAgentStringTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsUserAgentStringTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1167,8 +1177,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testUserAgentStringBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsUserAgentStringTestHelper(views.getView0(), views.getClient0()), - new AwSettingsUserAgentStringTestHelper(views.getView1(), views.getClient1())); + new AwSettingsUserAgentStringTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsUserAgentStringTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1176,8 +1186,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDomStorageEnabledNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsDomStorageEnabledTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDomStorageEnabledTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDomStorageEnabledTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDomStorageEnabledTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1185,8 +1195,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDomStorageEnabledIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsDomStorageEnabledTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDomStorageEnabledTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDomStorageEnabledTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDomStorageEnabledTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1194,8 +1204,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testDomStorageEnabledBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsDomStorageEnabledTestHelper(views.getView0(), views.getClient0()), - new AwSettingsDomStorageEnabledTestHelper(views.getView1(), views.getClient1())); + new AwSettingsDomStorageEnabledTestHelper(views.getContents0(), views.getClient0()), + new AwSettingsDomStorageEnabledTestHelper(views.getContents1(), views.getClient1())); } @SmallTest @@ -1203,8 +1213,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testUniversalAccessFromFilesNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsUniversalAccessFromFilesTestHelper(views.getView0(), views.getClient0()), - new AwSettingsUniversalAccessFromFilesTestHelper(views.getView1(), views.getClient1())); + new AwSettingsUniversalAccessFromFilesTestHelper(views.getContents0(), + views.getClient0()), + new AwSettingsUniversalAccessFromFilesTestHelper(views.getContents1(), + views.getClient1())); } @SmallTest @@ -1212,8 +1224,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testUniversalAccessFromFilesIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsUniversalAccessFromFilesTestHelper(views.getView0(), views.getClient0()), - new AwSettingsUniversalAccessFromFilesTestHelper(views.getView1(), views.getClient1())); + new AwSettingsUniversalAccessFromFilesTestHelper(views.getContents0(), + views.getClient0()), + new AwSettingsUniversalAccessFromFilesTestHelper(views.getContents1(), + views.getClient1())); } @SmallTest @@ -1221,8 +1235,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testUniversalAccessFromFilesBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsUniversalAccessFromFilesTestHelper(views.getView0(), views.getClient0()), - new AwSettingsUniversalAccessFromFilesTestHelper(views.getView1(), views.getClient1())); + new AwSettingsUniversalAccessFromFilesTestHelper(views.getContents0(), + views.getClient0()), + new AwSettingsUniversalAccessFromFilesTestHelper(views.getContents1(), + views.getClient1())); } // This test verifies that local image resources can be loaded from file: @@ -1233,14 +1249,15 @@ public class AwSettingsTest extends AndroidWebViewTestBase { final String imageContainerUrl = UrlUtils.getTestFileUrl("webview/image_access.html"); final String imageHeight = "16"; final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + ContentSettings settings = getContentSettingsOnUiThread(awContents); settings.setJavaScriptEnabled(true); settings.setAllowUniversalAccessFromFileURLs(false); settings.setAllowFileAccessFromFileURLs(false); - loadUrlSync(contentView, contentClient.getOnPageFinishedHelper(), imageContainerUrl); - assertEquals(imageHeight, getTitleOnUiThread(contentView)); + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), imageContainerUrl); + assertEquals(imageHeight, getTitleOnUiThread(awContents)); } @SmallTest @@ -1249,9 +1266,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( new AwSettingsFileAccessFromFilesIframeTestHelper( - views.getView0(), views.getClient0()), + views.getContents0(), views.getClient0()), new AwSettingsFileAccessFromFilesIframeTestHelper( - views.getView1(), views.getClient1())); + views.getContents1(), views.getClient1())); } @SmallTest @@ -1260,9 +1277,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( new AwSettingsFileAccessFromFilesIframeTestHelper( - views.getView0(), views.getClient0()), + views.getContents0(), views.getClient0()), new AwSettingsFileAccessFromFilesIframeTestHelper( - views.getView1(), views.getClient1())); + views.getContents1(), views.getClient1())); } @SmallTest @@ -1271,9 +1288,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( new AwSettingsFileAccessFromFilesIframeTestHelper( - views.getView0(), views.getClient0()), + views.getContents0(), views.getClient0()), new AwSettingsFileAccessFromFilesIframeTestHelper( - views.getView1(), views.getClient1())); + views.getContents1(), views.getClient1())); } @SmallTest @@ -1281,8 +1298,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testFileAccessFromFilesXhrNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsFileAccessFromFilesXhrTestHelper(views.getView0(), views.getClient0()), - new AwSettingsFileAccessFromFilesXhrTestHelper(views.getView1(), views.getClient1())); + new AwSettingsFileAccessFromFilesXhrTestHelper(views.getContents0(), + views.getClient0()), + new AwSettingsFileAccessFromFilesXhrTestHelper(views.getContents1(), + views.getClient1())); } @SmallTest @@ -1290,8 +1309,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testFileAccessFromFilesXhrIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsFileAccessFromFilesXhrTestHelper(views.getView0(), views.getClient0()), - new AwSettingsFileAccessFromFilesXhrTestHelper(views.getView1(), views.getClient1())); + new AwSettingsFileAccessFromFilesXhrTestHelper(views.getContents0(), + views.getClient0()), + new AwSettingsFileAccessFromFilesXhrTestHelper(views.getContents1(), + views.getClient1())); } @SmallTest @@ -1299,8 +1320,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testFileAccessFromFilesXhrBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsFileAccessFromFilesXhrTestHelper(views.getView0(), views.getClient0()), - new AwSettingsFileAccessFromFilesXhrTestHelper(views.getView1(), views.getClient1())); + new AwSettingsFileAccessFromFilesXhrTestHelper(views.getContents0(), + views.getClient0()), + new AwSettingsFileAccessFromFilesXhrTestHelper(views.getContents1(), + views.getClient1())); } /** @@ -1312,8 +1335,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testFileUrlAccessNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsFileUrlAccessTestHelper(views.getView0(), views.getClient0(), 0), - new AwSettingsFileUrlAccessTestHelper(views.getView1(), views.getClient1(), 1)); + new AwSettingsFileUrlAccessTestHelper(views.getContents0(), views.getClient0(), 0), + new AwSettingsFileUrlAccessTestHelper(views.getContents1(), views.getClient1(), 1)); } /** @@ -1325,8 +1348,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testFileUrlAccessIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsFileUrlAccessTestHelper(views.getView0(), views.getClient0(), 0), - new AwSettingsFileUrlAccessTestHelper(views.getView1(), views.getClient1(), 1)); + new AwSettingsFileUrlAccessTestHelper(views.getContents0(), views.getClient0(), 0), + new AwSettingsFileUrlAccessTestHelper(views.getContents1(), views.getClient1(), 1)); } /** @@ -1338,8 +1361,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testFileUrlAccessBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsFileUrlAccessTestHelper(views.getView0(), views.getClient0(), 0), - new AwSettingsFileUrlAccessTestHelper(views.getView1(), views.getClient1(), 1)); + new AwSettingsFileUrlAccessTestHelper(views.getContents0(), views.getClient0(), 0), + new AwSettingsFileUrlAccessTestHelper(views.getContents1(), views.getClient1(), 1)); } /** @@ -1351,8 +1374,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testContentUrlAccessNormal() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, NORMAL_VIEW); runPerViewSettingsTest( - new AwSettingsContentUrlAccessTestHelper(views.getView0(), views.getClient0(), 0), - new AwSettingsContentUrlAccessTestHelper(views.getView1(), views.getClient1(), 1)); + new AwSettingsContentUrlAccessTestHelper(views.getContents0(), views.getClient0(), 0), + new AwSettingsContentUrlAccessTestHelper(views.getContents1(), views.getClient1(), 1)); } /** @@ -1364,8 +1387,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testContentUrlAccessIncognito() throws Throwable { ViewPair views = createViews(INCOGNITO_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsContentUrlAccessTestHelper(views.getView0(), views.getClient0(), 0), - new AwSettingsContentUrlAccessTestHelper(views.getView1(), views.getClient1(), 1)); + new AwSettingsContentUrlAccessTestHelper(views.getContents0(), views.getClient0(), 0), + new AwSettingsContentUrlAccessTestHelper(views.getContents1(), views.getClient1(), 1)); } /** @@ -1377,16 +1400,17 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public void testContentUrlAccessBoth() throws Throwable { ViewPair views = createViews(NORMAL_VIEW, INCOGNITO_VIEW); runPerViewSettingsTest( - new AwSettingsContentUrlAccessTestHelper(views.getView0(), views.getClient0(), 0), - new AwSettingsContentUrlAccessTestHelper(views.getView1(), views.getClient1(), 1)); + new AwSettingsContentUrlAccessTestHelper(views.getContents0(), views.getClient0(), 0), + new AwSettingsContentUrlAccessTestHelper(views.getContents1(), views.getClient1(), 1)); } @SmallTest @Feature({"Android-WebView", "Preferences", "Navigation"}) public void testBlockingContentUrlsFromDataUrls() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); final String target = "content_from_data"; final String page = "<html><body>" + "<img src=\"" + @@ -1394,7 +1418,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { "</body></html>"; resetResourceRequestCountInContentProvider(target); loadDataSync( - contentView, + awContents, contentClient.getOnPageFinishedHelper(), page, "text/html", @@ -1404,8 +1428,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { private void doTestContentUrlFromFile(boolean addQueryParameters) throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); final String target = "content_from_file_" + addQueryParameters; Context context = getInstrumentation().getTargetContext(); final String fileName = context.getCacheDir() + "/" + target + ".html"; @@ -1418,7 +1443,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { (addQueryParameters ? "?weather=sunny&life=good" : "") + "\">"); loadUrlSync( - contentView, + awContents, contentClient.getOnPageFinishedHelper(), "file:///" + fileName); ensureResourceRequestCountInContentProvider(target, 1); @@ -1452,28 +1477,30 @@ public class AwSettingsTest extends AndroidWebViewTestBase { @Feature({"Android-WebView", "Preferences"}) public void testBlockNetworkImagesDoesNotBlockDataUrlImage() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - final ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + final ContentSettings settings = getContentSettingsOnUiThread(awContents); ImagePageGenerator generator = new ImagePageGenerator(0, false); settings.setJavaScriptEnabled(true); settings.setImagesEnabled(false); - loadDataSync(contentView, + loadDataSync(awContents, contentClient.getOnPageFinishedHelper(), generator.getPageSource(), "text/html", false); - assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(contentView)); + assertEquals(ImagePageGenerator.IMAGE_LOADED_STRING, getTitleOnUiThread(awContents)); } @SmallTest @Feature({"Android-WebView", "Preferences"}) public void testBlockNetworkImagesBlocksNetworkImageAndReloadInPlace() throws Throwable { final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - final ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + final ContentSettings settings = getContentSettingsOnUiThread(awContents); settings.setJavaScriptEnabled(true); ImagePageGenerator generator = new ImagePageGenerator(0, false); @@ -1489,9 +1516,9 @@ public class AwSettingsTest extends AndroidWebViewTestBase { final String httpImageUrl = webServer.setResponse(pagePath, httpUrlImageHtml, null); settings.setImagesEnabled(false); - loadUrlSync(contentView, contentClient.getOnPageFinishedHelper(), httpImageUrl); + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), httpImageUrl); assertEquals(ImagePageGenerator.IMAGE_NOT_LOADED_STRING, - getTitleOnUiThread(contentView)); + getTitleOnUiThread(awContents)); settings.setImagesEnabled(true); assertTrue(CriteriaHelper.pollForCriteria(new Criteria() { @@ -1499,7 +1526,7 @@ public class AwSettingsTest extends AndroidWebViewTestBase { public boolean isSatisfied() { try { return ImagePageGenerator.IMAGE_NOT_LOADED_STRING.equals( - getTitleOnUiThread(contentView)); + getTitleOnUiThread(awContents)); } catch (Throwable t) { t.printStackTrace(); fail("Failed to getTitleOnUIThread: " + t.toString()); @@ -1520,14 +1547,15 @@ public class AwSettingsTest extends AndroidWebViewTestBase { // below. final String expectedTitle = "Asset File"; final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); try { useTestResourceContext(); - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), "file:///android_asset/asset_file.html"); - assertEquals(expectedTitle, getTitleOnUiThread(contentView)); + assertEquals(expectedTitle, getTitleOnUiThread(awContents)); } finally { resetResourceContext(); } @@ -1541,14 +1569,15 @@ public class AwSettingsTest extends AndroidWebViewTestBase { // below. final String expectedTitle = "Resource File"; final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); try { useTestResourceContext(); - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), "file:///android_res/raw/resource_file.html"); - assertEquals(expectedTitle, getTitleOnUiThread(contentView)); + assertEquals(expectedTitle, getTitleOnUiThread(awContents)); } finally { resetResourceContext(); } @@ -1562,16 +1591,17 @@ public class AwSettingsTest extends AndroidWebViewTestBase { // below. final String expectedTitle = "Asset File"; final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - final ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + final ContentSettings settings = getContentSettingsOnUiThread(awContents); try { useTestResourceContext(); settings.setAllowFileAccess(false); - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), "file:///android_asset/asset_file.html"); - assertEquals(expectedTitle, getTitleOnUiThread(contentView)); + assertEquals(expectedTitle, getTitleOnUiThread(awContents)); } finally { resetResourceContext(); } @@ -1585,45 +1615,46 @@ public class AwSettingsTest extends AndroidWebViewTestBase { // below. final String expectedTitle = "Resource File"; final TestAwContentsClient contentClient = new TestAwContentsClient(); - final ContentViewCore contentView = - createAwTestContainerViewOnMainSync(false, contentClient).getContentViewCore(); - final ContentSettings settings = getContentSettingsOnUiThread(contentView); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentClient); + final AwContents awContents = testContainerView.getAwContents(); + final ContentSettings settings = getContentSettingsOnUiThread(awContents); try { useTestResourceContext(); settings.setAllowFileAccess(false); - loadUrlSync(contentView, + loadUrlSync(awContents, contentClient.getOnPageFinishedHelper(), "file:///android_res/raw/resource_file.html"); - assertEquals(expectedTitle, getTitleOnUiThread(contentView)); + assertEquals(expectedTitle, getTitleOnUiThread(awContents)); } finally { resetResourceContext(); } } class ViewPair { - private final ContentViewCore view0; + private final AwContents contents0; private final TestAwContentsClient client0; - private final ContentViewCore view1; + private final AwContents contents1; private final TestAwContentsClient client1; - ViewPair(ContentViewCore view0, TestAwContentsClient client0, - ContentViewCore view1, TestAwContentsClient client1) { - this.view0 = view0; + ViewPair(AwContents contents0, TestAwContentsClient client0, + AwContents contents1, TestAwContentsClient client1) { + this.contents0 = contents0; this.client0 = client0; - this.view1 = view1; + this.contents1 = contents1; this.client1 = client1; } - ContentViewCore getView0() { - return view0; + AwContents getContents0() { + return contents0; } TestAwContentsClient getClient0() { return client0; } - ContentViewCore getView1() { - return view1; + AwContents getContents1() { + return contents1; } TestAwContentsClient getClient1() { @@ -1684,10 +1715,10 @@ public class AwSettingsTest extends AndroidWebViewTestBase { TestAwContentsClient client1 = new TestAwContentsClient(); return new ViewPair( createAwTestContainerViewOnMainSync( - firstIsIncognito, client0).getContentViewCore(), + firstIsIncognito, client0).getAwContents(), client0, createAwTestContainerViewOnMainSync( - secondIsIncognito, client1).getContentViewCore(), + secondIsIncognito, client1).getAwContents(), client1); } @@ -1732,8 +1763,8 @@ public class AwSettingsTest extends AndroidWebViewTestBase { final TestAwContentsClient contentClient = new TestAwContentsClient(); final AwTestContainerView testContainer = createAwTestContainerViewOnMainSync(false, contentClient); - final ContentViewCore contentView = testContainer.getContentViewCore(); - final ContentSettings contentSettings = getContentSettingsOnUiThread(contentView); + final AwContents awContents = testContainer.getAwContents(); + final ContentSettings contentSettings = getContentSettingsOnUiThread(awContents); final AwSettings awSettings = getAwSettingsOnUiThread(testContainer.getAwContents()); contentSettings.setJavaScriptEnabled(true); ImagePageGenerator generator = new ImagePageGenerator(0, false); @@ -1760,20 +1791,20 @@ public class AwSettingsTest extends AndroidWebViewTestBase { // Actual test. Blocking should trigger onerror handler. awSettings.setBlockNetworkLoads(true); loadUrlSync( - contentView, + awContents, contentClient.getOnPageFinishedHelper(), "file:///" + fileName); assertEquals(0, webServer.getRequestCount(httpPath)); - assertEquals("img_onerror_fired", getTitleOnUiThread(contentView)); + assertEquals("img_onerror_fired", getTitleOnUiThread(awContents)); // Unblock should load normally. awSettings.setBlockNetworkLoads(false); loadUrlSync( - contentView, + awContents, contentClient.getOnPageFinishedHelper(), "file:///" + fileName); assertEquals(1, webServer.getRequestCount(httpPath)); - assertEquals("img_onload_fired", getTitleOnUiThread(contentView)); + assertEquals("img_onload_fired", getTitleOnUiThread(awContents)); } finally { if (fileName != null) TestFileUtil.deleteFile(fileName); if (webServer != null) webServer.shutdown(); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ClearHistoryTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ClearHistoryTest.java index 3ba0bf4..569a7b8 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ClearHistoryTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ClearHistoryTest.java @@ -6,6 +6,7 @@ package org.chromium.android_webview.test; import android.test.suitebuilder.annotation.SmallTest; +import org.chromium.android_webview.AwContents; import org.chromium.base.test.util.Feature; import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.test.util.HistoryUtils; @@ -27,12 +28,14 @@ public class ClearHistoryTest extends AndroidWebViewTestBase { @Feature({"History", "Main"}) public void testClearHistory() throws Throwable { final TestAwContentsClient contentsClient = new TestAwContentsClient(); - final ContentViewCore contentViewCore = - createAwTestContainerViewOnMainSync(contentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(contentsClient); + final AwContents awContents = testContainerView.getAwContents(); + final ContentViewCore contentViewCore = testContainerView.getContentViewCore(); OnPageFinishedHelper onPageFinishedHelper = contentsClient.getOnPageFinishedHelper(); for (int i = 0; i < 3; i++) { - loadUrlSync(contentViewCore, onPageFinishedHelper, URLS[i]); + loadUrlSync(awContents, onPageFinishedHelper, URLS[i]); } HistoryUtils.goBackSync(getInstrumentation(), contentViewCore, onPageFinishedHelper); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ClientAddMessageToConsoleTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ClientAddMessageToConsoleTest.java index f374b79..4c61559 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ClientAddMessageToConsoleTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ClientAddMessageToConsoleTest.java @@ -7,9 +7,9 @@ package org.chromium.android_webview.test; import android.test.suitebuilder.annotation.SmallTest; import android.util.Log; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.AwWebContentsDelegate; import org.chromium.base.test.util.Feature; -import org.chromium.content.browser.ContentViewCore; /** * Tests for the ContentViewClient.addMessageToConsole() method. @@ -24,19 +24,20 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { private static final String TEST_MESSAGE_TWO = "The second test message."; private TestAwContentsClient mContentsClient; - private ContentViewCore mContentViewCore; + private AwContents mAwContents; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); getInstrumentation().runOnMainSync(new Runnable() { @Override public void run() { - mContentViewCore.getContentSettings().setJavaScriptEnabled(true); + mAwContents.getContentViewCore().getContentSettings().setJavaScriptEnabled(true); } }); } @@ -63,7 +64,7 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { mContentsClient.getAddMessageToConsoleHelper(); int callCount = addMessageToConsoleHelper.getCallCount(); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getLogMessageJavaScriptData("error", "msg"), "text/html", false); addMessageToConsoleHelper.waitForCallback(callCount); @@ -71,7 +72,7 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { addMessageToConsoleHelper.getLevel()); callCount = addMessageToConsoleHelper.getCallCount(); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getLogMessageJavaScriptData("warn", "msg"), "text/html", false); addMessageToConsoleHelper.waitForCallback(callCount); @@ -79,7 +80,7 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { addMessageToConsoleHelper.getLevel()); callCount = addMessageToConsoleHelper.getCallCount(); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getLogMessageJavaScriptData("log", "msg"), "text/html", false); addMessageToConsoleHelper.waitForCallback(callCount); @@ -97,7 +98,7 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { mContentsClient.getAddMessageToConsoleHelper(); int callCount = addMessageToConsoleHelper.getCallCount(); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getLogMessageJavaScriptData("log", TEST_MESSAGE_ONE), "text/html", false); Log.w("test", getLogMessageJavaScriptData("log", TEST_MESSAGE_ONE)); @@ -105,7 +106,7 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { assertEquals(TEST_MESSAGE_ONE, addMessageToConsoleHelper.getMessage()); callCount = addMessageToConsoleHelper.getCallCount(); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getLogMessageJavaScriptData("log", TEST_MESSAGE_TWO), "text/html", false); addMessageToConsoleHelper.waitForCallback(callCount); @@ -120,7 +121,7 @@ public class ClientAddMessageToConsoleTest extends AndroidWebViewTestBase { int callCount = addMessageToConsoleHelper.getCallCount(); String data = getLogMessageJavaScriptData("log", TEST_MESSAGE_ONE); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), data, "text/html", false); addMessageToConsoleHelper.waitForCallback(callCount); assertTrue("Url [" + addMessageToConsoleHelper.getSourceId() + "] expected to end with [" + diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnPageFinishedTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnPageFinishedTest.java index 9d9cc16..ff9e660 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnPageFinishedTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnPageFinishedTest.java @@ -7,9 +7,9 @@ package org.chromium.android_webview.test; import android.test.FlakyTest; import android.test.suitebuilder.annotation.MediumTest; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.test.util.TestWebServer; import org.chromium.base.test.util.Feature; -import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.test.util.TestCallbackHelperContainer; /** @@ -18,14 +18,15 @@ import org.chromium.content.browser.test.util.TestCallbackHelperContainer; public class ClientOnPageFinishedTest extends AndroidWebViewTestBase { private TestAwContentsClient mContentsClient; - private ContentViewCore mContentViewCore; + private AwContents mAwContents; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); } @MediumTest @@ -36,7 +37,7 @@ public class ClientOnPageFinishedTest extends AndroidWebViewTestBase { String html = "<html><body>Simple page.</body></html>"; int currentCallCount = onPageFinishedHelper.getCallCount(); - loadDataAsync(mContentViewCore, html, "text/html", false); + loadDataAsync(mAwContents, html, "text/html", false); onPageFinishedHelper.waitForCallback(currentCallCount); assertEquals("data:text/html," + html, onPageFinishedHelper.getUrl()); @@ -57,7 +58,7 @@ public class ClientOnPageFinishedTest extends AndroidWebViewTestBase { String url = "http://man.id.be.really.surprised.if.this.address.existed.blah/"; int onReceivedErrorCallCount = onReceivedErrorHelper.getCallCount(); int onPageFinishedCallCount = onPageFinishedHelper.getCallCount(); - loadUrlAsync(mContentViewCore, url); + loadUrlAsync(mAwContents, url); onReceivedErrorHelper.waitForCallback(onReceivedErrorCallCount); onPageFinishedHelper.waitForCallback(onPageFinishedCallCount); @@ -83,7 +84,7 @@ public class ClientOnPageFinishedTest extends AndroidWebViewTestBase { assertEquals(0, onPageFinishedHelper.getCallCount()); final int pageWithSubresourcesCallCount = onPageFinishedHelper.getCallCount(); - loadDataAsync(mContentViewCore, + loadDataAsync(mAwContents, "<html><iframe src=\"" + testPath + "\" /></html>", "text/html", false); @@ -95,7 +96,7 @@ public class ClientOnPageFinishedTest extends AndroidWebViewTestBase { // we get is for the synchronizationUrl we know that the previous load did not schedule // a callback for the iframe. final int synchronizationPageCallCount = onPageFinishedHelper.getCallCount(); - loadUrlAsync(mContentViewCore, syncUrl); + loadUrlAsync(mAwContents, syncUrl); onPageFinishedHelper.waitForCallback(synchronizationPageCallCount); assertEquals(syncUrl, onPageFinishedHelper.getUrl()); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnReceivedErrorTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnReceivedErrorTest.java index a4a2be2..4a99f6a 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnReceivedErrorTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ClientOnReceivedErrorTest.java @@ -7,6 +7,7 @@ package org.chromium.android_webview.test; import android.test.FlakyTest; import android.test.suitebuilder.annotation.MediumTest; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.AndroidProtocolHandler; import org.chromium.android_webview.ErrorCodeConversionHelper; import org.chromium.base.test.util.DisabledTest; @@ -20,14 +21,15 @@ import org.chromium.content.browser.test.util.TestCallbackHelperContainer; public class ClientOnReceivedErrorTest extends AndroidWebViewTestBase { private TestAwContentsClient mContentsClient; - private ContentViewCore mContentViewCore; + private AwContents mAwContents; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); } /*@MediumTest @@ -41,7 +43,7 @@ public class ClientOnReceivedErrorTest extends AndroidWebViewTestBase { String url = "http://man.id.be.really.surprised.if.this.address.existed.blah/"; int onReceivedErrorCallCount = onReceivedErrorHelper.getCallCount(); - loadUrlAsync(mContentViewCore, url); + loadUrlAsync(mAwContents, url); onReceivedErrorHelper.waitForCallback(onReceivedErrorCallCount); assertEquals(ErrorCodeConversionHelper.ERROR_HOST_LOOKUP, @@ -62,7 +64,7 @@ public class ClientOnReceivedErrorTest extends AndroidWebViewTestBase { String url = "foo://some/resource"; int onReceivedErrorCallCount = onReceivedErrorHelper.getCallCount(); - loadUrlAsync(mContentViewCore, url); + loadUrlAsync(mAwContents, url); onReceivedErrorHelper.waitForCallback(onReceivedErrorCallCount); assertEquals(ErrorCodeConversionHelper.ERROR_UNSUPPORTED_SCHEME, @@ -80,7 +82,7 @@ public class ClientOnReceivedErrorTest extends AndroidWebViewTestBase { mContentsClient.getOnPageFinishedHelper(); int currentCallCount = onPageFinishedHelper.getCallCount(); - loadDataAsync(mContentViewCore, + loadDataAsync(mAwContents, "<html><iframe src=\"http//invalid.url.co/\" /></html>", "text/html", false); @@ -98,7 +100,7 @@ public class ClientOnReceivedErrorTest extends AndroidWebViewTestBase { final String url = "file:///android_asset/does_not_exist.html"; int onReceivedErrorCallCount = onReceivedErrorHelper.getCallCount(); useTestResourceContext(); - loadUrlAsync(mContentViewCore, url); + loadUrlAsync(mAwContents, url); onReceivedErrorHelper.waitForCallback(onReceivedErrorCallCount); assertEquals(ErrorCodeConversionHelper.ERROR_UNKNOWN, @@ -119,7 +121,7 @@ public class ClientOnReceivedErrorTest extends AndroidWebViewTestBase { final String url = "file:///android_res/raw/does_not_exist.html"; int onReceivedErrorCallCount = onReceivedErrorHelper.getCallCount(); useTestResourceContext(); - loadUrlAsync(mContentViewCore, url); + loadUrlAsync(mAwContents, url); onReceivedErrorHelper.waitForCallback(onReceivedErrorCallCount); assertEquals(ErrorCodeConversionHelper.ERROR_UNKNOWN, diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewMiscTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewMiscTest.java index ffac55a..0fc0282 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewMiscTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewMiscTest.java @@ -14,6 +14,7 @@ import android.net.Proxy; import android.test.mock.MockContext; import android.test.suitebuilder.annotation.SmallTest; +import org.chromium.android_webview.AwContents; import org.chromium.base.test.util.DisabledTest; import org.chromium.base.test.util.Feature; import org.chromium.base.ThreadUtils; @@ -31,14 +32,17 @@ import java.util.concurrent.Callable; public class ContentViewMiscTest extends AndroidWebViewTestBase { private TestAwContentsClient mContentsClient; + private AwContents mAwContents; private ContentViewCore mContentViewCore; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); + mContentViewCore = testContainerView.getContentViewCore(); } /** @@ -53,7 +57,7 @@ public class ContentViewMiscTest extends AndroidWebViewTestBase { .append("</body></html>"); // Test flinging in the y axis - loadUrlSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadUrlSync(mAwContents , mContentsClient.getOnPageFinishedHelper(), testPage.toString()); assertEquals(0, mContentViewCore.getNativeScrollYForTest()); ThreadUtils.runOnUiThread(new Runnable() { @@ -151,7 +155,7 @@ public class ContentViewMiscTest extends AndroidWebViewTestBase { */ @DisabledTest public void testSetGetBackgroundColor() throws Throwable { - loadUrlSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), "about:blank"); + loadUrlSync(mAwContents , mContentsClient.getOnPageFinishedHelper(), "about:blank"); ThreadUtils.runOnUiThreadBlocking(new Runnable() { @Override public void run() { diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewZoomTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewZoomTest.java index b1c6451..9409ff7 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewZoomTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/ContentViewZoomTest.java @@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.SmallTest; import android.view.View; import android.view.ViewConfiguration; +import org.chromium.android_webview.AwContents; import org.chromium.base.ThreadUtils; import org.chromium.base.test.util.DisabledTest; import org.chromium.base.test.util.Feature; @@ -29,14 +30,17 @@ public class ContentViewZoomTest extends AndroidWebViewTestBase { private static final int CHECK_INTERVAL_MS = 100; private TestAwContentsClient mContentsClient; + private AwContents mAwContents; private ContentViewCore mContentViewCore; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); + mContentViewCore = testContainerView.getContentViewCore(); } private String getZoomableHtml() { @@ -189,11 +193,11 @@ public class ContentViewZoomTest extends AndroidWebViewTestBase { } private void runMagnificationTest(boolean supportZoom) throws Throwable { - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getZoomableHtml(), "text/html", false); // It takes some time for scaling to settle down. waitUntilCanNotZoomOut(); - getContentSettingsOnUiThread(mContentViewCore).setSupportZoom(supportZoom); + getContentSettingsOnUiThread(mAwContents).setSupportZoom(supportZoom); assertTrue("Should be able to zoom in", canZoomInOnUiThread()); assertFalse("Should not be able to zoom out", canZoomOutOnUiThread()); @@ -233,18 +237,18 @@ public class ContentViewZoomTest extends AndroidWebViewTestBase { @SmallTest @Feature({"Android-WebView"}) public void testZoomUsingMultiTouch() throws Throwable { - ContentSettings webSettings = getContentSettingsOnUiThread(mContentViewCore); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + ContentSettings webSettings = getContentSettingsOnUiThread(mAwContents); + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getZoomableHtml(), "text/html", false); assertTrue(webSettings.supportZoom()); assertFalse(webSettings.getBuiltInZoomControls()); assertFalse(isMultiTouchZoomSupportedOnUiThread()); - getContentSettingsOnUiThread(mContentViewCore).setBuiltInZoomControls(true); + getContentSettingsOnUiThread(mAwContents).setBuiltInZoomControls(true); assertTrue(isMultiTouchZoomSupportedOnUiThread()); - getContentSettingsOnUiThread(mContentViewCore).setSupportZoom(false); + getContentSettingsOnUiThread(mAwContents).setSupportZoom(false); assertFalse(isMultiTouchZoomSupportedOnUiThread()); } @@ -255,8 +259,8 @@ public class ContentViewZoomTest extends AndroidWebViewTestBase { */ @FlakyTest public void testZoomControls() throws Throwable { - ContentSettings webSettings = getContentSettingsOnUiThread(mContentViewCore); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + ContentSettings webSettings = getContentSettingsOnUiThread(mAwContents); + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getZoomableHtml(), "text/html", false); assertTrue(webSettings.supportZoom()); @@ -279,8 +283,8 @@ public class ContentViewZoomTest extends AndroidWebViewTestBase { @SmallTest @Feature({"Android-WebView"}) public void testZoomControlsOnNonZoomableContent() throws Throwable { - ContentSettings webSettings = getContentSettingsOnUiThread(mContentViewCore); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + ContentSettings webSettings = getContentSettingsOnUiThread(mAwContents); + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getNonZoomableHtml(), "text/html", false); // ContentView must update itself according to the viewport setup. @@ -302,8 +306,8 @@ public class ContentViewZoomTest extends AndroidWebViewTestBase { */ @DisabledTest public void testZoomControlsOnOrientationChange() throws Throwable { - ContentSettings webSettings = getContentSettingsOnUiThread(mContentViewCore); - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + ContentSettings webSettings = getContentSettingsOnUiThread(mAwContents); + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), getZoomableHtml(), "text/html", false); assertTrue(webSettings.supportZoom()); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java index 6309296..24f8ded 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java @@ -9,10 +9,10 @@ import android.test.suitebuilder.annotation.MediumTest; import android.test.suitebuilder.annotation.SmallTest; import android.util.Pair; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.CookieManager; import org.chromium.android_webview.test.util.TestWebServer; import org.chromium.base.test.util.Feature; -import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.test.util.Criteria; import org.chromium.content.browser.test.util.CriteriaHelper; import org.chromium.content.browser.test.util.TestCallbackHelperContainer.OnEvaluateJavaScriptResultHelper; @@ -31,18 +31,19 @@ import java.util.concurrent.atomic.AtomicInteger; public class CookieManagerTest extends AndroidWebViewTestBase { private CookieManager mCookieManager; - private TestAwContentsClient mContentViewClient; - private ContentViewCore mContentViewCore; + private TestAwContentsClient mContentsClient; + private AwContents mAwContents; @Override protected void setUp() throws Exception { super.setUp(); mCookieManager = new CookieManager(); - mContentViewClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentViewClient).getContentViewCore(); - mContentViewCore.getContentSettings().setJavaScriptEnabled(true); + mContentsClient = new TestAwContentsClient(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); + mAwContents.getContentViewCore().getContentSettings().setJavaScriptEnabled(true); assertNotNull(mCookieManager); } @@ -72,7 +73,7 @@ public class CookieManagerTest extends AndroidWebViewTestBase { assertFalse(mCookieManager.acceptCookie()); assertFalse(mCookieManager.hasCookies()); - loadUrlSync(mContentViewCore, mContentViewClient.getOnPageFinishedHelper(), url); + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), url); setCookie("test1", "value1"); assertNull(mCookieManager.getCookie(url)); @@ -80,14 +81,14 @@ public class CookieManagerTest extends AndroidWebViewTestBase { responseHeaders.add( Pair.create("Set-Cookie", "header-test1=header-value1; path=" + path)); url = webServer.setResponse(path, responseStr, responseHeaders); - loadUrlSync(mContentViewCore, mContentViewClient.getOnPageFinishedHelper(), url); + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), url); assertNull(mCookieManager.getCookie(url)); mCookieManager.setAcceptCookie(true); assertTrue(mCookieManager.acceptCookie()); url = webServer.setResponse(path, responseStr, null); - loadUrlSync(mContentViewCore, mContentViewClient.getOnPageFinishedHelper(), url); + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), url); setCookie("test2", "value2"); waitForCookie(url); String cookie = mCookieManager.getCookie(url); @@ -98,7 +99,7 @@ public class CookieManagerTest extends AndroidWebViewTestBase { responseHeaders.add( Pair.create("Set-Cookie", "header-test2=header-value2 path=" + path)); url = webServer.setResponse(path, responseStr, responseHeaders); - loadUrlSync(mContentViewCore, mContentViewClient.getOnPageFinishedHelper(), url); + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), url); waitForCookie(url); cookie = mCookieManager.getCookie(url); assertNotNull(cookie); @@ -114,13 +115,13 @@ public class CookieManagerTest extends AndroidWebViewTestBase { private void setCookie(final String name, final String value) throws Throwable { OnEvaluateJavaScriptResultHelper onEvaluateJavaScriptResultHelper = - mContentViewClient.getOnEvaluateJavaScriptResultHelper(); + mContentsClient.getOnEvaluateJavaScriptResultHelper(); int currentCallCount = onEvaluateJavaScriptResultHelper.getCallCount(); final AtomicInteger requestId = new AtomicInteger(); runTestOnUiThread(new Runnable() { @Override public void run() { - requestId.set(mContentViewCore.evaluateJavaScript( + requestId.set(mAwContents.getContentViewCore().evaluateJavaScript( "var expirationDate = new Date();" + "expirationDate.setDate(expirationDate.getDate() + 5);" + "document.cookie='" + name + "=" + value + diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/GetTitleTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/GetTitleTest.java index c157249..366cfcc 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/GetTitleTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/GetTitleTest.java @@ -7,10 +7,10 @@ package org.chromium.android_webview.test; import android.test.suitebuilder.annotation.SmallTest; import android.test.suitebuilder.annotation.Smoke; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.test.util.TestWebServer; import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.DisabledTest; -import org.chromium.content.browser.ContentViewCore; /** * A test suite for ContentView.getTitle(). @@ -23,7 +23,7 @@ public class GetTitleTest extends AndroidWebViewTestBase { private static final String GET_TITLE_TEST_NO_TITLE_PATH = "/get_title_test_no_title.html"; private TestAwContentsClient mContentsClient; - private ContentViewCore mContentViewCore; + private AwContents mAwContents; private static class PageInfo { public final String mTitle; @@ -39,8 +39,9 @@ public class GetTitleTest extends AndroidWebViewTestBase { public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); } private static final String getHtml(String title) { @@ -54,9 +55,9 @@ public class GetTitleTest extends AndroidWebViewTestBase { } private String loadFromDataAndGetTitle(String html) throws Throwable { - loadDataSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadDataSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), html, "text/html", false); - return getTitleOnUiThread(mContentViewCore); + return getTitleOnUiThread(mAwContents); } private PageInfo loadFromUrlAndGetTitle(String html, String filename) throws Throwable { @@ -65,8 +66,8 @@ public class GetTitleTest extends AndroidWebViewTestBase { webServer = new TestWebServer(false); final String url = webServer.setResponse(filename, html, null); - loadUrlSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), url); - return new PageInfo(getTitleOnUiThread(mContentViewCore), + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), url); + return new PageInfo(getTitleOnUiThread(mAwContents), url.replaceAll("http:\\/\\/", "")); } finally { diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/LoadDataWithBaseUrlTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/LoadDataWithBaseUrlTest.java index 609b62e..7f6e72d 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/LoadDataWithBaseUrlTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/LoadDataWithBaseUrlTest.java @@ -6,6 +6,7 @@ package org.chromium.android_webview.test; import android.test.suitebuilder.annotation.SmallTest; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.test.util.CommonResources; import org.chromium.android_webview.test.util.TestWebServer; import org.chromium.base.test.util.Feature; @@ -23,14 +24,17 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { protected static int WAIT_TIMEOUT_SECONDS = 15; private TestAwContentsClient mContentsClient; + private AwContents mAwContents; private ContentViewCore mContentViewCore; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); + mContentViewCore = testContainerView.getContentViewCore(); } protected void loadDataWithBaseUrlSync( @@ -50,7 +54,7 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { runTestOnUiThread(new Runnable() { @Override public void run() { - mContentViewCore.loadUrl(LoadUrlParams.createLoadDataParamsWithBaseUrl( + mAwContents.loadUrl(LoadUrlParams.createLoadDataParamsWithBaseUrl( data, mimeType, isBase64Encoded, baseUrl, historyUrl)); } }); @@ -92,7 +96,7 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { webServer.setResponseBase64("/" + CommonResources.FAVICON_FILENAME, CommonResources.FAVICON_DATA_BASE64, CommonResources.getImagePngHeaders(true)); - ContentSettings contentSettings = getContentSettingsOnUiThread(mContentViewCore); + ContentSettings contentSettings = getContentSettingsOnUiThread(mAwContents); contentSettings.setImagesEnabled(true); contentSettings.setJavaScriptEnabled(true); @@ -100,7 +104,7 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { CommonResources.getOnImageLoadedHtml(CommonResources.FAVICON_FILENAME), "text/html", false, webServer.getBaseUrl(), null); - assertEquals("5", getTitleOnUiThread(mContentViewCore)); + assertEquals("5", getTitleOnUiThread(mAwContents)); } finally { if (webServer != null) webServer.shutdown(); } @@ -117,9 +121,9 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { CommonResources.getTextJavascriptHeaders(true)); final String pageHtml = getScriptFileTestPageHtml(scriptUrl); - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); loadDataWithBaseUrlSync(pageHtml, "text/html", false, webServer.getBaseUrl(), null); - assertEquals(SCRIPT_LOADED, getTitleOnUiThread(mContentViewCore)); + assertEquals(SCRIPT_LOADED, getTitleOnUiThread(mAwContents)); } finally { if (webServer != null) webServer.shutdown(); @@ -136,9 +140,9 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { CommonResources.ABOUT_HTML, CommonResources.getTextHtmlHeaders(true)); final String html = getCrossOriginAccessTestPageHtml(frameUrl); - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); loadDataWithBaseUrlSync(html, "text/html", false, webServer.getBaseUrl(), null); - assertEquals(frameUrl, getTitleOnUiThread(mContentViewCore)); + assertEquals(frameUrl, getTitleOnUiThread(mAwContents)); } finally { if (webServer != null) webServer.shutdown(); @@ -156,12 +160,12 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { final String html = getCrossOriginAccessTestPageHtml(frameUrl); final String baseUrl = webServer.getBaseUrl().replaceFirst("localhost", "127.0.0.1"); - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); loadDataWithBaseUrlSync(html, "text/html", false, baseUrl, null); // TODO(mnaganov): Catch a security exception and set the title accordingly, // once https://bugs.webkit.org/show_bug.cgi?id=43504 is fixed. - assertEquals("undefined", getTitleOnUiThread(mContentViewCore)); + assertEquals("undefined", getTitleOnUiThread(mAwContents)); } finally { if (webServer != null) webServer.shutdown(); @@ -171,11 +175,11 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { @SmallTest @Feature({"Android-WebView"}) public void testNullBaseUrl() throws Throwable { - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); final String pageHtml = "<html><body onload='document.title=document.location.href'>" + "</body></html>"; loadDataWithBaseUrlSync(pageHtml, "text/html", false, null, null); - assertEquals("about:blank", getTitleOnUiThread(mContentViewCore)); + assertEquals("about:blank", getTitleOnUiThread(mAwContents)); } @SmallTest @@ -209,23 +213,23 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { @SmallTest @Feature({"Android-WebView"}) public void testAccessToLocalFile() throws Throwable { - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); final String baseUrl = UrlUtils.getTestFileUrl("webview/"); final String scriptFile = baseUrl + "script.js"; final String pageHtml = getScriptFileTestPageHtml(scriptFile); loadDataWithBaseUrlSync(pageHtml, "text/html", false, baseUrl, null); - assertEquals(SCRIPT_LOADED, getTitleOnUiThread(mContentViewCore)); + assertEquals(SCRIPT_LOADED, getTitleOnUiThread(mAwContents)); } @SmallTest @Feature({"Android-WebView"}) public void testFailedAccessToLocalFile() throws Throwable { - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); final String scriptFile = UrlUtils.getTestFileUrl("webview/script.js"); final String pageHtml = getScriptFileTestPageHtml(scriptFile); final String baseUrl = "http://example.com"; loadDataWithBaseUrlSync(pageHtml, "text/html", false, baseUrl, null); - assertEquals(SCRIPT_NOT_LOADED, getTitleOnUiThread(mContentViewCore)); + assertEquals(SCRIPT_NOT_LOADED, getTitleOnUiThread(mAwContents)); } @SmallTest @@ -242,7 +246,7 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { "<body>" + page1Title + "</body></html>"; loadDataWithBaseUrlSync(page1Html, "text/html", false, null, historyUrl); - assertEquals(page1Title, getTitleOnUiThread(mContentViewCore)); + assertEquals(page1Title, getTitleOnUiThread(mAwContents)); final String page2Title = "Page2"; final String page2Html = "<html><head><title>" + page2Title + "</title>" + @@ -250,12 +254,12 @@ public class LoadDataWithBaseUrlTest extends AndroidWebViewTestBase { final TestCallbackHelperContainer.OnPageFinishedHelper onPageFinishedHelper = mContentsClient.getOnPageFinishedHelper(); - loadDataSync(mContentViewCore, onPageFinishedHelper, page2Html, "text/html", false); - assertEquals(page2Title, getTitleOnUiThread(mContentViewCore)); + loadDataSync(mAwContents, onPageFinishedHelper, page2Html, "text/html", false); + assertEquals(page2Title, getTitleOnUiThread(mAwContents)); HistoryUtils.goBackSync(getInstrumentation(), mContentViewCore, onPageFinishedHelper); // The title of the 'about.html' specified via historyUrl. - assertEquals(CommonResources.ABOUT_TITLE, getTitleOnUiThread(mContentViewCore)); + assertEquals(CommonResources.ABOUT_TITLE, getTitleOnUiThread(mAwContents)); } finally { if (webServer != null) webServer.shutdown(); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/NavigationHistoryTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/NavigationHistoryTest.java index 35d6478..30ecd95 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/NavigationHistoryTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/NavigationHistoryTest.java @@ -7,11 +7,11 @@ package org.chromium.android_webview.test; import android.test.FlakyTest; import android.test.suitebuilder.annotation.SmallTest; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.test.util.CommonResources; import org.chromium.android_webview.test.util.TestWebServer; import org.chromium.base.ThreadUtils; import org.chromium.base.test.util.DisabledTest; -import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.NavigationEntry; import org.chromium.content.browser.NavigationHistory; import org.chromium.content.browser.test.util.HistoryUtils; @@ -22,22 +22,23 @@ import java.util.concurrent.Callable; public class NavigationHistoryTest extends AndroidWebViewTestBase { private TestAwContentsClient mContentsClient; - private ContentViewCore mContentViewCore; + private AwContents mAwContents; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + final AwTestContainerView testContainerView = + createAwTestContainerViewOnMainSync(mContentsClient); + mAwContents = testContainerView.getAwContents(); } - private NavigationHistory getNavigationHistory(final ContentViewCore contentViewCore) + private NavigationHistory getNavigationHistory(final AwContents awContents) throws Exception { return ThreadUtils.runOnUiThreadBlocking(new Callable<NavigationHistory>() { @Override public NavigationHistory call() { - return contentViewCore.getNavigationHistory(); + return awContents.getContentViewCore().getNavigationHistory(); } }); } @@ -60,12 +61,12 @@ public class NavigationHistoryTest extends AndroidWebViewTestBase { */ @FlakyTest public void testNavigateOneUrl() throws Throwable { - NavigationHistory history = getNavigationHistory(mContentViewCore); + NavigationHistory history = getNavigationHistory(mAwContents); assertEquals(0, history.getEntryCount()); - loadUrlSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), "chrome://newtab/"); - history = getNavigationHistory(mContentViewCore); + history = getNavigationHistory(mAwContents); checkHistoryItem(history.getEntryAtIndex(0), "chrome://newtab/#bookmarks", "chrome://newtab/", @@ -77,15 +78,15 @@ public class NavigationHistoryTest extends AndroidWebViewTestBase { @SmallTest public void testNavigateTwoUrls() throws Throwable { - NavigationHistory list = getNavigationHistory(mContentViewCore); + NavigationHistory list = getNavigationHistory(mAwContents); assertEquals(0, list.getEntryCount()); final TestCallbackHelperContainer.OnPageFinishedHelper onPageFinishedHelper = mContentsClient.getOnPageFinishedHelper(); - loadUrlSync(mContentViewCore, onPageFinishedHelper, "chrome://newtab/"); - loadUrlSync(mContentViewCore, onPageFinishedHelper, "chrome://version"); + loadUrlSync(mAwContents, onPageFinishedHelper, "chrome://newtab/"); + loadUrlSync(mAwContents, onPageFinishedHelper, "chrome://version"); - list = getNavigationHistory(mContentViewCore); + list = getNavigationHistory(mAwContents); // Make sure there is a new entry entry the list assertEquals(2, list.getEntryCount()); @@ -112,13 +113,14 @@ public class NavigationHistoryTest extends AndroidWebViewTestBase { public void testNavigateTwoUrlsAndBack() throws Throwable { final TestCallbackHelperContainer.OnPageFinishedHelper onPageFinishedHelper = mContentsClient.getOnPageFinishedHelper(); - NavigationHistory list = getNavigationHistory(mContentViewCore); + NavigationHistory list = getNavigationHistory(mAwContents); assertEquals(0, list.getEntryCount()); - loadUrlSync(mContentViewCore, onPageFinishedHelper, "chrome://newtab/"); - loadUrlSync(mContentViewCore, onPageFinishedHelper, "chrome://version"); - HistoryUtils.goBackSync(getInstrumentation(), mContentViewCore, onPageFinishedHelper); - list = getNavigationHistory(mContentViewCore); + loadUrlSync(mAwContents, onPageFinishedHelper, "chrome://newtab/"); + loadUrlSync(mAwContents, onPageFinishedHelper, "chrome://version"); + HistoryUtils.goBackSync(getInstrumentation(), mAwContents.getContentViewCore(), + onPageFinishedHelper); + list = getNavigationHistory(mAwContents); // Make sure the first entry is still okay checkHistoryItem(list.getEntryAtIndex(0), @@ -146,7 +148,7 @@ public class NavigationHistoryTest extends AndroidWebViewTestBase { */ @DisabledTest public void testFavicon() throws Throwable { - NavigationHistory list = getNavigationHistory(mContentViewCore); + NavigationHistory list = getNavigationHistory(mAwContents); String url; TestWebServer webServer = null; @@ -158,14 +160,14 @@ public class NavigationHistoryTest extends AndroidWebViewTestBase { url = webServer.setResponse("/favicon.html", CommonResources.FAVICON_STATIC_HTML, null); assertEquals(0, list.getEntryCount()); - getContentSettingsOnUiThread(mContentViewCore).setImagesEnabled(true); - loadUrlSync(mContentViewCore, mContentsClient.getOnPageFinishedHelper(), url); + getContentSettingsOnUiThread(mAwContents).setImagesEnabled(true); + loadUrlSync(mAwContents, mContentsClient.getOnPageFinishedHelper(), url); } finally { if (webServer != null) webServer.shutdown(); } - list = getNavigationHistory(mContentViewCore); + list = getNavigationHistory(mAwContents); // Make sure the first entry is still okay. checkHistoryItem(list.getEntryAtIndex(0), url, url, "", false); diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/UserAgentTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/UserAgentTest.java index e2e289d..786b9ed 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/UserAgentTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/UserAgentTest.java @@ -9,20 +9,19 @@ import android.test.suitebuilder.annotation.SmallTest; import java.util.regex.Matcher; import java.util.regex.Pattern; +import org.chromium.android_webview.AwContents; import org.chromium.base.test.util.Feature; -import org.chromium.content.browser.ContentViewCore; public class UserAgentTest extends AndroidWebViewTestBase { private TestAwContentsClient mContentsClient; - private ContentViewCore mContentViewCore; + private AwContents mAwContents; @Override public void setUp() throws Exception { super.setUp(); mContentsClient = new TestAwContentsClient(); - mContentViewCore = - createAwTestContainerViewOnMainSync(mContentsClient).getContentViewCore(); + mAwContents = createAwTestContainerViewOnMainSync(mContentsClient).getAwContents(); } /** @@ -32,16 +31,16 @@ public class UserAgentTest extends AndroidWebViewTestBase { @SmallTest @Feature({"Android-WebView"}) public void testNoExtraSpaceBeforeBuildName() throws Throwable { - getContentSettingsOnUiThread(mContentViewCore).setJavaScriptEnabled(true); + getContentSettingsOnUiThread(mAwContents).setJavaScriptEnabled(true); loadDataSync( - mContentViewCore, + mAwContents, mContentsClient.getOnPageFinishedHelper(), // Spaces are replaced with underscores to avoid consecutive spaces compression. "<html>" + "<body onload='document.title=navigator.userAgent.replace(/ /g, \"_\")'></body>" + "</html>", "text/html", false); - final String ua = getTitleOnUiThread(mContentViewCore); + final String ua = getTitleOnUiThread(mAwContents); Matcher matcher = Pattern.compile("Android_[^;]+;_[^_]").matcher(ua); assertTrue(matcher.find()); } diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/WebViewFindApisTestBase.java b/android_webview/javatests/src/org/chromium/android_webview/test/WebViewFindApisTestBase.java index bdefbfb..dc235179 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/WebViewFindApisTestBase.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/WebViewFindApisTestBase.java @@ -62,7 +62,7 @@ public class WebViewFindApisTestBase extends AndroidWebViewTestBase { final AwContents contents = createAwTestContainerViewOnMainSync(contentsClient).getAwContents(); final String data = "<html><head></head><body>" + html + "</body></html>"; - loadDataSync(contents.getContentViewCore(), contentsClient.getOnPageFinishedHelper(), + loadDataSync(contents, contentsClient.getOnPageFinishedHelper(), data, "text/html", false); return contents; } diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/WebViewModalDialogOverrideTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/WebViewModalDialogOverrideTest.java index 4597da4..e679214 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/WebViewModalDialogOverrideTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/WebViewModalDialogOverrideTest.java @@ -11,10 +11,10 @@ package org.chromium.android_webview.test; import android.test.suitebuilder.annotation.MediumTest; import android.test.suitebuilder.annotation.SmallTest; +import org.chromium.android_webview.AwContents; import org.chromium.android_webview.JsPromptResultReceiver; import org.chromium.android_webview.JsResultReceiver; import org.chromium.base.test.util.Feature; -import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.test.util.CallbackHelper; import java.util.concurrent.atomic.AtomicBoolean; @@ -48,12 +48,12 @@ public class WebViewModalDialogOverrideTest extends AndroidWebViewTestBase { } }; AwTestContainerView view = createAwTestContainerViewOnMainSync(client); - final ContentViewCore contentViewCore = view.getContentViewCore(); + final AwContents awContents = view.getAwContents(); - enableJavaScriptOnUiThread(contentViewCore); - loadDataSync(contentViewCore, client.getOnPageFinishedHelper(), + enableJavaScriptOnUiThread(awContents); + loadDataSync(awContents, client.getOnPageFinishedHelper(), EMPTY_PAGE, "text/html", false); - executeJavaScriptAndWaitForResult(contentViewCore, client, + executeJavaScriptAndWaitForResult(awContents, client, "alert('" + ALERT_TEXT + "')"); assertTrue(callbackCalled.get()); } @@ -81,12 +81,12 @@ public class WebViewModalDialogOverrideTest extends AndroidWebViewTestBase { } }; AwTestContainerView view = createAwTestContainerViewOnMainSync(client); - ContentViewCore contentViewCore = view.getContentViewCore(); + final AwContents awContents = view.getAwContents(); - enableJavaScriptOnUiThread(contentViewCore); - loadDataSync(contentViewCore, client.getOnPageFinishedHelper(), + enableJavaScriptOnUiThread(awContents); + loadDataSync(awContents, client.getOnPageFinishedHelper(), EMPTY_PAGE, "text/html", false); - String result = executeJavaScriptAndWaitForResult(contentViewCore, client, + String result = executeJavaScriptAndWaitForResult(awContents, client, "prompt('" + PROMPT_TEXT + "','" + PROMPT_DEFAULT + "')"); assertTrue(called.get()); assertEquals("\"" + PROMPT_RESULT + "\"", result); @@ -111,12 +111,12 @@ public class WebViewModalDialogOverrideTest extends AndroidWebViewTestBase { } }; AwTestContainerView view = createAwTestContainerViewOnMainSync(client); - ContentViewCore contentViewCore = view.getContentViewCore(); - enableJavaScriptOnUiThread(contentViewCore); + final AwContents awContents = view.getAwContents(); + enableJavaScriptOnUiThread(awContents); - loadDataSync(contentViewCore, client.getOnPageFinishedHelper(), + loadDataSync(awContents, client.getOnPageFinishedHelper(), EMPTY_PAGE, "text/html", false); - String result = executeJavaScriptAndWaitForResult(contentViewCore, client, + String result = executeJavaScriptAndWaitForResult(awContents, client, "confirm('" + CONFIRM_TEXT + "')"); assertTrue(called.get()); assertEquals("true", result); @@ -141,12 +141,12 @@ public class WebViewModalDialogOverrideTest extends AndroidWebViewTestBase { } }; AwTestContainerView view = createAwTestContainerViewOnMainSync(client); - ContentViewCore contentViewCore = view.getContentViewCore(); - enableJavaScriptOnUiThread(contentViewCore); + final AwContents awContents = view.getAwContents(); + enableJavaScriptOnUiThread(awContents); - loadDataSync(contentViewCore, client.getOnPageFinishedHelper(), + loadDataSync(awContents, client.getOnPageFinishedHelper(), EMPTY_PAGE, "text/html", false); - String result = executeJavaScriptAndWaitForResult(contentViewCore, client, + String result = executeJavaScriptAndWaitForResult(awContents, client, "confirm('" + CONFIRM_TEXT + "')"); assertTrue(called.get()); assertEquals("false", result); @@ -167,16 +167,16 @@ public class WebViewModalDialogOverrideTest extends AndroidWebViewTestBase { } }; AwTestContainerView view = createAwTestContainerViewOnMainSync(client); - ContentViewCore contentViewCore = view.getContentViewCore(); - enableJavaScriptOnUiThread(contentViewCore); + final AwContents awContents = view.getAwContents(); + enableJavaScriptOnUiThread(awContents); - loadDataSync(contentViewCore, client.getOnPageFinishedHelper(), BEFORE_UNLOAD_URL, + loadDataSync(awContents, client.getOnPageFinishedHelper(), BEFORE_UNLOAD_URL, "text/html", false); - enableJavaScriptOnUiThread(contentViewCore); + enableJavaScriptOnUiThread(awContents); // Don't wait synchronously because we don't leave the page. int currentCallCount = jsBeforeUnloadHelper.getCallCount(); - loadDataAsync(contentViewCore, EMPTY_PAGE, "text/html", false); + loadDataAsync(awContents, EMPTY_PAGE, "text/html", false); jsBeforeUnloadHelper.waitForCallback(currentCallCount); } } |