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

Unified Diff: chrome/browser/apps/guest_view/web_view_browsertest.cc

Issue 2781783002: Allow webview guests to skip WAR checks in ShouldAllowOpenURL (Merge to M58) (Closed)
Patch Set: Allow webview guests to skip WAR checks in ShouldAllowOpenURL. Created 3 years, 9 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
« no previous file with comments | « no previous file | chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/apps/guest_view/web_view_browsertest.cc
diff --git a/chrome/browser/apps/guest_view/web_view_browsertest.cc b/chrome/browser/apps/guest_view/web_view_browsertest.cc
index b57c6b770f99f9b6f6df361d70c35cebc060551d..39e770c59d8c9c4821feb39d52f98d8e4e2e08dc 100644
--- a/chrome/browser/apps/guest_view/web_view_browsertest.cc
+++ b/chrome/browser/apps/guest_view/web_view_browsertest.cc
@@ -3255,6 +3255,24 @@ IN_PROC_BROWSER_TEST_P(WebViewTest, LoadWebviewAccessibleResource) {
"web_view/load_webview_accessible_resource", NEEDS_TEST_SERVER);
}
+// Tests that a WebView can reload a WebView accessible resource. See
+// https://crbug.com/691941.
+IN_PROC_BROWSER_TEST_P(WebViewTest, ReloadWebviewAccessibleResource) {
+ TestHelper("testReloadWebviewAccessibleResource",
+ "web_view/load_webview_accessible_resource", NEEDS_TEST_SERVER);
+
+ content::WebContents* embedder_contents = GetEmbedderWebContents();
+ content::WebContents* web_view_contents =
+ GetGuestViewManager()->GetLastGuestCreated();
+ ASSERT_TRUE(embedder_contents);
+ ASSERT_TRUE(web_view_contents);
+
+ GURL embedder_url(embedder_contents->GetLastCommittedURL());
+ GURL webview_url(embedder_url.GetOrigin().spec() + "assets/foo.html");
+
+ EXPECT_EQ(webview_url, web_view_contents->GetLastCommittedURL());
+}
+
IN_PROC_BROWSER_TEST_P(WebViewAccessibilityTest, LoadWebViewAccessibility) {
LoadAppWithGuest("web_view/focus_accessibility");
content::WebContents* web_contents = GetFirstAppWindowWebContents();
« no previous file with comments | « no previous file | chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698