Add the chrome.bluetoothLowEnergy.setAdvertisingInterval API.

chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.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 "chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <iterator>
 #include <utility>
@@ skipping 69 matching lines @@
     "The characteristic does not have the indicate property set.";
 const char kErrorServiceNotRegistered[] =
     "The characteristic is not owned by a service that is registered.";
 const char kErrorUnknownNotificationError[] =
     "An unknown notification error occured.";
 
 const char kStatusAdvertisementAlreadyExists[] =
     "An advertisement is already advertising";
 const char kStatusAdvertisementDoesNotExist[] =
     "This advertisement does not exist";
+const char kStatusInvalidAdvertisingInterval[] =
+    "Invalid advertising interval specified.";
 
 // Returns the correct error string based on error status |status|. This is used
 // to set the value of |chrome.runtime.lastError.message| and should not be
 // passed |BluetoothLowEnergyEventRouter::kStatusSuccess|.
 std::string StatusToString(BluetoothLowEnergyEventRouter::Status status) {
   switch (status) {
     case BluetoothLowEnergyEventRouter::kStatusErrorAlreadyConnected:
       return kErrorAlreadyConnected;
     case BluetoothLowEnergyEventRouter::kStatusErrorAlreadyNotifying:
       return kErrorAlreadyNotifying;
@@ skipping 1191 matching lines @@
     case device::BluetoothAdvertisement::ErrorCode::
         ERROR_ADVERTISEMENT_DOES_NOT_EXIST:
       SetError(kStatusAdvertisementDoesNotExist);
       break;
     default:
       SetError(kErrorOperationFailed);
   }
   SendResponse(false);
 }
 
+// SetAdvertisingInterval:
+
+template class BLEPeripheralExtensionFunction<
+    apibtle::SetAdvertisingInterval::Params>;
+
+void BluetoothLowEnergySetAdvertisingIntervalFunction::DoWork() {
+  BluetoothLowEnergyEventRouter* event_router =
+      GetEventRouter(browser_context());
+
+  event_router->adapter()->SetAdvertisingInterval(
+      base::TimeDelta::FromMilliseconds(params_->min_interval),
+      base::TimeDelta::FromMilliseconds(params_->max_interval),
+      base::Bind(
+          &BluetoothLowEnergySetAdvertisingIntervalFunction::SuccessCallback,
+          this),
+      base::Bind(
+          &BluetoothLowEnergySetAdvertisingIntervalFunction::ErrorCallback,
+          this));
+}
+
+void BluetoothLowEnergySetAdvertisingIntervalFunction::SuccessCallback() {
+  Respond(NoArguments());
+}
+
+void BluetoothLowEnergySetAdvertisingIntervalFunction::ErrorCallback(
+    device::BluetoothAdvertisement::ErrorCode status) {
+  switch (status) {
+    case device::BluetoothAdvertisement::ErrorCode::
+        ERROR_INVALID_ADVERTISEMENT_INTERVAL:
+      Respond(Error(kStatusInvalidAdvertisingInterval));
+      return;

[xiyuan, 2016/09/21 16:35:23]

nit: remove since we have "break" ? Or keep this and remove "break" ?

[Rahul Chaturvedi, 2016/09/21 18:54:12]

Done.

+      break;
+    default:
+      Respond(Error(kErrorOperationFailed));
+  }
+}
+
 // createService:
 
 template class BLEPeripheralExtensionFunction<apibtle::CreateService::Params>;
 
 void BluetoothLowEnergyCreateServiceFunction::DoWork() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 // Causes link error on Windows. API will never be on Windows, so #ifdefing.
 // TODO: Ideally this should be handled by our feature system, so that this
 // code doesn't even compile on OSes it isn't being used on, but currently this
 // is not possible.
@@ skipping 177 matching lines @@
                         params_->response.value->end());
   }
   event_router_->HandleRequestResponse(
       extension(), params_->response.request_id, params_->response.is_error,
       uint8_vector);
   Respond(NoArguments());
 }
 
 }  // namespace api
 }  // namespace extensions