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

Side by Side Diff: third_party/WebKit/Source/platform/battery/battery_dispatcher_proxy_unittest.cc

Issue 1538803002: Migrates battery_status from content/renderer/ to WebKit/platform/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added a TODO comment. Created 4 years, 10 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 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 "content/renderer/battery_status/battery_status_dispatcher.h" 5 #include "platform/battery/battery_dispatcher_proxy.h"
6 6
7 #include <utility> 7 #include "platform/battery/battery_status.h"
8 #include "platform/testing/MojoTestHelper.h"
9 #include "testing/gtest/include/gtest/gtest.h"
10 #include "wtf/Noncopyable.h"
11 #include "wtf/OwnPtr.h"
12 #include "wtf/PassOwnPtr.h"
8 13
9 #include "base/macros.h" 14 namespace blink {
10 #include "content/public/test/mock_render_thread.h"
11 #include "content/public/test/test_utils.h"
12 #include "testing/gtest/include/gtest/gtest.h"
13 #include "third_party/WebKit/public/platform/WebBatteryStatusListener.h"
14 15
15 namespace content { 16 class MockBatteryStatusListener : public BatteryDispatcherProxy::Listener {
16 17 WTF_MAKE_NONCOPYABLE(MockBatteryStatusListener);
17 class MockBatteryStatusListener : public blink::WebBatteryStatusListener {
18 public: 18 public:
19 MockBatteryStatusListener() : did_change_battery_status_(false) {} 19 MockBatteryStatusListener() : did_change_battery_status_(false) {}
20 ~MockBatteryStatusListener() override {}
21 20
22 // blink::WebBatteryStatusListener method. 21 // BatteryDispatcherProxy::Listener method.
23 void updateBatteryStatus(const blink::WebBatteryStatus& status) override { 22 void OnUpdateBatteryStatus(const BatteryStatus& status) override {
24 status_ = status; 23 status_ = status;
25 did_change_battery_status_ = true; 24 did_change_battery_status_ = true;
26 } 25 }
27 26
28 const blink::WebBatteryStatus& status() const { return status_; } 27 const BatteryStatus& status() const { return status_; }
29 bool did_change_battery_status() const { return did_change_battery_status_; } 28 bool did_change_battery_status() const { return did_change_battery_status_; }
30 29
31 private: 30 private:
32 bool did_change_battery_status_; 31 bool did_change_battery_status_;
33 blink::WebBatteryStatus status_; 32 BatteryStatus status_;
34
35 DISALLOW_COPY_AND_ASSIGN(MockBatteryStatusListener);
36 }; 33 };
37 34
38 class BatteryStatusDispatcherTest : public testing::Test { 35 class BatteryDispatcherProxyTest : public testing::Test {
39 public: 36 public:
40 void UpdateBatteryStatus(const device::BatteryStatus& status) { 37 void UpdateBatteryStatus(const device::BatteryStatus& status) {
41 device::BatteryStatusPtr status_ptr(device::BatteryStatus::New()); 38 device::BatteryStatusPtr status_ptr(device::BatteryStatus::New());
42 *status_ptr = status; 39 *status_ptr = status;
43 dispatcher_->DidChange(std::move(status_ptr)); 40 dispatcher_->OnDidChange(std::move(status_ptr));
44 } 41 }
45 42
46 const MockBatteryStatusListener& listener() const { 43 const MockBatteryStatusListener& listener() const { return listener_; }
47 return listener_;
48 }
49 44
50 protected: 45 protected:
51 void SetUp() override { 46 void SetUp() override {
52 dispatcher_.reset(new BatteryStatusDispatcher(&listener_)); 47 dispatcher_ = adoptPtr(new BatteryDispatcherProxy(&listener_));
53 } 48 }
54 49
55 private: 50 private:
56 // We need to create a MockRenderThread so RenderThread::Get() doesn't return 51 MojoTestHelper mojo_test_helper_;
57 // null.
58 MockRenderThread render_thread_;
59 MockBatteryStatusListener listener_; 52 MockBatteryStatusListener listener_;
60 scoped_ptr<BatteryStatusDispatcher> dispatcher_; 53 OwnPtr<BatteryDispatcherProxy> dispatcher_;
61 }; 54 };
62 55
63 TEST_F(BatteryStatusDispatcherTest, UpdateListener) { 56 TEST_F(BatteryDispatcherProxyTest, UpdateListener) {
64 // TODO(darin): This test isn't super interesting. It just exercises 57 // TODO(darin): This test isn't super interesting. It just exercises
65 // conversion b/w device::BatteryStatus and blink::WebBatteryStatus. 58 // conversion b/w device::BatteryStatus and blink::BatteryStatus.
66 59
67 device::BatteryStatus status; 60 device::BatteryStatus status;
68 status.charging = true; 61 status.charging = true;
69 status.charging_time = 100; 62 status.charging_time = 100;
70 status.discharging_time = 200; 63 status.discharging_time = 200;
71 status.level = 0.5; 64 status.level = 0.5;
72 65
73 UpdateBatteryStatus(status); 66 UpdateBatteryStatus(status);
74 67
75 const blink::WebBatteryStatus& received_status = listener().status(); 68 const BatteryStatus& received_status = listener().status();
76 EXPECT_TRUE(listener().did_change_battery_status()); 69 EXPECT_TRUE(listener().did_change_battery_status());
77 EXPECT_EQ(status.charging, received_status.charging); 70 EXPECT_EQ(status.charging, received_status.charging());
78 EXPECT_EQ(status.charging_time, received_status.chargingTime); 71 EXPECT_EQ(status.charging_time, received_status.charging_time());
79 EXPECT_EQ(status.discharging_time, received_status.dischargingTime); 72 EXPECT_EQ(status.discharging_time, received_status.discharging_time());
80 EXPECT_EQ(status.level, received_status.level); 73 EXPECT_EQ(status.level, received_status.level());
81 } 74 }
82 75
83 } // namespace content 76 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698