Remove |browser_| from WebsiteSettingsPopupView.

chrome/browser/ui/views/website_settings/website_settings_popup_view.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/ui/views/website_settings/permission_selector_view_observer.h"
 #include "chrome/browser/ui/website_settings/website_settings_ui.h"
 #include "ui/views/bubble/bubble_delegate.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/button/label_button.h"
 #include "ui/views/controls/link_listener.h"
 #include "ui/views/controls/tabbed_pane/tabbed_pane_listener.h"
 
 class Browser;

[sky, 2015/08/10 15:08:17]

If you can, remove this and any unnecessary includes.

[jackhou1, 2015/08/11 01:03:08]

Done.

 class GURL;
 class PermissionSelectorView;
 class PopupHeaderView;
 class Profile;
 
 namespace content {
 struct SSLStatus;
 class WebContents;
 }
 
@@ skipping 12 matching lines @@
       public views::LinkListener,
       public views::TabbedPaneListener,
       public WebsiteSettingsUI {
  public:
   ~WebsiteSettingsPopupView() override;
 
   static void ShowPopup(views::View* anchor_view,
                         Profile* profile,
                         content::WebContents* web_contents,
                         const GURL& url,
-                        const content::SSLStatus& ssl,
-                        Browser* browser);
+                        const content::SSLStatus& ssl);
 
   static bool IsPopupShowing();
 
  private:
   WebsiteSettingsPopupView(views::View* anchor_view,
                            Profile* profile,
                            content::WebContents* web_contents,
                            const GURL& url,
-                           const content::SSLStatus& ssl,
-                           Browser* browser);
+                           const content::SSLStatus& ssl);
 
   // PermissionSelectorViewObserver implementation.
   void OnPermissionChanged(
       const WebsiteSettingsUI::PermissionInfo& permission) override;
 
   // views::BubbleDelegateView implementation.
   void OnWidgetDestroying(views::Widget* widget) override;
 
   // views::ButtonListener implementation.
   void ButtonPressed(views::Button* button, const ui::Event& event) override;
@@ skipping 45 matching lines @@
 
   // Used to asynchronously handle clicks since these calls may cause the
   // destruction of the settings view and the base class window still needs to
   // be alive to finish handling the mouse or keyboard click.
   void HandleLinkClickedAsync(views::Link* source);
 
   // The web contents of the current tab. The popup can't live longer than a
   // tab.
   content::WebContents* web_contents_;
 
-  // The Browser is used to load the help center page.
-  Browser* browser_;
-
   // The presenter that controls the Website Settings UI.
   scoped_ptr<WebsiteSettings> presenter_;
 
   PopupHeaderView* header_;  // Owned by views hierarchy.
 
   // The |TabbedPane| that contains the tabs of the Website Settings UI.
   views::TabbedPane* tabbed_pane_;
 
   // The view that contains the permissions tab contents.
   views::View* permissions_tab_;
@@ skipping 35 matching lines @@
   views::Link* site_settings_link_;
 
   views::View* connection_info_content_;
 
   base::WeakPtrFactory<WebsiteSettingsPopupView> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettingsPopupView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_