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

Unified Diff: ash/common/wm/overview/window_selector_item.h

Issue 2239233002: [ash-md] Fades overview header in and out (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: [ash-md] Fades overview header in and out (rebase) Created 4 years, 4 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: ash/common/wm/overview/window_selector_item.h
diff --git a/ash/common/wm/overview/window_selector_item.h b/ash/common/wm/overview/window_selector_item.h
index d3467685e639fd637ca260f2d90830debf0eee42..1df2091bac7956c0cda4be60a18560bdd6209253 100644
--- a/ash/common/wm/overview/window_selector_item.h
+++ b/ash/common/wm/overview/window_selector_item.h
@@ -161,6 +161,9 @@ class ASH_EXPORT WindowSelectorItem : public views::ButtonListener,
// Updates the close buttons accessibility name.
void UpdateCloseButtonAccessibilityName();
+ // Fades out a window caption when exiting overview mode.
+ void FadeOut(std::unique_ptr<views::Widget> widget);
+
static bool hide_header() { return use_mask_ || use_shape_; }
// True if the item is being shown in the overview, false if it's being
@@ -181,6 +184,14 @@ class ASH_EXPORT WindowSelectorItem : public views::ButtonListener,
// a window layer for display on another monitor.
bool in_bounds_update_;
+ // True when |this| item is visually selected. Item header is made transparent
+ // when the item is selected.
+ bool selected_;
+
+ // True initially until the UpdateHeaderLayout is invoked for the first time.
+ // Provided in order to set the initial bounds for the header to animate from.
+ bool first_time_;
bruthig 2016/08/30 17:57:33 nit: A more explicit var name would be very helpfu
varkha 2016/09/02 11:22:51 How is this?
+
// Label displaying its name (active tab for tabbed windows).
// With Material Design this Widget owns |caption_container_view_| and is
// shown above the |transform_window_|.

Powered by Google App Engine
This is Rietveld 408576698