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

Unified Diff: chrome/browser/ui/cocoa/extensions/toolbar_actions_bar_bubble_views_presenter.h

Issue 2630473003: MacViews: Harmony for toolbar actions bubbles. (Closed)
Patch Set: base::EraseIf Created 3 years, 9 months 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/cocoa/extensions/toolbar_actions_bar_bubble_views_presenter.h
diff --git a/chrome/browser/ui/cocoa/extensions/toolbar_actions_bar_bubble_views_presenter.h b/chrome/browser/ui/cocoa/extensions/toolbar_actions_bar_bubble_views_presenter.h
new file mode 100644
index 0000000000000000000000000000000000000000..9d32da85422999f98b6f0f42bbe3adaea778cb43
--- /dev/null
+++ b/chrome/browser/ui/cocoa/extensions/toolbar_actions_bar_bubble_views_presenter.h
@@ -0,0 +1,56 @@
+// Copyright 2017 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_EXTENSIONS_TOOLBAR_ACTIONS_BAR_BUBBLE_VIEWS_PRESENTER_H_
+#define CHROME_BROWSER_UI_COCOA_EXTENSIONS_TOOLBAR_ACTIONS_BAR_BUBBLE_VIEWS_PRESENTER_H_
+
+#import <Foundation/Foundation.h>
+
+#include <memory>
+
+#include "base/macros.h"
+#include "ui/views/widget/widget_observer.h"
+
+@class NSWindow;
+
+namespace test {
+class ToolbarActionsBarBubbleViewsPresenterTestApi;
+}
+
+@class BrowserActionsController;
+class ToolbarActionsBarBubbleDelegate;
+class ToolbarActionsBarBubbleViews;
+
+// Helper class for showing a toolkit-views toolbar action bubble on a Cocoa
+// browser.
+class ToolbarActionsBarBubbleViewsPresenter : public views::WidgetObserver {
+ public:
+ explicit ToolbarActionsBarBubbleViewsPresenter(
+ BrowserActionsController* owner);
+ ~ToolbarActionsBarBubbleViewsPresenter() override;
+
+ // Presents |bubble| attached to the provided browser |parent_window| at
+ // |point_in_screen|. |anchored_to_action_view| indicates that the anchor is
+ // a specific browser action view, rather than something more general.
+ void PresentAt(std::unique_ptr<ToolbarActionsBarBubbleDelegate> bubble,
+ NSWindow* parent_window,
+ NSPoint point_in_screen,
+ bool anchored_to_action_view);
+
+ private:
+ friend class test::ToolbarActionsBarBubbleViewsPresenterTestApi;
+
+ // WidgetObserver:
+ void OnWidgetClosing(views::Widget* widget) override;
+ void OnWidgetDestroying(views::Widget* widget) override;
+
+ BrowserActionsController* owner_; // Weak. Owns |this|.
+
+ // Weak. Owns by its Widget (observed by |this|).
+ ToolbarActionsBarBubbleViews* active_bubble_;
+
+ DISALLOW_COPY_AND_ASSIGN(ToolbarActionsBarBubbleViewsPresenter);
+};
+
+#endif // CHROME_BROWSER_UI_COCOA_EXTENSIONS_TOOLBAR_ACTIONS_BAR_BUBBLE_VIEWS_PRESENTER_H_

Powered by Google App Engine
This is Rietveld 408576698