[Bootstrapping] Introduce an accelerator to put a ChromeOS device into Slave OOBE process.

chrome/browser/chromeos/login/bluetooth_host_pairing_browsertest.cc

 // Copyright 2016 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 "base/memory/ref_counted.h"
 #include "base/run_loop.h"
 #include "chrome/browser/chromeos/login/test/oobe_base_test.h"
 #include "chrome/browser/chromeos/login/test/oobe_screen_waiter.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "components/pairing/bluetooth_host_pairing_controller.h"
@@ skipping 27 matching lines @@
 
  private:
   bool finished_ = false;
   std::unique_ptr<base::RunLoop> run_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(TestDelegate);
 };
 
 }  // namespace
 
-class BluetoothHostPairingTest : public OobeBaseTest {
+// This is the class to simulate the OOBE process for devices that don't have
+// sufficient input, i.e., the first screen of OOBE is the HID detection screen.
+// The device will put itself in Bluetooth discoverable mode.
+class BluetoothHostPairingTestNoInput : public OobeBaseTest {

[achuithb, 2016/10/05 22:24:10]

Isn't BluetoothHostPairingNoInputTest better? I think we want Test to be the
last word

[xdai1, 2016/10/05 23:57:29]

Done.

  public:
   using InputDeviceInfo = device::InputServiceLinux::InputDeviceInfo;
 
-  BluetoothHostPairingTest() {
+  BluetoothHostPairingTestNoInput() {
     InputServiceProxy::SetThreadIdForTesting(content::BrowserThread::UI);
     input_service_linux_.reset(new device::FakeInputServiceLinux);
     device::InputServiceLinux::SetForTesting(input_service_linux_.get());
-
-    AddUsbMouse();
-    AddUsbKeyboard();
   }
-  ~BluetoothHostPairingTest() override {}
+  ~BluetoothHostPairingTestNoInput() override {}
 
   // OobeBaseTest override:
   void SetUpOnMainThread() override {
     OobeBaseTest::SetUpOnMainThread();
     delegate_.reset(new TestDelegate);
-    controller()->SetDelegateForTesting(delegate_.get());
-    bluetooth_adapter_ = controller()->GetAdapterForTesting();
+    if (controller()) {
+      controller()->SetDelegateForTesting(delegate_.get());
+      bluetooth_adapter_ = controller()->GetAdapterForTesting();
+    }
   }
 
   pairing_chromeos::BluetoothHostPairingController* controller() {
     WizardController* wizard_controller =

[achuithb, 2016/10/05 22:24:10]

You could just get rid of this and inlign below.

[xdai1, 2016/10/05 23:57:29]

Done.

         WizardController::default_controller();
-    return wizard_controller->GetSharkConnectionListenerForTesting()
-        ->GetControllerForTesting();
+    pairing_chromeos::SharkConnectionListener* shark_listener =
+        wizard_controller->GetSharkConnectionListenerForTesting();
+    return shark_listener ? shark_listener->GetControllerForTesting() : nullptr;
   }
 
   device::BluetoothAdapter* bluetooth_adapter() {
     return bluetooth_adapter_.get();
   }
 
   TestDelegate* delegate() { return delegate_.get(); }
 
   void AddUsbMouse() {
     InputDeviceInfo mouse;
@@ skipping 20 matching lines @@
     mouse.type = InputDeviceInfo::TYPE_BLUETOOTH;
     mouse.is_mouse = true;
     input_service_linux_->AddDeviceForTesting(mouse);
   }
 
  private:
   std::unique_ptr<device::FakeInputServiceLinux> input_service_linux_;
   scoped_refptr<device::BluetoothAdapter> bluetooth_adapter_;
   std::unique_ptr<TestDelegate> delegate_;
 
-  DISALLOW_COPY_AND_ASSIGN(BluetoothHostPairingTest);
+  DISALLOW_COPY_AND_ASSIGN(BluetoothHostPairingTestNoInput);
 };
 
-// Test that in normal user OOBE login flow, if there is no Bluetooth device
-// connected, the Bluetooth adapter should be disabled when OOBE reaches login
-// screen (which means OOBE has been completed).
-IN_PROC_BROWSER_TEST_F(BluetoothHostPairingTest, NoBluetoothDeviceConnected) {
-  OobeScreenWaiter(OobeScreen::SCREEN_OOBE_NETWORK).Wait();
+// Test that in normal user OOBE login flow for devices lacking input devices,
+// if there is no Bluetooth device connected, the Bluetooth adapter should be
+// disabled when OOBE reaches login screen (which means OOBE has been completed)
+IN_PROC_BROWSER_TEST_F(BluetoothHostPairingTestNoInput,
+                       NoBluetoothDeviceConnected) {
+  OobeScreenWaiter(OobeScreen::SCREEN_OOBE_HID_DETECTION).Wait();
   EXPECT_EQ(bluetooth_adapter()->IsPowered(), true);
   WizardController::default_controller()->SkipToLoginForTesting(
       LoginScreenContext());
   OobeScreenWaiter(OobeScreen::SCREEN_GAIA_SIGNIN).Wait();
   delegate()->WaitUntilAdapterReset();
   EXPECT_EQ(bluetooth_adapter()->IsPowered(), false);
 }
 
-// Test that in normal user OOBE login flow, if there is any Bluetooth device
-// connected, the Bluetooth adapter should not be disabled after OOBE completes.
-IN_PROC_BROWSER_TEST_F(BluetoothHostPairingTest, BluetoothDeviceConnected) {
-  OobeScreenWaiter(OobeScreen::SCREEN_OOBE_NETWORK).Wait();
+// Test that in normal user OOBE login flow for devices lacking input devices,
+// if there is any Bluetooth device connected, the Bluetooth adapter should not
+// be disabled after OOBE completes.
+IN_PROC_BROWSER_TEST_F(BluetoothHostPairingTestNoInput,
+                       BluetoothDeviceConnected) {
+  OobeScreenWaiter(OobeScreen::SCREEN_OOBE_HID_DETECTION).Wait();
   AddBluetoothMouse();
   EXPECT_EQ(bluetooth_adapter()->IsPowered(), true);
   WizardController::default_controller()->SkipToLoginForTesting(
       LoginScreenContext());
   OobeScreenWaiter(OobeScreen::SCREEN_GAIA_SIGNIN).Wait();
   delegate()->WaitUntilAdapterReset();
   EXPECT_EQ(bluetooth_adapter()->IsPowered(), true);
 }
 
+// This is the class to simulate the OOBE process for devices that have
+// sufficient input, i.e., the first screen of OOBE is the network screen.
+// The device will not put itself in Bluetooth discoverable mode until the user
+// manually trigger it using the proper accelerator.
+class BluetoothHostPairingTestWithInput

[achuithb, 2016/10/05 22:24:10]

Test last

[xdai1, 2016/10/05 23:57:29]

Done.

+    : public BluetoothHostPairingTestNoInput {
+ public:
+  BluetoothHostPairingTestWithInput() {
+    AddUsbMouse();
+    AddUsbKeyboard();
+  }
+  ~BluetoothHostPairingTestWithInput() override {}
+
+  DISALLOW_COPY_AND_ASSIGN(BluetoothHostPairingTestWithInput);

[achuithb, 2016/10/05 22:24:10]

This must be private

[xdai1, 2016/10/05 23:57:29]

Done.

+};
+
+// Test that in normal user OOBE login flow for devices that have input devices,
+// the Bluetooth is disabled by default.
+IN_PROC_BROWSER_TEST_F(BluetoothHostPairingTestWithInput,
+                       BluetoothDisableByDefault) {
+  OobeScreenWaiter(OobeScreen::SCREEN_OOBE_NETWORK).Wait();
+  EXPECT_FALSE(controller());
+  EXPECT_FALSE(bluetooth_adapter());
+}
+
 }  // namespace chromeos