Refactor ChooserBubbleController

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

 // Copyright 2015 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_CHOOSER_BUBBLE_UI_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_CHOOSER_BUBBLE_UI_VIEW_H_
 
 #include "components/bubble/bubble_ui.h"
 #include "ui/views/bubble/bubble_border.h"
 
 namespace views {
 class View;
 }
 
 class Browser;
-class ChooserBubbleController;
+class ChooserController;
 class ChooserBubbleUiViewDelegate;
 
 // ChooserBubbleUiView implements a chooser-based permission model,
 // it uses table view to show a list of items (such as usb devices, etc.)
 // for user to grant permission. It can be used by WebUsb, WebBluetooth.
 class ChooserBubbleUiView : public BubbleUi {
  public:
-  ChooserBubbleUiView(Browser* browser, ChooserBubbleController* controller);
+  ChooserBubbleUiView(Browser* browser, ChooserController* controller);

[sky, 2016/05/17 21:10:45]

Document ownership.

[juncai, 2016/05/19 23:29:14]

Done.

   ~ChooserBubbleUiView() override;
 
   // BubbleUi:
   void Show(BubbleReference bubble_reference) override;
   void Close() override;
   void UpdateAnchorPosition() override;
 
  private:
   friend class ChooserBubbleUiViewDelegate;
   views::View* GetAnchorView();
   views::BubbleBorder::Arrow GetAnchorArrow();
 
   Browser* browser_;
-  ChooserBubbleController* controller_;
+  ChooserController* controller_;
   ChooserBubbleUiViewDelegate* chooser_bubble_ui_view_delegate_;
 };

[sky, 2016/05/17 21:10:45]

nit: DISALLOW_...

[juncai, 2016/05/19 23:29:14]

Done.

 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_CHOOSER_BUBBLE_UI_VIEW_H_