Replace Chrome IPC with Mojo IPC for querying BatteryStatus service

device/battery/battery_status_service_unittest.cc

Index: device/battery/battery_status_service_unittest.cc
diff --git a/content/browser/battery_status/battery_status_service_unittest.cc b/device/battery/battery_status_service_unittest.cc
similarity index 78%
rename from content/browser/battery_status/battery_status_service_unittest.cc
rename to device/battery/battery_status_service_unittest.cc
index 54d4833c52bd9855b75f544eae640b7aee32f3c1..a072630e3f4248a39cf7be70d49447ec457f96a9 100644
--- a/content/browser/battery_status/battery_status_service_unittest.cc
+++ b/device/battery/battery_status_service_unittest.cc
@@ -2,14 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "content/browser/battery_status/battery_status_service.h"
-
 #include "base/bind.h"
+#include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
-#include "content/browser/battery_status/battery_status_manager.h"
-#include "content/public/test/test_browser_thread_bundle.h"
+#include "device/battery/battery_status_manager.h"
+#include "device/battery/battery_status_service.h"
+#include "testing/gtest/include/gtest/gtest.h"
 
-namespace content {
+namespace device {
 
 namespace {
 
@@ -30,7 +30,7 @@ class FakeBatteryManager : public BatteryStatusManager {
     stop_invoked_count_++;
   }
 
-  void InvokeUpdateCallback(const blink::WebBatteryStatus& status) {
+  void InvokeUpdateCallback(const BatteryStatus& status) {
     callback_.Run(status);
   }
 
@@ -48,9 +48,8 @@ class FakeBatteryManager : public BatteryStatusManager {
 class BatteryStatusServiceTest : public testing::Test {
  public:
     BatteryStatusServiceTest()
-        : thread_bundle_(content::TestBrowserThreadBundle::IO_MAINLOOP),
-          battery_service_(0),
-          battery_manager_(0),
+        : battery_service_(NULL),
+          battery_manager_(NULL),
           callback1_invoked_count_(0),
           callback2_invoked_count_(0) {
     }
@@ -65,14 +64,21 @@ class BatteryStatusServiceTest : public testing::Test {
     callback2_ = base::Bind(&BatteryStatusServiceTest::Callback2,
                             base::Unretained(this));
     battery_service_ = BatteryStatusService::GetInstance();
-    battery_manager_ = new FakeBatteryManager(
-        battery_service_->GetUpdateCallbackForTesting());
-    battery_service_->SetBatteryManagerForTesting(battery_manager_);
+
+    // We keep a raw pointer to the FakeBatteryManager, which we expect to
+    // remain valid for the lifetime of the BatteryStatusService.
+    scoped_ptr<FakeBatteryManager> battery_manager(new FakeBatteryManager(
+        battery_service_->GetUpdateCallbackForTesting()));
+    battery_manager_ = battery_manager.get();
+
+    battery_service_->SetBatteryManagerForTesting(
+        battery_manager.PassAs<BatteryStatusManager>());
   }
 
   virtual void TearDown() override {
     base::RunLoop().RunUntilIdle();
-    battery_service_->SetBatteryManagerForTesting(0);
+    battery_service_->SetBatteryManagerForTesting(
+        scoped_ptr<BatteryStatusManager>());
   }
 
   FakeBatteryManager* battery_manager() {
@@ -92,7 +98,7 @@ class BatteryStatusServiceTest : public testing::Test {
     return callback2_invoked_count_;
   }
 
-  const blink::WebBatteryStatus& battery_status() const {
+  const BatteryStatus& battery_status() const {
     return battery_status_;
   }
 
@@ -105,24 +111,24 @@ class BatteryStatusServiceTest : public testing::Test {
   }
 
  private:
-  void Callback1(const blink::WebBatteryStatus& status) {
+  void Callback1(const BatteryStatus& status) {
     callback1_invoked_count_++;
     battery_status_ = status;
   }
 
-  void Callback2(const blink::WebBatteryStatus& status) {
+  void Callback2(const BatteryStatus& status) {
     callback2_invoked_count_++;
     battery_status_ = status;
   }
 
-  content::TestBrowserThreadBundle thread_bundle_;
+  base::MessageLoop message_loop_;
   BatteryStatusService* battery_service_;
   FakeBatteryManager* battery_manager_;
   BatteryStatusService::BatteryUpdateCallback callback1_;
   BatteryStatusService::BatteryUpdateCallback callback2_;
   int callback1_invoked_count_;
   int callback2_invoked_count_;
-  blink::WebBatteryStatus battery_status_;
+  BatteryStatus battery_status_;
 
   DISALLOW_COPY_AND_ASSIGN(BatteryStatusServiceTest);
 };
@@ -138,7 +144,7 @@ TEST_F(BatteryStatusServiceTest, AddFirstCallback) {
 
 TEST_F(BatteryStatusServiceTest, AddCallbackAfterUpdate) {
   scoped_ptr<BatterySubscription> subscription1 = AddCallback(callback1());
-  blink::WebBatteryStatus status;
+  BatteryStatus status;
   battery_manager()->InvokeUpdateCallback(status);
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(1, callback1_invoked_count());
@@ -153,10 +159,10 @@ TEST_F(BatteryStatusServiceTest, TwoCallbacksUpdate) {
   scoped_ptr<BatterySubscription> subscription1 = AddCallback(callback1());
   scoped_ptr<BatterySubscription> subscription2 = AddCallback(callback2());
 
-  blink::WebBatteryStatus status;
+  BatteryStatus status;
   status.charging = true;
-  status.chargingTime = 100;
-  status.dischargingTime = 200;
+  status.charging_time = 100;
+  status.discharging_time = 200;
   status.level = 0.5;
   battery_manager()->InvokeUpdateCallback(status);
   base::RunLoop().RunUntilIdle();
@@ -164,8 +170,8 @@ TEST_F(BatteryStatusServiceTest, TwoCallbacksUpdate) {
   EXPECT_EQ(1, callback1_invoked_count());
   EXPECT_EQ(1, callback2_invoked_count());
   EXPECT_EQ(status.charging, battery_status().charging);
-  EXPECT_EQ(status.chargingTime, battery_status().chargingTime);
-  EXPECT_EQ(status.dischargingTime, battery_status().dischargingTime);
+  EXPECT_EQ(status.charging_time, battery_status().charging_time);
+  EXPECT_EQ(status.discharging_time, battery_status().discharging_time);
   EXPECT_EQ(status.level, battery_status().level);
 }
 
@@ -173,7 +179,7 @@ TEST_F(BatteryStatusServiceTest, RemoveOneCallback) {
   scoped_ptr<BatterySubscription> subscription1 = AddCallback(callback1());
   scoped_ptr<BatterySubscription> subscription2 = AddCallback(callback2());
 
-  blink::WebBatteryStatus status;
+  BatteryStatus status;
   battery_manager()->InvokeUpdateCallback(status);
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(1, callback1_invoked_count());
@@ -188,4 +194,4 @@ TEST_F(BatteryStatusServiceTest, RemoveOneCallback) {
 
 }  // namespace
 
-}  // namespace content
+}  // namespace device