Remove browser from web intents UI code

chrome/browser/ui/views/web_intent_picker_views.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 <vector>
 
 #include "base/memory/scoped_vector.h"
 #include "base/time.h"
 #include "chrome/browser/tab_contents/tab_util.h"
-#include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/intents/web_intent_inline_disposition_delegate.h"
 #include "chrome/browser/ui/intents/web_intent_picker.h"
 #include "chrome/browser/ui/intents/web_intent_picker_delegate.h"
 #include "chrome/browser/ui/intents/web_intent_picker_model.h"
 #include "chrome/browser/ui/intents/web_intent_picker_model_observer.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/browser/ui/views/constrained_window_views.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/location_bar/location_icon_view.h"
@@ skipping 633 matching lines @@
 
 // Views implementation of WebIntentPicker.
 class WebIntentPickerViews : public views::ButtonListener,
                              public views::DialogDelegate,
                              public views::LinkListener,
                              public WebIntentPicker,
                              public WebIntentPickerModelObserver,
                              public ServiceButtonsView::Delegate,
                              public SuggestedExtensionsRowView::Delegate {
  public:
-  WebIntentPickerViews(Browser* browser,
-                       TabContentsWrapper* tab_contents,
+  WebIntentPickerViews(TabContentsWrapper* tab_contents,
                        WebIntentPickerDelegate* delegate,
                        WebIntentPickerModel* model);
   virtual ~WebIntentPickerViews();
 
   // views::ButtonListener implementation.
   virtual void ButtonPressed(views::Button* sender,
                              const views::Event& event) OVERRIDE;
 
   // views::DialogDelegate implementation.
   virtual void WindowClosing() OVERRIDE;
@@ skipping 59 matching lines @@
 
   // A weak pointer to the header label for the extension suggestions.
   views::Label* suggestions_label_;
 
   // A weak pointer to the extensions view.
   SuggestedExtensionsView* extensions_;
 
   // Delegate for inline disposition tab contents.
   scoped_ptr<WebIntentInlineDispositionDelegate> inline_disposition_delegate_;
 
-  // A weak pointer to the browser this picker is in.
-  Browser* browser_;
+  // A weak pointer to the wrapper of the WebContents this picker is in.
+  TabContentsWrapper* wrapper_;
 
   // A weak pointer to the WebView that hosts the WebContents being displayed.
   views::WebView* webview_;
 
   // A weak pointer to the view that contains all other views in the picker.
   views::View* contents_;
 
   // A weak pointer to the constrained window.
   ConstrainedWindowViews* window_;
 
   // A weak pointer to the more suggestions link.
   views::Link* more_suggestions_link_;
 
   // A weak pointer to the choose another service link.
   views::Link* choose_another_service_link_;
 
   // Set to true when displaying the inline disposition web contents. Used to
   // prevent laying out the inline disposition widgets twice.
   bool displaying_web_contents_;
 
   // Ownership of the WebContents we are displaying in the inline disposition.
   scoped_ptr<WebContents> inline_web_contents_;
 
   DISALLOW_COPY_AND_ASSIGN(WebIntentPickerViews);
 };
 
 // static
-WebIntentPicker* WebIntentPicker::Create(Browser* browser,
-                                         TabContentsWrapper* wrapper,
+WebIntentPicker* WebIntentPicker::Create(TabContentsWrapper* wrapper,
                                          WebIntentPickerDelegate* delegate,
                                          WebIntentPickerModel* model) {
   WebIntentPickerViews* picker =
-      new WebIntentPickerViews(browser, wrapper, delegate, model);
+      new WebIntentPickerViews(wrapper, delegate, model);
 
   return picker;
 }
 
-WebIntentPickerViews::WebIntentPickerViews(Browser* browser,
-                                           TabContentsWrapper* wrapper,
+WebIntentPickerViews::WebIntentPickerViews(TabContentsWrapper* wrapper,
                                            WebIntentPickerDelegate* delegate,
                                            WebIntentPickerModel* model)
     : delegate_(delegate),
       model_(model),
       service_buttons_(NULL),
       action_label_(NULL),
       suggestions_label_(NULL),
       extensions_(NULL),
-      browser_(browser),
-      webview_(new views::WebView(browser->profile())),
+      wrapper_(wrapper),
+      webview_(new views::WebView(wrapper->profile())),
       contents_(NULL),
       window_(NULL),
       more_suggestions_link_(NULL),
       choose_another_service_link_(NULL),
       displaying_web_contents_(false) {
   model_->set_observer(this);
   InitContents();
 
   // Show the dialog.
   window_ = new ConstrainedWindowViews(wrapper, this);
@@ skipping 153 matching lines @@
     WebIntentPickerModel* model,
     const string16& extension_id) {
   extensions_->Update();
   contents_->Layout();
   SizeToContents();
 }
 
 void WebIntentPickerViews::OnInlineDisposition(
     WebIntentPickerModel* model, const GURL& url) {
   inline_web_contents_.reset(WebContents::Create(
-      browser_->profile(),
-      tab_util::GetSiteInstanceForNewTab(browser_->profile(), url),
+      wrapper_->profile(),
+      tab_util::GetSiteInstanceForNewTab(wrapper_->profile(), url),
       MSG_ROUTING_NONE, NULL, NULL));
   // Does not take ownership, so we keep a scoped_ptr
   // for the WebContents locally.
   webview_->SetWebContents(inline_web_contents_.get());
   inline_disposition_delegate_.reset(
       new WebIntentInlineDispositionDelegate(this));
   content::WebContents* web_contents = webview_->GetWebContents();
   web_contents->SetDelegate(inline_disposition_delegate_.get());
 
   const WebIntentPickerModel::InstalledService* service =
@@ skipping 134 matching lines @@
   views::ImageButton* close_button = new views::ImageButton(this);
   close_button->SetImage(views::CustomButton::BS_NORMAL,
                           rb.GetBitmapNamed(IDR_CLOSE_BAR));
   close_button->SetImage(views::CustomButton::BS_HOT,
                           rb.GetBitmapNamed(IDR_CLOSE_BAR_H));
   close_button->SetImage(views::CustomButton::BS_PUSHED,
                           rb.GetBitmapNamed(IDR_CLOSE_BAR_P));
   return close_button;
 }
 #endif