Make PageInfoBubble not use BrowserList::GetLastActive and instead obtain a navigator at creation t…

chrome/browser/ui/gtk/page_info_bubble_gtk.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 <gtk/gtk.h>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/i18n/rtl.h"
 #include "base/utf_string_conversions.h"
@@ skipping 22 matching lines @@
 
 namespace {
 
 class PageInfoBubbleGtk : public PageInfoModelObserver,
                           public BubbleDelegateGtk {
  public:
   PageInfoBubbleGtk(gfx::NativeWindow parent,
                     Profile* profile,
                     const GURL& url,
                     const SSLStatus& ssl,
-                    bool show_history);
+                    bool show_history,
+                    content::PageNavigator* navigator);
   virtual ~PageInfoBubbleGtk();
 
   // PageInfoModelObserver implementation.
   virtual void OnPageInfoModelChanged() OVERRIDE;
 
   // BubbleDelegateGtk implementation.
   virtual void BubbleClosing(BubbleGtk* bubble, bool closed_by_escape) OVERRIDE;
 
  private:
   // Layouts the different sections retrieved from the model.
@@ skipping 24 matching lines @@
   // The widget relative to which we are positioned.
   GtkWidget* anchor_;
 
   // Provides colors and stuff.
   GtkThemeService* theme_service_;
 
   BubbleGtk* bubble_;
 
   Profile* profile_;
 
+  // Used for loading pages.
+  content::PageNavigator* navigator_;
+
   DISALLOW_COPY_AND_ASSIGN(PageInfoBubbleGtk);
 };
 
 PageInfoBubbleGtk::PageInfoBubbleGtk(gfx::NativeWindow parent,
                                      Profile* profile,
                                      const GURL& url,
                                      const SSLStatus& ssl,
-                                     bool show_history)
+                                     bool show_history,
+                                     content::PageNavigator* navigator)
     : ALLOW_THIS_IN_INITIALIZER_LIST(model_(profile, url, ssl,
                                             show_history, this)),
       url_(url),
       cert_id_(ssl.cert_id),
       parent_(parent),
       contents_(NULL),
       theme_service_(GtkThemeService::GetFrom(profile)),
-      profile_(profile) {
+      profile_(profile),
+      navigator_(navigator) {
   BrowserWindowGtk* browser_window =
       BrowserWindowGtk::GetBrowserWindowForNativeWindow(parent);
 
   anchor_ = browser_window->
       GetToolbar()->GetLocationBarView()->location_icon_widget();
 
   InitContents();
 
   BubbleGtk::ArrowLocationGtk arrow_location = base::i18n::IsRTL() ?
       BubbleGtk::ARROW_LOCATION_TOP_RIGHT :
@@ skipping 107 matching lines @@
 
   return section_box;
 }
 
 void PageInfoBubbleGtk::OnViewCertLinkClicked(GtkWidget* widget) {
   ShowCertificateViewerByID(GTK_WINDOW(parent_), cert_id_);
   bubble_->Close();
 }
 
 void PageInfoBubbleGtk::OnHelpLinkClicked(GtkWidget* widget) {
-  Browser* browser = browser::FindLastActiveWithProfile(profile_);
-  browser->OpenURL(OpenURLParams(
-      GURL(chrome::kPageInfoHelpCenterURL), content::Referrer(),
-      NEW_FOREGROUND_TAB, content::PAGE_TRANSITION_LINK, false));
+  navigator_->OpenURL(OpenURLParams(GURL(chrome::kPageInfoHelpCenterURL),
+                      content::Referrer(),
+                      NEW_FOREGROUND_TAB,
+                      content::PAGE_TRANSITION_LINK,
+                      false));
   bubble_->Close();
 }
 
 }  // namespace
 
 namespace browser {
 
 void ShowPageInfoBubble(gfx::NativeWindow parent,
                         Profile* profile,
                         const GURL& url,
                         const SSLStatus& ssl,
-                        bool show_history) {
-  new PageInfoBubbleGtk(parent, profile, url, ssl, show_history);
+                        bool show_history,
+                        content::PageNavigator* navigator) {
+  new PageInfoBubbleGtk(parent, profile, url, ssl, show_history, navigator);
 }
 
 }  // namespace browser