Change ChooserController ownership model

chrome/browser/ui/cocoa/website_settings/chooser_bubble_ui_cocoa.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_COCOA_WEBSITE_SETTINGS_CHOOSER_BUBBLE_UI_COCOA_H_
 #define CHROME_BROWSER_UI_COCOA_WEBSITE_SETTINGS_CHOOSER_BUBBLE_UI_COCOA_H_
 
 #import <Cocoa/Cocoa.h>
 
+#include <memory>
+
 #include "base/macros.h"
 #include "components/bubble/bubble_ui.h"
-#include "components/chooser_controller/chooser_controller.h"
 
 class Browser;
 @class ChooserBubbleUiController;
+class ChooserController;
 
 // ChooserBubbleUiCocoa implements a chooser-based permission model.
 // It uses |NSTableView| to show a list of options for user to grant
 // permission. It can be used by the WebUSB or WebBluetooth APIs.
 // It is owned by the BubbleController, which is owned by the BubbleManager.
-class ChooserBubbleUiCocoa : public BubbleUi,
-                             public ChooserController::Observer {
+class ChooserBubbleUiCocoa : public BubbleUi {
  public:
-  ChooserBubbleUiCocoa(Browser* browser, ChooserController* chooser_controller);
+  ChooserBubbleUiCocoa(Browser* browser,
+                       std::unique_ptr<ChooserController> chooser_controller);
   ~ChooserBubbleUiCocoa() override;
 
   // BubbleUi:
   void Show(BubbleReference bubble_reference) override;
   void Close() override;
   void UpdateAnchorPosition() override;
 
-  // ChooserController::Observer:
-  void OnOptionsInitialized() override;
-  void OnOptionAdded(size_t index) override;
-  void OnOptionRemoved(size_t index) override;
-
   // Called when |chooser_bubble_ui_controller_| is closing.
   void OnBubbleClosing();
 
  private:
-  Browser* browser_;                       // Weak.
-  ChooserController* chooser_controller_;  // Weak.
+  Browser* browser_;  // Weak.
+  // |chooser_controller_| is not owned by this class.
+  // It is owned by ChooserContentViewCocoa.
+  std::unique_ptr<ChooserController> chooser_controller_;

[Reilly Grant (use Gerrit), 2016/06/22 00:01:32]

The comment is confusing because it seems to be directly contradicted by the
fact that this is a unique_ptr. I think instead you mean to say that this field
only temporarily owns the ChooserController. It is moved into the
ChooserContentViewCocoa when BuildBubbleUi is called.

[juncai, 2016/06/22 01:17:45]

Done.

   // Cocoa-side chooser bubble UI controller. Weak, as it will close itself.
   ChooserBubbleUiController* chooser_bubble_ui_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(ChooserBubbleUiCocoa);
 };
 
 #endif  // CHROME_BROWSER_UI_COCOA_WEBSITE_SETTINGS_CHOOSER_BUBBLE_UI_COCOA_H_