Move gamepad_service out of content/ and into device/

device/gamepad/gamepad_service_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "content/browser/gamepad/gamepad_service.h"
+#include "device/gamepad/gamepad_service.h"
 
 #include <string.h>
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/run_loop.h"
-#include "content/public/test/test_browser_thread_bundle.h"
 #include "device/gamepad/gamepad_consumer.h"
 #include "device/gamepad/gamepad_test_helpers.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-namespace content {
+namespace device {
 
 namespace {
 static const int kNumberOfGamepads = blink::WebGamepads::itemsLengthCap;
 }
 
 using blink::WebGamepads;
 
 class ConnectionListener : public device::GamepadConsumer {
  public:
-  ConnectionListener() {
-    ClearCounters();
-  }
+  ConnectionListener() { ClearCounters(); }
 
   void OnGamepadConnected(unsigned index,
                           const blink::WebGamepad& gamepad) override {
     connected_counter_++;
   }
   void OnGamepadDisconnected(unsigned index,
                              const blink::WebGamepad& gamepad) override {
     disconnected_counter_++;
   }
 
@@ skipping 20 matching lines @@
 
   int GetConnectedCounter() const {
     return connection_listener_->connected_counter();
   }
   int GetDisconnectedCounter() const {
     return connection_listener_->disconnected_counter();
   }
 
   void SetUp() override;
 
+  void TearDown() override { base::RunLoop().RunUntilIdle(); }

[blundell, 2016/12/09 13:35:22]

Why is this necessary?

[ke.he, 2016/12/09 15:15:23]

not necessary, removed.

+
  private:
+  base::MessageLoop message_loop_;
   device::MockGamepadDataFetcher* fetcher_;
   GamepadService* service_;
   std::unique_ptr<ConnectionListener> connection_listener_;
-  TestBrowserThreadBundle browser_thread_;
   WebGamepads test_data_;
 
   DISALLOW_COPY_AND_ASSIGN(GamepadServiceTest);
 };
 
-GamepadServiceTest::GamepadServiceTest()
-    : browser_thread_(TestBrowserThreadBundle::IO_MAINLOOP) {
+GamepadServiceTest::GamepadServiceTest() {
   memset(&test_data_, 0, sizeof(test_data_));
 
   // Set it so that we have user gesture.
   test_data_.items[0].buttonsLength = 1;
   test_data_.items[0].buttons[0].value = 1.f;
   test_data_.items[0].buttons[0].pressed = true;
 }
 
 GamepadServiceTest::~GamepadServiceTest() {
   delete service_;
 }
 
 void GamepadServiceTest::SetUp() {
   fetcher_ = new device::MockGamepadDataFetcher(test_data_);
-  service_ = new GamepadService(
-      std::unique_ptr<device::GamepadDataFetcher>(fetcher_));
+  service_ =
+      new GamepadService(std::unique_ptr<device::GamepadDataFetcher>(fetcher_));
   connection_listener_.reset((new ConnectionListener));
   service_->SetSanitizationEnabled(false);
   service_->ConsumerBecameActive(connection_listener_.get());
 }
 
 void GamepadServiceTest::SetPadsConnected(bool connected) {
   for (int i = 0; i < kNumberOfGamepads; ++i) {
     test_data_.items[i].connected = connected;
   }
   fetcher_->SetTestData(test_data_);
@@ skipping 18 matching lines @@
   SetPadsConnected(false);
   WaitForData();
   EXPECT_EQ(0, GetConnectedCounter());
   EXPECT_EQ(kNumberOfGamepads, GetDisconnectedCounter());
 
   WaitForData();
   EXPECT_EQ(0, GetConnectedCounter());
   EXPECT_EQ(0, GetDisconnectedCounter());
 }
 
-}  // namespace content
+}  // namespace device