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

Side by Side Diff: ash/accelerators/accelerator_interactive_uitest_chromeos.cc

Issue 2734933004: ash: Use SessionController instead of SessionStateDelegate (Closed)
Patch Set: rebase Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ash/common/accelerators/accelerator_controller.h" 5 #include "ash/common/accelerators/accelerator_controller.h"
6 6
7 #include "ash/common/shell_observer.h" 7 #include "ash/common/shell_observer.h"
8 #include "ash/common/system/chromeos/network/network_observer.h" 8 #include "ash/common/system/chromeos/network/network_observer.h"
9 #include "ash/common/system/tray/system_tray_delegate.h" 9 #include "ash/common/system/tray/system_tray_delegate.h"
10 #include "ash/common/system/tray/system_tray_notifier.h" 10 #include "ash/common/system/tray/system_tray_notifier.h"
11 #include "ash/common/wm/window_state.h" 11 #include "ash/common/wm/window_state.h"
12 #include "ash/common/wm_shell.h" 12 #include "ash/common/wm_shell.h"
13 #include "ash/shell.h" 13 #include "ash/shell.h"
14 #include "ash/test/ash_interactive_ui_test_base.h" 14 #include "ash/test/ash_interactive_ui_test_base.h"
15 #include "ash/test/test_screenshot_delegate.h" 15 #include "ash/test/test_screenshot_delegate.h"
16 #include "ash/wm/window_state_aura.h" 16 #include "ash/wm/window_state_aura.h"
17 #include "ash/wm/window_util.h" 17 #include "ash/wm/window_util.h"
18 #include "base/run_loop.h" 18 #include "base/run_loop.h"
19 #include "base/test/user_action_tester.cc" 19 #include "base/test/user_action_tester.cc"
20 #include "chromeos/network/network_handler.h" 20 #include "chromeos/network/network_handler.h"
21 #include "mojo/edk/embedder/embedder.h"
22 #include "ui/app_list/presenter/app_list.h" 21 #include "ui/app_list/presenter/app_list.h"
23 #include "ui/app_list/presenter/test/test_app_list_presenter.h" 22 #include "ui/app_list/presenter/test/test_app_list_presenter.h"
24 #include "ui/base/test/ui_controls.h" 23 #include "ui/base/test/ui_controls.h"
25 24
26 namespace ash { 25 namespace ash {
27 namespace test { 26 namespace test {
28 27
29 namespace { 28 namespace {
30 29
31 // A network observer to watch for the toggle wifi events. 30 // A network observer to watch for the toggle wifi events.
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
191 EXPECT_TRUE(network_observer.wifi_enabled_status()); 190 EXPECT_TRUE(network_observer.wifi_enabled_status());
192 SendKeyPressSync(ui::VKEY_WLAN, false, false, false); 191 SendKeyPressSync(ui::VKEY_WLAN, false, false, false);
193 EXPECT_FALSE(network_observer.wifi_enabled_status()); 192 EXPECT_FALSE(network_observer.wifi_enabled_status());
194 193
195 WmShell::Get()->system_tray_notifier()->RemoveNetworkObserver( 194 WmShell::Get()->system_tray_notifier()->RemoveNetworkObserver(
196 &network_observer); 195 &network_observer);
197 } 196 }
198 197
199 // Tests the app list accelerator. 198 // Tests the app list accelerator.
200 TEST_F(AcceleratorInteractiveUITest, MAYBE_ToggleAppList) { 199 TEST_F(AcceleratorInteractiveUITest, MAYBE_ToggleAppList) {
201 mojo::edk::Init();
xiyuan 2017/03/17 07:08:15 Removed because mojo init is added to AshInteracti
202 app_list::test::TestAppListPresenter test_app_list_presenter; 200 app_list::test::TestAppListPresenter test_app_list_presenter;
203 Shell::Get()->app_list()->SetAppListPresenter( 201 Shell::Get()->app_list()->SetAppListPresenter(
204 test_app_list_presenter.CreateInterfacePtrAndBind()); 202 test_app_list_presenter.CreateInterfacePtrAndBind());
205 203
206 EXPECT_EQ(0u, test_app_list_presenter.toggle_count()); 204 EXPECT_EQ(0u, test_app_list_presenter.toggle_count());
207 SendKeyPressSync(ui::VKEY_LWIN, false, false, false); 205 SendKeyPressSync(ui::VKEY_LWIN, false, false, false);
208 RunAllPendingInMessageLoop(); 206 RunAllPendingInMessageLoop();
209 EXPECT_EQ(1u, test_app_list_presenter.toggle_count()); 207 EXPECT_EQ(1u, test_app_list_presenter.toggle_count());
210 SendKeyPressSync(ui::VKEY_LWIN, false, false, false); 208 SendKeyPressSync(ui::VKEY_LWIN, false, false, false);
211 RunAllPendingInMessageLoop(); 209 RunAllPendingInMessageLoop();
212 EXPECT_EQ(2u, test_app_list_presenter.toggle_count()); 210 EXPECT_EQ(2u, test_app_list_presenter.toggle_count());
213 } 211 }
214 212
215 } // namespace test 213 } // namespace test
216 } // namespace ash 214 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698