API Bindings for GATT server functionality.

chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc

Index: chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
diff --git a/chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc b/chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
index 8328f82cc883d6be535276e0460c8883534d1f20..e01e0e3805c0a29e098a3308cdcf6797af2b4e44 100644
--- a/chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
+++ b/chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
@@ -6,20 +6,22 @@
 
 #include <stdint.h>
 #include <algorithm>
-#include <utility>
+#include <iterator>
+#include <vector>
 
 #include "base/bind.h"
+#include "base/callback.h"
+#include "base/callback_forward.h"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
-#include "base/strings/stringprintf.h"
-#include "build/build_config.h"
-#include "chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_api_advertisement.h"
+#include "base/logging.h"
+#include "base/values.h"
 #include "chrome/browser/extensions/api/bluetooth_low_energy/utils.h"
 #include "chrome/common/extensions/api/bluetooth_low_energy.h"
 #include "content/public/browser/browser_thread.h"
-#include "extensions/browser/event_router.h"
+#include "device/bluetooth/bluetooth_adapter.h"
 #include "extensions/common/api/bluetooth/bluetooth_manifest_data.h"
-#include "extensions/common/permissions/permissions_data.h"
+#include "extensions/common/extension.h"
 #include "extensions/common/switches.h"
 
 #if defined(OS_CHROMEOS)
@@ -127,7 +129,8 @@ extensions::BluetoothLowEnergyEventRouter* GetEventRouter(
   return extensions::BluetoothLowEnergyAPI::Get(context)->event_router();
 }
 
-void DoWorkCallback(const base::Callback<bool()>& callback) {
+template <typename T>
+void DoWorkCallback(const base::Callback<T()>& callback) {
   DCHECK(!callback.is_null());
   callback.Run();
 }
@@ -189,13 +192,13 @@ void BluetoothLowEnergyAPI::Shutdown() {
 
 namespace api {
 
-BluetoothLowEnergyExtensionFunction::BluetoothLowEnergyExtensionFunction() {
-}
+BluetoothLowEnergyExtensionFunctionDeprecated::
+    BluetoothLowEnergyExtensionFunctionDeprecated() {}
 
-BluetoothLowEnergyExtensionFunction::~BluetoothLowEnergyExtensionFunction() {
-}
+BluetoothLowEnergyExtensionFunctionDeprecated::
+    ~BluetoothLowEnergyExtensionFunctionDeprecated() {}
 
-bool BluetoothLowEnergyExtensionFunction::RunAsync() {
+bool BluetoothLowEnergyExtensionFunctionDeprecated::RunAsync() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   if (!BluetoothManifestData::CheckLowEnergyPermitted(extension())) {
@@ -212,8 +215,9 @@ bool BluetoothLowEnergyExtensionFunction::RunAsync() {
 
   // It is safe to pass |this| here as ExtensionFunction is refcounted.
   if (!event_router->InitializeAdapterAndInvokeCallback(base::Bind(
-          &DoWorkCallback,
-          base::Bind(&BluetoothLowEnergyExtensionFunction::DoWork, this)))) {
+          &DoWorkCallback<bool>,
+          base::Bind(&BluetoothLowEnergyExtensionFunctionDeprecated::DoWork,
+                     this)))) {
     SetError(kErrorAdapterNotInitialized);
     return false;
   }
@@ -221,6 +225,33 @@ bool BluetoothLowEnergyExtensionFunction::RunAsync() {
   return true;
 }
 
+BluetoothLowEnergyExtensionFunction::BluetoothLowEnergyExtensionFunction() {}
+
+BluetoothLowEnergyExtensionFunction::~BluetoothLowEnergyExtensionFunction() {}
+
+ExtensionFunction::ResponseAction BluetoothLowEnergyExtensionFunction::Run() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  if (!BluetoothManifestData::CheckLowEnergyPermitted(extension())) {
+    return RespondNow(Error(kErrorPermissionDenied));
+  }
+
+  BluetoothLowEnergyEventRouter* event_router =
+      GetEventRouter(browser_context());
+  if (!event_router->IsBluetoothSupported()) {
+    return RespondNow(Error(kErrorPlatformNotSupported));
+  }
+
+  // It is safe to pass |this| here as ExtensionFunction is refcounted.
+  if (!event_router->InitializeAdapterAndInvokeCallback(base::Bind(
+          &DoWorkCallback<void>,
+          base::Bind(&BluetoothLowEnergyExtensionFunction::DoWork, this)))) {
+    return RespondNow(Error(kErrorAdapterNotInitialized));
+  }
+
+  return RespondLater();
+}
+
 bool BluetoothLowEnergyConnectFunction::DoWork() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
@@ -874,7 +905,7 @@ void BluetoothLowEnergyAdvertisementFunction::RemoveAdvertisement(
 
 bool BluetoothLowEnergyAdvertisementFunction::RunAsync() {
   Initialize();
-  return BluetoothLowEnergyExtensionFunction::RunAsync();
+  return BluetoothLowEnergyExtensionFunctionDeprecated::RunAsync();
 }
 
 void BluetoothLowEnergyAdvertisementFunction::Initialize() {
@@ -1065,5 +1096,45 @@ void BluetoothLowEnergyUnregisterAdvertisementFunction::ErrorCallback(
   SendResponse(false);
 }
 
+void BluetoothLowEnergyCreateServiceFunction::DoWork() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+// Using apibtle::CreateService::Results::Create causes a link error on
+// Windows for some reason, despite it being almost identical to its
+// characteristic and descriptor counterparts. Since there is no plan to
+// provide this API on Windows, ifdef'ing this out is fine.
+#if !defined(OS_WIN)
+  Respond(ArgumentList(apibtle::CreateService::Results::Create(std::string())));

[Devlin, 2016/04/29 17:13:13]

What's the error?

This isn't quite fine, because then the extension function never responds - it's
just left hanging.  Also, who says this isn't on Windows?  If it's not, why is
it being compiled?

[rkc, 2016/04/29 20:15:58]

Added to the permissions, this function will only be available on
ChromeOS/Linux.

+#endif
+}
+
+void BluetoothLowEnergyCreateCharacteristicFunction::DoWork() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  Respond(ArgumentList(
+      apibtle::CreateCharacteristic::Results::Create(std::string())));
+}
+
+void BluetoothLowEnergyCreateDescriptorFunction::DoWork() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  Respond(
+      ArgumentList(apibtle::CreateDescriptor::Results::Create(std::string())));
+}
+
+void BluetoothLowEnergyRegisterServiceFunction::DoWork() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  Respond(ArgumentList(apibtle::RegisterService::Results::Create(
+      apibtle::SERVICE_RESULT_SUCCESS)));
+}
+
+void BluetoothLowEnergyUnregisterServiceFunction::DoWork() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  Respond(ArgumentList(apibtle::UnregisterService::Results::Create(
+      apibtle::SERVICE_RESULT_SUCCESS)));
+}
+
+void BluetoothLowEnergySendRequestResponseFunction::DoWork() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  Respond(NoArguments());
+}
+
 }  // namespace api
 }  // namespace extensions