Remove some WebContents::GetRenderViewHost() calls.

content/test/test_web_contents.cc

Index: content/test/test_web_contents.cc
diff --git a/content/test/test_web_contents.cc b/content/test/test_web_contents.cc
index 2fc99008b35edb35733e07c0c0b8bfd41f91190f..c0546a56c99f1da184f69cf2b7cf6f110108caf9 100644
--- a/content/test/test_web_contents.cc
+++ b/content/test/test_web_contents.cc
@@ -265,7 +265,7 @@ void TestWebContents::SetOpener(TestWebContents* opener) {
 
 void TestWebContents::AddPendingContents(TestWebContents* contents) {
   // This is normally only done in WebContentsImpl::CreateNewWindow.
-  ProcessRoutingIdPair key(contents->GetRenderViewHost()->GetProcess()->GetID(),
+  ProcessRoutingIdPair key(contents->GetMainFrame()->GetProcess()->GetID(),

[Charlie Reis, 2016/11/28 21:00:41]

We shouldn't change this one.  We should always get the process ID and routing
ID from the same object.  (It *should* be the case that the RVH and RFH will
give the same process ID here, but it's quite confusing to get the process ID
from one and the route ID from the other.)

Nick has a CL converting CreateNewWindow logic away from RVH, so we can leave
this one for that CL.

[Lei Zhang, 2016/11/30 10:27:00]

Reverted.

                            contents->GetRenderViewHost()->GetRoutingID());
   pending_contents_[key] = contents;
   AddDestructionObserver(contents);