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

Unified Diff: content/browser/frame_host/navigation_controller_impl_browsertest.cc

Issue 2541063002: Revert of Remove about:srcdoc url conversion. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/frame_host/navigation_controller_impl_browsertest.cc
diff --git a/content/browser/frame_host/navigation_controller_impl_browsertest.cc b/content/browser/frame_host/navigation_controller_impl_browsertest.cc
index 677ef7d0c737b45128a16c2d901ede2ee10d491b..64fe1da373172c0f6badd4a1dc316cc26c3483ba 100644
--- a/content/browser/frame_host/navigation_controller_impl_browsertest.cc
+++ b/content/browser/frame_host/navigation_controller_impl_browsertest.cc
@@ -3300,7 +3300,7 @@
// nested iframe.
GURL main_url(embedded_test_server()->GetURL(
"/navigation_controller/inject_iframe_srcdoc_with_nested_frame.html"));
- GURL srcdoc_url(content::kAboutSrcDocURL);
+ GURL blank_url(url::kAboutBlankURL);
GURL inner_url(
embedded_test_server()->GetURL("/navigation_controller/form.html"));
EXPECT_TRUE(NavigateToURL(shell(), main_url));
@@ -3315,7 +3315,7 @@
ASSERT_EQ(1U, root->child_at(0)->child_count());
ASSERT_EQ(0U, root->child_at(0)->child_at(0)->child_count());
EXPECT_EQ(main_url, root->current_url());
- EXPECT_EQ(srcdoc_url, root->child_at(0)->current_url());
+ EXPECT_EQ(blank_url, root->child_at(0)->current_url());
EXPECT_EQ(inner_url, root->child_at(0)->child_at(0)->current_url());
EXPECT_EQ(1, controller.GetEntryCount());
@@ -3325,7 +3325,7 @@
// The entry should have FrameNavigationEntries for the subframes.
if (SiteIsolationPolicy::UseSubframeNavigationEntries()) {
ASSERT_EQ(1U, entry->root_node()->children.size());
- EXPECT_EQ(srcdoc_url, entry->root_node()->children[0]->frame_entry->url());
+ EXPECT_EQ(blank_url, entry->root_node()->children[0]->frame_entry->url());
EXPECT_EQ(inner_url,
entry->root_node()->children[0]->children[0]->frame_entry->url());
}
@@ -3352,10 +3352,12 @@
back_load_observer.Wait();
}
ASSERT_EQ(1U, root->child_count());
+ // TODO(creis): This line is unexpectedly failing in PlzNavigate, so the test
+ // is disabled there for now.
ASSERT_EQ(1U, root->child_at(0)->child_count());
ASSERT_EQ(0U, root->child_at(0)->child_at(0)->child_count());
EXPECT_EQ(main_url, root->current_url());
- EXPECT_EQ(srcdoc_url, root->child_at(0)->current_url());
+ EXPECT_EQ(blank_url, root->child_at(0)->current_url());
// Verify that the inner iframe went to the correct URL.
EXPECT_EQ(inner_url, root->child_at(0)->child_at(0)->current_url());
@@ -3367,7 +3369,7 @@
// The entry should have FrameNavigationEntries for the subframes.
if (SiteIsolationPolicy::UseSubframeNavigationEntries()) {
ASSERT_EQ(1U, entry->root_node()->children.size());
- EXPECT_EQ(srcdoc_url, entry->root_node()->children[0]->frame_entry->url());
+ EXPECT_EQ(blank_url, entry->root_node()->children[0]->frame_entry->url());
EXPECT_EQ(inner_url,
entry->root_node()->children[0]->children[0]->frame_entry->url());
}
« no previous file with comments | « content/browser/frame_host/interstitial_page_navigator_impl.cc ('k') | content/browser/frame_host/navigation_entry_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698