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

Unified Diff: content/browser/web_contents/web_contents_impl_unittest.cc

Issue 2527133002: Remove some WebContents::GetRenderViewHost() calls. (Closed)
Patch Set: Created 4 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: content/browser/web_contents/web_contents_impl_unittest.cc
diff --git a/content/browser/web_contents/web_contents_impl_unittest.cc b/content/browser/web_contents/web_contents_impl_unittest.cc
index d2aee22f5a546bdf7fff77efdfad2de233a402df..884d50ead73f63558f884c340101a27543456cbc 100644
--- a/content/browser/web_contents/web_contents_impl_unittest.cc
+++ b/content/browser/web_contents/web_contents_impl_unittest.cc
@@ -2621,7 +2621,7 @@ TEST_F(WebContentsImplTest, PendingContents) {
std::unique_ptr<TestWebContents> other_contents(
static_cast<TestWebContents*>(CreateTestWebContents()));
contents()->AddPendingContents(other_contents.get());
- int process_id = other_contents->GetRenderViewHost()->GetProcess()->GetID();
+ int process_id = other_contents->GetMainFrame()->GetProcess()->GetID();
int route_id = other_contents->GetRenderViewHost()->GetRoutingID();
other_contents.reset();
EXPECT_EQ(nullptr, contents()->GetCreatedWindow(process_id, route_id));

Powered by Google App Engine
This is Rietveld 408576698