bluetooth: Move FactoryWrapper to device and expose a function for testing in chooser controller

content/browser/bluetooth/bluetooth_device_chooser_controller.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 "content/browser/bluetooth/bluetooth_device_chooser_controller.h"
 
 #include <set>
 #include <string>
 #include <unordered_set>
 
@@ skipping 10 matching lines @@
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "device/bluetooth/bluetooth_adapter.h"
 #include "device/bluetooth/bluetooth_common.h"
 #include "device/bluetooth/bluetooth_discovery_session.h"
 
 using device::BluetoothUUID;
 
 namespace content {
 
+bool BluetoothDeviceChooserController::use_test_scan_duration_ = false;
+
 namespace {
 constexpr size_t kMaxLengthForDeviceName =
     29;  // max length of device name in filter.
 
+// The duration of a Bluetooth Scan in seconds.
+constexpr int kScanDuration = 10;
+constexpr int kTestScanDuration = 0;
+
 void LogRequestDeviceOptions(
     const blink::mojom::WebBluetoothRequestDeviceOptionsPtr& options) {
   VLOG(1) << "requestDevice called with the following filters: ";
   int i = 0;
   for (const auto& filter : options->filters) {
     VLOG(1) << "Filter #" << ++i;
     if (!filter->name.is_null())
       VLOG(1) << "Name: " << filter->name;
 
     if (!filter->name_prefix.is_null())
@@ skipping 129 matching lines @@
   }
   NOTREACHED();
   return UMARequestDeviceOutcome::SUCCESS;
 }
 
 }  // namespace
 
 BluetoothDeviceChooserController::BluetoothDeviceChooserController(
     WebBluetoothServiceImpl* web_bluetooth_service,
     RenderFrameHost* render_frame_host,
-    device::BluetoothAdapter* adapter,
-    base::TimeDelta scan_duration)
+    device::BluetoothAdapter* adapter)
     : adapter_(adapter),
       web_bluetooth_service_(web_bluetooth_service),
       render_frame_host_(render_frame_host),
       web_contents_(WebContents::FromRenderFrameHost(render_frame_host_)),
       discovery_session_timer_(
           FROM_HERE,
           // TODO(jyasskin): Add a way for tests to control the dialog
           // directly, and change this to a reasonable discovery timeout.
-          scan_duration,
+          base::TimeDelta::FromSeconds(
+              use_test_scan_duration_ ? kTestScanDuration : kScanDuration),
           base::Bind(&BluetoothDeviceChooserController::StopDeviceDiscovery,
                      // base::Timer guarantees it won't call back after its
                      // destructor starts.
                      base::Unretained(this)),
           /*is_repeating=*/false),
       weak_ptr_factory_(this) {
   CHECK(adapter_);
 }
 
 BluetoothDeviceChooserController::~BluetoothDeviceChooserController() {
@@ skipping 144 matching lines @@
       OnBluetoothChooserEvent(BluetoothChooser::Event::RESCAN,
                               "" /* device_address */);
     }
   }
 
   if (!powered) {
     discovery_session_timer_.Stop();
   }
 }
 
+void BluetoothDeviceChooserController::SetTestScanDurationForTesting() {
+  BluetoothDeviceChooserController::use_test_scan_duration_ = true;
+}
+
 void BluetoothDeviceChooserController::PopulateFoundDevices() {
   VLOG(1) << "Populating " << adapter_->GetDevices().size()
           << " devices in chooser.";
   for (const device::BluetoothDevice* device : adapter_->GetDevices()) {
     AddFilteredDevice(*device);
   }
 }
 
 void BluetoothDeviceChooserController::StartDeviceDiscovery() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
@@ skipping 100 matching lines @@
 
 void BluetoothDeviceChooserController::PostErrorCallback(
     blink::mojom::WebBluetoothError error) {
   if (!base::ThreadTaskRunnerHandle::Get()->PostTask(
           FROM_HERE, base::Bind(error_callback_, error))) {
     LOG(WARNING) << "No TaskRunner.";
   }
 }
 
 }  // namespace content