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

Unified Diff: ui/app_list/presenter/app_list_presenter.h

Issue 2534953006: Fix shelf auto-hide calculation for app-list visibility. (Closed)
Patch Set: Cleanup. Created 4 years 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/presenter/app_list_presenter.h
diff --git a/ui/app_list/presenter/app_list_presenter.h b/ui/app_list/presenter/app_list_presenter.h
index 320564b5f40f024d086514ccf44f12cbef3633aa..62b70d2dfcebb5e680e32ba9a6206afad42384b4 100644
--- a/ui/app_list/presenter/app_list_presenter.h
+++ b/ui/app_list/presenter/app_list_presenter.h
@@ -8,6 +8,7 @@
#include <stdint.h>
#include "ui/app_list/presenter/app_list_presenter_export.h"
+#include "ui/display/types/display_constants.h"
namespace app_list {
@@ -29,11 +30,15 @@ class APP_LIST_PRESENTER_EXPORT AppListPresenter {
virtual void ToggleAppList(int64_t display_id) = 0;
// Returns current visibility of the app list.
- virtual bool IsVisible() const = 0;
+ // Pass a valid display id to check visibility for a specific display.
+ virtual bool IsVisible(
+ int64_t display_id = display::kInvalidDisplayId) const = 0;
// Returns target visibility. This may differ from IsVisible() if a
// visibility transition is in progress.
- virtual bool GetTargetVisibility() const = 0;
+ // Pass a valid display id to check target visibility for a specific display.
+ virtual bool GetTargetVisibility(
+ int64_t display_id = display::kInvalidDisplayId) const = 0;
};
} // namespace app_list

Powered by Google App Engine
This is Rietveld 408576698