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

Unified Diff: chrome/browser/ui/views/app_list/linux/app_list_linux.cc

Issue 253983002: Use centered app list position whenever virtual keyboard is enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed ExampleAppListViewDelegate (caused tree to close). 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: chrome/browser/ui/views/app_list/linux/app_list_linux.cc
diff --git a/chrome/browser/ui/views/app_list/linux/app_list_linux.cc b/chrome/browser/ui/views/app_list/linux/app_list_linux.cc
index 278d37404566626a2d23d9a900c0929a852eb849..e513121756ca360d80d6c8c1d15a30898d603dbf 100644
--- a/chrome/browser/ui/views/app_list/linux/app_list_linux.cc
+++ b/chrome/browser/ui/views/app_list/linux/app_list_linux.cc
@@ -53,11 +53,12 @@ AppListPositioner::ScreenEdge AppListLinux::ShelfLocationInDisplay(
gfx::Point AppListLinux::FindAnchorPoint(const gfx::Size& view_size,
const gfx::Display& display,
const gfx::Point& cursor,
- AppListPositioner::ScreenEdge edge) {
+ AppListPositioner::ScreenEdge edge,
+ bool center_window) {
AppListPositioner positioner(display, view_size, 0);
// Special case for app list in the center of the screen.
- if (app_list::switches::IsCenteredAppListEnabled())
+ if (center_window)
return positioner.GetAnchorPointForScreenCenter();
gfx::Point anchor;
@@ -101,6 +102,9 @@ void AppListLinux::MoveNearCursor(app_list::AppListView* view) {
edge = AppListPositioner::SCREEN_EDGE_LEFT;
else
edge = ShelfLocationInDisplay(display);
- view->SetAnchorPoint(
- FindAnchorPoint(view->GetPreferredSize(), display, cursor, edge));
+ view->SetAnchorPoint(FindAnchorPoint(view->GetPreferredSize(),
+ display,
+ cursor,
+ edge,
+ view->ShouldCenterWindow()));
}

Powered by Google App Engine
This is Rietveld 408576698