Get rid of ShowPageInfo on WebContents and WebContentsDelegate. Chrome shouldn't have to go throu...

chrome/browser/tab_contents/render_view_context_menu.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 #include <set>
 #include <utility>
 
 #include "chrome/browser/tab_contents/render_view_context_menu.h"
 
@@ skipping 72 matching lines @@
 #include "ui/base/text/text_elider.h"
 #include "ui/gfx/favicon_size.h"
 #include "webkit/glue/webmenuitem.h"
 
 #ifdef FILE_MANAGER_EXTENSION
 #include "chrome/browser/extensions/file_manager_util.h"
 #endif
 
 using content::ChildProcessSecurityPolicy;
 using content::DownloadManager;
+using content::NavigationController;
 using content::NavigationEntry;
 using content::OpenURLParams;
 using content::SSLStatus;
 using content::UserMetricsAction;
 using content::WebContents;
 using WebKit::WebContextMenuData;
 using WebKit::WebMediaPlayerAction;
 using WebKit::WebPluginAction;
 using WebKit::WebURL;
 using WebKit::WebString;
@@ skipping 1568 matching lines @@
 
     case IDC_VIEW_SOURCE:
       source_web_contents_->ViewSource();
       break;
 
     case IDC_CONTENT_CONTEXT_INSPECTELEMENT:
       Inspect(params_.x, params_.y);
       break;
 
     case IDC_CONTENT_CONTEXT_VIEWPAGEINFO: {
-      NavigationEntry* nav_entry =
-          source_web_contents_->GetController().GetActiveEntry();
-      source_web_contents_->ShowPageInfo(nav_entry->GetURL(),
-                                         nav_entry->GetSSL(), true);
+      NavigationController* controller = &source_web_contents_->GetController();
+      NavigationEntry* nav_entry = controller->GetActiveEntry();
+      Browser* browser = Browser::GetBrowserForController(controller, NULL);      
+      browser->ShowPageInfo(nav_entry->GetURL(), nav_entry->GetSSL(), true);
       break;
     }
 
     case IDC_CONTENT_CONTEXT_TRANSLATE: {
       // A translation might have been triggered by the time the menu got
       // selected, do nothing in that case.
       TabContentsWrapper* tab_contents_wrapper =
           TabContentsWrapper::GetCurrentWrapperForContents(
               source_web_contents_);
       if (!tab_contents_wrapper)
@@ skipping 19 matching lines @@
     case IDC_CONTENT_CONTEXT_RELOADFRAME:
       rvh->ReloadFrame();
       break;
 
     case IDC_CONTENT_CONTEXT_VIEWFRAMESOURCE:
       source_web_contents_->ViewFrameSource(params_.frame_url,
                                             params_.frame_content_state);
       break;
 
     case IDC_CONTENT_CONTEXT_VIEWFRAMEINFO: {
-      source_web_contents_->ShowPageInfo(params_.frame_url,
-                                         params_.security_info,
-                                         false);  // Don't show the history.
+      Browser* browser = Browser::GetBrowserForController(
+          &source_web_contents_->GetController(), NULL);
+      browser->ShowPageInfo(params_.frame_url, params_.security_info, false);
       break;
     }
 
     case IDC_CONTENT_CONTEXT_UNDO:
       rvh->Undo();
       break;
 
     case IDC_CONTENT_CONTEXT_REDO:
       rvh->Redo();
       break;
@@ skipping 248 matching lines @@
   source_web_contents_->GetRenderViewHost()->
       ExecuteMediaPlayerActionAtLocation(location, action);
 }
 
 void RenderViewContextMenu::PluginActionAt(
     const gfx::Point& location,
     const WebPluginAction& action) {
   source_web_contents_->GetRenderViewHost()->
       ExecutePluginActionAtLocation(location, action);
 }