Move ClosePage() from RenderViewHost to WebContents.

content/browser/devtools/render_frame_devtools_agent_host.cc

Index: content/browser/devtools/render_frame_devtools_agent_host.cc
diff --git a/content/browser/devtools/render_frame_devtools_agent_host.cc b/content/browser/devtools/render_frame_devtools_agent_host.cc
index 09ecb598b4250e533c1b2f01a295712568b87c2a..01dd5b0ad2ce7da5e2ff9901817891da06a19680 100644
--- a/content/browser/devtools/render_frame_devtools_agent_host.cc
+++ b/content/browser/devtools/render_frame_devtools_agent_host.cc
@@ -474,7 +474,7 @@ bool RenderFrameDevToolsAgentHost::Activate() {
 
 bool RenderFrameDevToolsAgentHost::Close() {
   if (render_frame_host_) {
-    render_frame_host_->GetRenderViewHost()->ClosePage();
+    WebContentsImpl::FromRenderFrameHost(render_frame_host_)->ClosePage();

[dgozman, 2015/04/21 05:13:32]

if (web_contents())
  web_contents()->ClosePage();

[nasko, 2015/04/29 17:09:27]

That looks fine to me. What I found strange is few places in this file use
WebContents* web_contents = GetWebContents(); which just returns web_contents().
Should this be cleaned up?

[dgozman, 2015/04/29 17:23:42]

web_contents() is a protected method of WCO, and we need a public one, which is
called GetWebContents(). I personally don't have preference for one over
another, but I'm open for improvements if you have any suggestions.

     return true;
   }
   return false;