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

Unified Diff: ash/app_list/app_list_presenter_delegate.cc

Issue 2802903003: Implementation of a full screen app list and re-alphabetized switches (Closed)
Patch Set: addressed comments, refactored Initialize, modified tests to add test coverage for the fullscreen l… Created 3 years, 8 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: ash/app_list/app_list_presenter_delegate.cc
diff --git a/ash/app_list/app_list_presenter_delegate.cc b/ash/app_list/app_list_presenter_delegate.cc
index dbc28e6e2237c28b47cc9a2fb335711877875c13..a5be5e05d18c9d3dcdf91d00843e46e0008df004 100644
--- a/ash/app_list/app_list_presenter_delegate.cc
+++ b/ash/app_list/app_list_presenter_delegate.cc
@@ -99,11 +99,15 @@ void AppListPresenterDelegate::Init(app_list::AppListView* view,
aura::Window* root_window = wm_root_window->aura_window();
aura::Window* container = GetRootWindowController(root_window)
->GetContainer(kShellWindowId_AppListContainer);
- view->InitAsBubble(container, current_apps_page);
- // The app list is centered over the display.
- view->SetAnchorPoint(GetCenterOfDisplayForWindow(
- wm_root_window, GetMinimumBoundsHeightForAppList(view)));
-
+ view->Initialize(
+ container, current_apps_page,
+ ash::ScreenUtil::GetDisplayWorkAreaBoundsInParent(container));
+
+ if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
+ app_list::switches::kEnableFullscreenAppList)) {
+ view->SetAnchorPoint(GetCenterOfDisplayForWindow(
+ wm_root_window, GetMinimumBoundsHeightForAppList(view)));
+ }
keyboard::KeyboardController* keyboard_controller =
keyboard::KeyboardController::GetInstance();
if (keyboard_controller)

Powered by Google App Engine
This is Rietveld 408576698