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

Unified Diff: chrome/browser/ui/views/exclusive_access_bubble_views_context.h

Issue 1654723002: Enable showing the toolkit-views simplified fullscreen UI on Mac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@20150112-MacViews-NewFullscreenBubble
Patch Set: Fix zero-height bubble Created 4 years, 11 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/views/exclusive_access_bubble_views_context.h
diff --git a/chrome/browser/ui/views/exclusive_access_bubble_views_context.h b/chrome/browser/ui/views/exclusive_access_bubble_views_context.h
index 5d0f3d457f3c3e1c7b8bb723e0cc4ef771c23718..c9e9e5be43ced94785d8e16003187ffae405193d 100644
--- a/chrome/browser/ui/views/exclusive_access_bubble_views_context.h
+++ b/chrome/browser/ui/views/exclusive_access_bubble_views_context.h
@@ -4,7 +4,9 @@
#ifndef CHROME_BROWSER_UI_VIEWS_EXCLUSIVE_ACCESS_BUBBLE_VIEWS_CONTEXT_H_
#define CHROME_BROWSER_UI_VIEWS_EXCLUSIVE_ACCESS_BUBBLE_VIEWS_CONTEXT_H_
+#include "ui/base/accelerators/accelerator.h"
#include "ui/gfx/geometry/rect.h"
+#include "ui/gfx/native_widget_types.h"
class ExclusiveAccessManager;
@@ -13,18 +15,26 @@ class Widget;
}
// Context in which the exclusive access bubble view is initiated.
-class ExclusiveAccessBubbleViewsContext {
+class ExclusiveAccessBubbleViewsContext : public ui::AcceleratorProvider {
msw 2016/02/03 18:52:24 Eh, I'm on the fence about this; should this inter
tapted 2016/02/03 21:58:25 Ah, that's nicer. I started with something like Ge
public:
- virtual ~ExclusiveAccessBubbleViewsContext() {}
-
// Returns ExclusiveAccessManager controlling exclusive access for the given
// webview.
virtual ExclusiveAccessManager* GetExclusiveAccessManager() = 0;
// Returns the Widget that hosts the view containing the exclusive access
- // bubble.
+ // bubble. Not used for the simplified fullscreen UI.
virtual views::Widget* GetBubbleAssociatedWidget() = 0;
+ // Returns the view used to parent the bubble Widget.
+ virtual gfx::NativeView GetParentView() const = 0;
msw 2016/02/03 18:52:24 nit: maybe |GetBubbleParentView|?
tapted 2016/02/03 21:58:25 Done.
+
+ // Return the current mouse cursor location, offset from the top-left of the
+ // parent window.
+ virtual gfx::Point GetCursorPointInParent() const = 0;
+
+ // Return the current bounds (not restored bounds) of the parent window.
+ virtual gfx::Rect GetClientAreaBoundsInScreen() const = 0;
+
// Returns true if immersive mode is enabled.
virtual bool IsImmersiveModeEnabled() = 0;

Powered by Google App Engine
This is Rietveld 408576698