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

Unified Diff: ash/mus/app_list_presenter_mus.cc

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: ash/mus/app_list_presenter_mus.cc
diff --git a/ash/mus/app_list_presenter_mus.cc b/ash/mus/app_list_presenter_mus.cc
index 9bb13fbf99566f98d53ea2570efc2e76cac3fbe8..be4c53ae3f1de2563ea3eaefc9b4cc93c2cd581c 100644
--- a/ash/mus/app_list_presenter_mus.cc
+++ b/ash/mus/app_list_presenter_mus.cc
@@ -37,11 +37,11 @@ void AppListPresenterMus::ToggleAppList(int64_t display_id) {
presenter_->ToggleAppList(display_id);
}
-bool AppListPresenterMus::IsVisible() const {
+bool AppListPresenterMus::IsVisible(int64_t display_id) const {
return false;
}
-bool AppListPresenterMus::GetTargetVisibility() const {
+bool AppListPresenterMus::GetTargetVisibility(int64_t display_id) const {
// TODO(mfomitchev): we have GetTargetVisibility() in mojom, but this
// shouldn't be a synchronous method. We should go through the call sites and
// either teach them to use a callback, or perhaps use a visibility observer.

Powered by Google App Engine
This is Rietveld 408576698