Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7759)

Unified Diff: chrome/browser/ui/website_settings/chooser_bubble_delegate.h

Issue 1473393003: Add chooser permission UI code for Mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@wpu_1
Patch Set: updated code to calculate width for table column Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/website_settings/chooser_bubble_delegate.h
diff --git a/chrome/browser/ui/website_settings/chooser_bubble_delegate.h b/chrome/browser/ui/website_settings/chooser_bubble_delegate.h
index 2ea9151da7f9bf7659bd3bbe014551daba675483..0b834da0fbf7fae4372c4f0b1693b21f640b600f 100644
--- a/chrome/browser/ui/website_settings/chooser_bubble_delegate.h
+++ b/chrome/browser/ui/website_settings/chooser_bubble_delegate.h
@@ -8,15 +8,22 @@
#include "base/macros.h"
#include "components/bubble/bubble_delegate.h"
+class Browser;
class ChooserOptions;
class ChooserBubbleDelegate : public BubbleDelegate {
public:
- ChooserBubbleDelegate();
+ explicit ChooserBubbleDelegate(Browser* browser);
~ChooserBubbleDelegate() override;
// BubbleDelegate:
std::string GetName() const override;
+ scoped_ptr<BubbleUi> BuildBubbleUi() override;
+
+ static scoped_ptr<BubbleUi> CreateBubbleUi(
+ Browser* browser,
+ ChooserOptions* chooser_options,
+ ChooserBubbleDelegate* chooser_bubble_delegate);
Reilly Grant (use Gerrit) 2015/12/08 19:26:57 This function does not need to be static. If it is
juncai 2015/12/09 03:23:54 Done.
virtual void Select(int index) = 0;
virtual void Cancel() = 0;
@@ -26,6 +33,7 @@ class ChooserBubbleDelegate : public BubbleDelegate {
scoped_ptr<ChooserOptions> chooser_options_;
private:
+ Browser* browser_;
DISALLOW_COPY_AND_ASSIGN(ChooserBubbleDelegate);
};

Powered by Google App Engine
This is Rietveld 408576698