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

Unified Diff: ash/mus/app_list_presenter_mus.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: ash/mus/app_list_presenter_mus.h
diff --git a/ash/mus/app_list_presenter_mus.h b/ash/mus/app_list_presenter_mus.h
index 2ed64ee8cc20e0aa7a466974c4104c1bf199e4d3..3205936a3e004518de8f4722435a76956df05f70 100644
--- a/ash/mus/app_list_presenter_mus.h
+++ b/ash/mus/app_list_presenter_mus.h
@@ -26,8 +26,10 @@ class AppListPresenterMus : public app_list::AppListPresenter {
void Show(int64_t display_id) override;
void Dismiss() override;
void ToggleAppList(int64_t display_id) override;
- bool IsVisible() const override;
- bool GetTargetVisibility() const override;
+ bool IsVisible(
+ int64_t display_id = display::kInvalidDisplayId) const override;
James Cook 2016/11/30 23:27:19 I don't think you're allowed to use default parame
+ bool GetTargetVisibility(
+ int64_t display_id = display::kInvalidDisplayId) const override;
private:
// Connect to the app list service in chrome if the connection hasn't

Powered by Google App Engine
This is Rietveld 408576698