Implement chrome.bluetoothLowEnergy.resetAdvertising().

device/bluetooth/dbus/bluetooth_le_advertising_manager_client.h

 // Copyright 2015 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.
 
 #ifndef DEVICE_BLUETOOTH_DBUS_BLUETOOTH_LE_ADVERTISING_MANAGER_CLIENT_H_
 #define DEVICE_BLUETOOTH_DBUS_BLUETOOTH_LE_ADVERTISING_MANAGER_CLIENT_H_
 
 #include <cstdint>
 #include <memory>
 #include <string>
@@ skipping 58 matching lines @@
       const ErrorCallback& error_callback) = 0;
 
   // Set's the advertising interval.
   virtual void SetAdvertisingInterval(
       const dbus::ObjectPath& manager_object_path,
       uint16_t min_interval_ms,
       uint16_t max_interval_ms,
       const base::Closure& callback,
       const ErrorCallback& error_callback) = 0;
 
+  // Resets advertising.
+  virtual void ResetAdvertising(const dbus::ObjectPath& manager_object_path,

[rkc, 2017/06/13 19:02:31]

This method isn't getting tested from anywhere. Can we add a unit tests for this
that goes all the way to the fake D-Bus client and performs a fake reset?

[Sonny Sasaka, 2017/06/15 23:19:12]

Actually I wrote a browser test that triggers ResetAdvertising of the fake dbus
client:
chrome/test/data/extensions/api_test/bluetooth_low_energy/reset_all_advertisements/runtest.js

Or did you mean to write another test which is not a browser test?

+                                const base::Closure& callback,
+                                const ErrorCallback& error_callback) = 0;
+
   // Creates the instance.
   static BluetoothLEAdvertisingManagerClient* Create();
 
   // Constants used to indicate exceptional error conditions.
   static const char kNoResponseError[];
 
  protected:
   BluetoothLEAdvertisingManagerClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BluetoothLEAdvertisingManagerClient);
 };
 
 }  // namespace bluez
 
 #endif  // DEVICE_BLUETOOTH_DBUS_BLUETOOTH_LE_ADVERTISING_MANAGER_CLIENT_H_