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

Unified Diff: ui/views/widget/widget.h

Issue 375183002: Add app.window.alphaEnabled() and onAlphaEnabledChanged. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: x11 Created 6 years, 5 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: ui/views/widget/widget.h
diff --git a/ui/views/widget/widget.h b/ui/views/widget/widget.h
index bc3f91473dd19de5175b0b939a69341424b7a329..0881e3e5214ef391c087d04da8c0fb9f116cfced 100644
--- a/ui/views/widget/widget.h
+++ b/ui/views/widget/widget.h
@@ -197,7 +197,8 @@ class VIEWS_EXPORT Widget : public internal::NativeWidgetDelegate,
// If NULL, a default implementation will be constructed.
WidgetDelegate* delegate;
bool child;
- // If TRANSLUCENT_WINDOW, the widget may be fully or partially transparent.
+ // If TRANSLUCENT_WINDOW, the widget may be fully or partially transparent,
+ // or not at all if IsTranslucentWindowOpacitySupported() is false.
sky 2014/07/17 15:52:32 That first sentence is meant to indicate what the
jackhou1 2014/07/18 00:41:15 Done.
// If OPAQUE_WINDOW, we can perform optimizations based on the widget being
// fully opaque. Defaults to TRANSLUCENT_WINDOW if
// ViewsDelegate::UseTransparentWindows(). Defaults to OPAQUE_WINDOW for
@@ -726,6 +727,9 @@ class VIEWS_EXPORT Widget : public internal::NativeWidgetDelegate,
// window sizing information to the window server on some platforms.
void OnRootViewLayout();
+ // Whether the widget can have a transparent background.
sky 2014/07/17 15:52:32 Whether the widget supports translucency.
jackhou1 2014/07/18 00:41:16 Done.
+ bool IsTranslucentWindowOpacitySupported() const;
+
// Notification that our owner is closing.
// NOTE: this is not invoked for aura as it's currently not needed there.
// Under aura menus close by way of activation getting reset when the owner

Powered by Google App Engine
This is Rietveld 408576698