Add chooser permission UI code for Mac

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 "base/macros.h"
+#include "chrome/browser/ui/website_settings/chooser_bubble_delegate.h"
+#include "components/bubble/bubble_ui.h"
+
+class Browser;
+class ChooserBubbleDelegate;
+@class ChooserBubbleUiController;
+
+// ChooserBubbleUiCocoa implements a chooser-based permission model,
+// it uses table view to show a list of options for user to grant
+// permission. It can be used by WebUsb, WebBluetooth.

[palmer, 2015/12/10 23:26:10]

Nits: The first sentence should have the 2 clauses joined by a semicolon, or
should be 2 sentences.

Is table view a specific class? If so, spell it the way the class is named ("...
uses |TableView|..." or whatever).

"WebUSB or WebBluetooth" (i.e the names of the APIs) or "|WebUsb| or
|WebBluetooth|" (if those are the names of classes).

[juncai, 2015/12/11 20:39:46]

Done.

+class ChooserBubbleUiCocoa : public BubbleUi,
+                             public ChooserBubbleDelegate::Observer {
+ public:
+  ChooserBubbleUiCocoa(Browser* browser,
+                       ChooserBubbleDelegate* chooser_bubble_delegate);
+  ~ChooserBubbleUiCocoa() override;
+
+  // BubbleUi:
+  void Show(BubbleReference bubble_reference) override;
+  void Close() override;
+  void UpdateAnchorPosition() override;
+
+  // ChooserBubbleDelegate::Observer:
+  void OnOptionsInitialized() override;
+  void OnOptionAdded(int index) override;
+  void OnOptionRemoved(int index) override;
+
+  // Called when |chooser_bubble_ui_controller_| is closing.

[palmer, 2015/12/10 23:26:10]

Passive voice/leaving out the agent is confusing here. What code calls it?

[juncai, 2015/12/11 20:39:46]

ChooserBubbleUiController's windowWillClose calls it when the window needs to be
closed.
It is similar to:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...
and it is called here:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

+  void OnBubbleClosing();
+
+ private:
+  Browser* browser_;                                // Weak.
+  ChooserBubbleDelegate* chooser_bubble_delegate_;  // Weak.
+  // 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_