bluetooth: FakeBluetoothChooser never timeout scan

Allows FakeBluetoothChooser to set the test scan duration to never
timeout.

BUG=719827

Change-Id: I38c74a1bbb0de026edb7d578b98258941d0c3349
Reviewed-on: https://chromium-review.googlesource.com/961544Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545105}

content/public/test/layouttest_support.h

@@ -153,8 +153,12 @@ gfx::ColorSpace GetTestingColorSpace(const std::string& name);
 void SetDeviceColorSpace(RenderView* render_view,
                          const gfx::ColorSpace& color_space);
 
-// Sets the scan duration to 0.
-void SetTestBluetoothScanDuration();
+// Sets the scan duration to reflect the given setting.
+enum class BluetoothTestScanDurationSetting {
+  kImmediateTimeout,  // Set the scan duration to 0 seconds.
+  kNeverTimeout,  // Set the scan duration to base::TimeDelta::Max() seconds.
+};
+void SetTestBluetoothScanDuration(BluetoothTestScanDurationSetting setting);
 
 // Enables or disables synchronous resize mode. When enabled, all window-sizing
 // machinery is short-circuited inside the renderer. This mode is necessary for

content/shell/browser/layout_test/fake_bluetooth_chooser.cc

@@ -8,15 +8,20 @@
 #include <utility>
 
 #include "content/public/browser/bluetooth_chooser.h"
+#include "content/public/test/layouttest_support.h"
 #include "content/shell/common/layout_test/fake_bluetooth_chooser.mojom.h"
 
 namespace content {
 
-FakeBluetoothChooser::~FakeBluetoothChooser() = default;
+FakeBluetoothChooser::~FakeBluetoothChooser() {
+  SetTestBluetoothScanDuration(
+      BluetoothTestScanDurationSetting::kImmediateTimeout);
+}
 
 // static
 std::unique_ptr<FakeBluetoothChooser> FakeBluetoothChooser::Create(
     mojom::FakeBluetoothChooserRequest request) {
+  SetTestBluetoothScanDuration(BluetoothTestScanDurationSetting::kNeverTimeout);
   return std::unique_ptr<FakeBluetoothChooser>(
       new FakeBluetoothChooser(std::move(request)));
 }

content/shell/browser/layout_test/fake_bluetooth_chooser.h

@@ -28,6 +28,7 @@ namespace content {
 class FakeBluetoothChooser : public mojom::FakeBluetoothChooser,
                              public BluetoothChooser {
  public:
+  // Resets the test scan duration to timeout immediately.
   ~FakeBluetoothChooser() override;
 
   // LayoutTestContentBrowserClient will create an instance of this class when a

content/shell/browser/layout_test/layout_test_bluetooth_fake_adapter_setter_impl.cc

@@ -32,7 +32,8 @@ void LayoutTestBluetoothFakeAdapterSetterImpl::Create(
 void LayoutTestBluetoothFakeAdapterSetterImpl::Set(
     const std::string& adapter_name,
     SetCallback callback) {
-  SetTestBluetoothScanDuration();
+  SetTestBluetoothScanDuration(
+      BluetoothTestScanDurationSetting::kImmediateTimeout);
 
   device::BluetoothAdapterFactoryWrapper::Get().SetBluetoothAdapterForTesting(
       LayoutTestBluetoothAdapterProvider::GetBluetoothAdapter(adapter_name));

content/test/layouttest_support.cc

@@ -507,8 +507,19 @@ void SetDeviceColorSpace(RenderView* render_view,
       ->SetDeviceColorSpaceForTesting(color_space);
 }
 
-void SetTestBluetoothScanDuration() {
-  BluetoothDeviceChooserController::SetTestScanDurationForTesting();
+void SetTestBluetoothScanDuration(BluetoothTestScanDurationSetting setting) {
+  switch (setting) {
+    case BluetoothTestScanDurationSetting::kImmediateTimeout:
+      BluetoothDeviceChooserController::SetTestScanDurationForTesting(
+          BluetoothDeviceChooserController::TestScanDurationSetting::
+              IMMEDIATE_TIMEOUT);
+      break;
+    case BluetoothTestScanDurationSetting::kNeverTimeout:
+      BluetoothDeviceChooserController::SetTestScanDurationForTesting(
+          BluetoothDeviceChooserController::TestScanDurationSetting::
+              NEVER_TIMEOUT);
+      break;
+  }
 }
 
 void UseSynchronousResizeMode(RenderView* render_view, bool enable) {