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

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

Issue 1126383005: [Android WebView] Send WebChromeClient.onReceivedTitle when navigating back (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: android_webview/javatests/src/org/chromium/android_webview/test/NavigationHistoryTest.java
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 5a8c75d0d588578f8a34dac518b8dd668931dac2..14f18b915a95e62fe65d86dde4e6e7916b5c381b 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
@@ -320,4 +320,23 @@ public class NavigationHistoryTest extends AwTestBase {
}
});
}
+
+ // See http://crbug.com/481570
+ @SmallTest
+ public void testTitleUpdatedWhenGoingBack() throws Throwable {
+ final TestCallbackHelperContainer.OnPageFinishedHelper onPageFinishedHelper =
+ mContentsClient.getOnPageFinishedHelper();
+ NavigationHistory list = getNavigationHistory(mAwContents);
+ assertEquals(0, list.getEntryCount());
+
+ final String page1Url = addPage1ToServer(mWebServer);
+ final String page2Url = addPage2ToServer(mWebServer);
+
+ loadUrlSync(mAwContents, onPageFinishedHelper, page1Url);
+ loadUrlSync(mAwContents, onPageFinishedHelper, page2Url);
+ assertEquals(PAGE_2_TITLE, mContentsClient.getUpdatedTitle());
+ HistoryUtils.goBackSync(getInstrumentation(), mAwContents.getWebContents(),
+ onPageFinishedHelper);
+ assertEquals(PAGE_1_TITLE, mContentsClient.getUpdatedTitle());
+ }
}

Powered by Google App Engine
This is Rietveld 408576698