Refactoring: Remove an uneeded argment from TranslateBubbleView::TranslateBubbleView

chrome/browser/ui/views/translate/translate_bubble_view.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "chrome/browser/ui/translate/language_combobox_model.h"
 #include "chrome/browser/ui/translate/translate_bubble_model.h"
 #include "chrome/common/translate/translate_errors.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "ui/views/bubble/bubble_delegate.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/combobox/combobox_listener.h"
 #include "ui/views/controls/link_listener.h"
 
 class Browser;

[sky, 2013/12/19 16:45:21]

I think you can nuke this too.

[hajimehoshi, 2013/12/20 03:25:45]

Done.

 class PrefService;
 class TranslateBubbleModel;
 
 namespace views {
 class Checkbox;
 class GridLayout;
 class LabelButton;
 class Link;
 class View;
 }
 
 class TranslateBubbleView : public views::BubbleDelegateView,
                             public views::ButtonListener,
                             public views::ComboboxListener,
                             public views::LinkListener,
                             public content::WebContentsObserver {
  public:
   virtual ~TranslateBubbleView();
 
   // Shows the Translate bubble.
   static void ShowBubble(views::View* anchor_view,
                          content::WebContents* web_contents,
                          TranslateBubbleModel::ViewState type,
-                         TranslateErrors::Type error_type,
-                         Browser* browser);
+                         TranslateErrors::Type error_type);
 
   // If true, the Translate bubble is being shown.
   static bool IsShowing();
 
   // Returns the bubble view currently shown. This may return NULL.
   static TranslateBubbleView* GetCurrentBubble();
 
   TranslateBubbleModel* model() { return model_.get(); }
 
   // views::BubbleDelegateView methods.
@@ skipping 57 matching lines @@
                            DoneButtonWithoutTranslating);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
                            CancelButtonReturningBeforeTranslate);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
                            CancelButtonReturningAfterTranslate);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, CancelButtonReturningError);
 
   TranslateBubbleView(views::View* anchor_view,
                       scoped_ptr<TranslateBubbleModel> model,
                       TranslateErrors::Type error_type,
-                      Browser* browser,
                       content::WebContents* web_contents);
 
   // Returns the current child view.
   views::View* GetCurrentView();
 
   // Handles the event when the user presses a button.
   void HandleButtonPressed(ButtonID sender_id);
 
   // Handles the event when the user clicks a link.
   void HandleLinkClicked(LinkID sender_id);
@@ skipping 51 matching lines @@
   views::LabelButton* advanced_cancel_button_;
   views::LabelButton* advanced_done_button_;
 
   scoped_ptr<TranslateBubbleModel> model_;
 
   TranslateErrors::Type error_type_;
 
   // Whether the window is an incognito window.
   const bool is_in_incognito_window_;
 
-  // The browser to open the help URL into a new tab.
-  Browser* browser_;
-
   // Whether the translation is acutually executed.
   bool translate_executed_;
 
   // Whether one of denial buttons is clicked.
   bool denial_button_clicked_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateBubbleView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_