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

Unified Diff: ui/app_list/views/app_list_view.h

Issue 276833002: Make the App Info Dialog appear modal (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed comment Created 6 years, 7 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/app_list/views/app_list_view.h
diff --git a/ui/app_list/views/app_list_view.h b/ui/app_list/views/app_list_view.h
index 03129774b83d77f39cd5647ffd644aeff9728179..ab2c2a3bcb50d06f76608f81570f3a85b0ad9d21 100644
--- a/ui/app_list/views/app_list_view.h
+++ b/ui/app_list/views/app_list_view.h
@@ -76,6 +76,11 @@ class APP_LIST_EXPORT AppListView : public views::BubbleDelegateView,
void UpdateBounds();
+ // When called with |visible| = true, displays a semi-transparent overlay over
+ // the app list (good for hiding the app list when a modal dialog is being
+ // shown). Call again with |visible| = false to hide.
+ void ShowAppListOverlay(bool visible);
tapted 2014/05/21 03:40:57 I'd go for SetAppListOverlayVisible(bool visible)
sashab 2014/05/22 07:20:21 Good suggestion. Made the comment simpler as well
+
// Returns true if the app list should be centered and in landscape mode.
bool ShouldCenterWindow() const;
@@ -149,6 +154,10 @@ class APP_LIST_EXPORT AppListView : public views::BubbleDelegateView,
SigninView* signin_view_;
SpeechView* speech_view_;
+ // A semi-transparent white overlay that covers the app list while dialogs are
+ // open.
+ views::View* overlay_view_;
+
ObserverList<AppListViewObserver> observers_;
scoped_ptr<HideViewAnimationObserver> animation_observer_;

Powered by Google App Engine
This is Rietveld 408576698