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

Unified Diff: android_webview/javatests/src/org/chromium/android_webview/test/LoadDataWithBaseUrlTest.java

Issue 68763012: [android_webview] Don't call onPageFinished for history API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: android_webview/javatests/src/org/chromium/android_webview/test/LoadDataWithBaseUrlTest.java
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 365645dc5481dc31d6f909c660080957e8ce6994..b2f510b36c5e6b3b3b4a9d55ef1afca5818a5eaf 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
@@ -214,6 +214,18 @@ public class LoadDataWithBaseUrlTest extends AwTestBase {
@SmallTest
@Feature({"AndroidWebView"})
+ public void testOnPageFinishedUrlIsBaseUrl() throws Throwable {
+ final String pageHtml = "<html><body>Hello, world!</body></html>";
+ final String baseUrl = "http://example.com/";
+ loadDataWithBaseUrlSync(pageHtml, "text/html", false, baseUrl, baseUrl);
+ loadDataWithBaseUrlSync(pageHtml, "text/html", false, baseUrl, baseUrl);
+ TestCallbackHelperContainer.OnPageFinishedHelper onPageFinishedHelper =
+ mContentsClient.getOnPageFinishedHelper();
+ assertEquals(baseUrl, onPageFinishedHelper.getUrl());
+ }
+
+ @SmallTest
+ @Feature({"AndroidWebView"})
public void testHistoryUrlIgnoredWithDataSchemeBaseUrl() throws Throwable {
final String pageHtml = "<html><body>bar</body></html>";
final String historyUrl = "http://history.com/";

Powered by Google App Engine
This is Rietveld 408576698