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

Side by Side Diff: extensions/shell/browser/shell_desktop_controller_aura_unittest.cc

Issue 696063008: Refactor ShellDesktopController and ShellNativeAppWindow to allow for non-aura implementations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: + Created 6 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "extensions/shell/browser/shell_desktop_controller.h" 5 #include "extensions/shell/browser/shell_desktop_controller_aura.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/time/time.h" 9 #include "base/time/time.h"
10 #include "ui/aura/test/aura_test_base.h" 10 #include "ui/aura/test/aura_test_base.h"
11 11
12 #if defined(OS_CHROMEOS) 12 #if defined(OS_CHROMEOS)
13 #include "chromeos/dbus/dbus_thread_manager.h" 13 #include "chromeos/dbus/dbus_thread_manager.h"
14 #include "chromeos/dbus/fake_power_manager_client.h" 14 #include "chromeos/dbus/fake_power_manager_client.h"
15 #endif 15 #endif
16 16
17 namespace extensions { 17 namespace extensions {
18 18
19 class ShellDesktopControllerTest : public aura::test::AuraTestBase { 19 class ShellDesktopControllerAuraTest : public aura::test::AuraTestBase {
20 public: 20 public:
21 ShellDesktopControllerTest() 21 ShellDesktopControllerAuraTest()
22 #if defined(OS_CHROMEOS) 22 #if defined(OS_CHROMEOS)
23 : power_manager_client_(NULL) 23 : power_manager_client_(NULL)
24 #endif 24 #endif
25 { 25 {
26 } 26 }
27 ~ShellDesktopControllerTest() override {} 27 ~ShellDesktopControllerAuraTest() override {}
28 28
29 void SetUp() override { 29 void SetUp() override {
30 #if defined(OS_CHROMEOS) 30 #if defined(OS_CHROMEOS)
31 scoped_ptr<chromeos::DBusThreadManagerSetter> dbus_setter = 31 scoped_ptr<chromeos::DBusThreadManagerSetter> dbus_setter =
32 chromeos::DBusThreadManager::GetSetterForTesting(); 32 chromeos::DBusThreadManager::GetSetterForTesting();
33 power_manager_client_ = new chromeos::FakePowerManagerClient(); 33 power_manager_client_ = new chromeos::FakePowerManagerClient();
34 dbus_setter->SetPowerManagerClient(make_scoped_ptr(power_manager_client_)); 34 dbus_setter->SetPowerManagerClient(make_scoped_ptr(power_manager_client_));
35 #endif 35 #endif
36 aura::test::AuraTestBase::SetUp(); 36 aura::test::AuraTestBase::SetUp();
37 controller_.reset(new ShellDesktopController()); 37 controller_.reset(new ShellDesktopControllerAura());
38 } 38 }
39 39
40 void TearDown() override { 40 void TearDown() override {
41 controller_.reset(); 41 controller_.reset();
42 aura::test::AuraTestBase::TearDown(); 42 aura::test::AuraTestBase::TearDown();
43 #if defined(OS_CHROMEOS) 43 #if defined(OS_CHROMEOS)
44 chromeos::DBusThreadManager::Shutdown(); 44 chromeos::DBusThreadManager::Shutdown();
45 #endif 45 #endif
46 } 46 }
47 47
48 protected: 48 protected:
49 scoped_ptr<ShellDesktopController> controller_; 49 scoped_ptr<ShellDesktopControllerAura> controller_;
50 50
51 #if defined(OS_CHROMEOS) 51 #if defined(OS_CHROMEOS)
52 chromeos::FakePowerManagerClient* power_manager_client_; // Not owned. 52 chromeos::FakePowerManagerClient* power_manager_client_; // Not owned.
53 #endif 53 #endif
54 54
55 private: 55 private:
56 DISALLOW_COPY_AND_ASSIGN(ShellDesktopControllerTest); 56 DISALLOW_COPY_AND_ASSIGN(ShellDesktopControllerAuraTest);
57 }; 57 };
58 58
59 #if defined(OS_CHROMEOS) 59 #if defined(OS_CHROMEOS)
60 // Tests that a shutdown request is sent to the power manager when the power 60 // Tests that a shutdown request is sent to the power manager when the power
61 // button is pressed. 61 // button is pressed.
62 TEST_F(ShellDesktopControllerTest, PowerButton) { 62 TEST_F(ShellDesktopControllerAuraTest, PowerButton) {
63 // Ignore button releases. 63 // Ignore button releases.
64 power_manager_client_->SendPowerButtonEvent(false /* down */, 64 power_manager_client_->SendPowerButtonEvent(false /* down */,
65 base::TimeTicks()); 65 base::TimeTicks());
66 EXPECT_EQ(0, power_manager_client_->num_request_shutdown_calls()); 66 EXPECT_EQ(0, power_manager_client_->num_request_shutdown_calls());
67 67
68 // A button press should trigger a shutdown request. 68 // A button press should trigger a shutdown request.
69 power_manager_client_->SendPowerButtonEvent(true /* down */, 69 power_manager_client_->SendPowerButtonEvent(true /* down */,
70 base::TimeTicks()); 70 base::TimeTicks());
71 EXPECT_EQ(1, power_manager_client_->num_request_shutdown_calls()); 71 EXPECT_EQ(1, power_manager_client_->num_request_shutdown_calls());
72 } 72 }
73 #endif 73 #endif
74 74
75 } // namespace extensions 75 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698