Bluetooth Extension API: Add a discovery API

chrome/browser/chromeos/extensions/bluetooth_event_router.cc

 // Copyright (c) 2012 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/chromeos/extensions/bluetooth_event_router.h"
 
 #include <map>
 
 #include "base/json/json_writer.h"
 #include "base/memory/ref_counted.h"
+#include "base/utf_string_conversions.h"
 #include "chrome/browser/chromeos/bluetooth/bluetooth_adapter.h"
+#include "chrome/browser/chromeos/bluetooth/bluetooth_device.h"
 #include "chrome/browser/chromeos/bluetooth/bluetooth_socket.h"
 #include "chrome/browser/extensions/extension_event_names.h"
 #include "chrome/browser/extensions/extension_event_router.h"
+#include "chrome/common/extensions/api/experimental_bluetooth.h"
 
 namespace chromeos {
 
 ExtensionBluetoothEventRouter::ExtensionBluetoothEventRouter(Profile* profile)
     : profile_(profile),
       adapter_(chromeos::BluetoothAdapter::CreateDefaultAdapter()),
       next_socket_id_(1) {
   DCHECK(profile_);
   DCHECK(adapter_.get());
   adapter_->AddObserver(this);
@@ skipping 27 matching lines @@
 }
 
 scoped_refptr<BluetoothSocket> ExtensionBluetoothEventRouter::GetSocket(
     int id) {
   SocketMap::iterator socket_entry = socket_map_.find(id);
   if (socket_entry == socket_map_.end())
     return NULL;
   return socket_entry->second;
 }
 
+void ExtensionBluetoothEventRouter::SetSendDiscoveryEvents(bool should_send) {
+  send_discovery_events_ = should_send;
+}
+
 void ExtensionBluetoothEventRouter::AdapterPresentChanged(
     chromeos::BluetoothAdapter* adapter, bool present) {
   DCHECK(adapter == adapter_.get());
-  DispatchEvent(extension_event_names::kBluetoothOnAvailabilityChanged,
+  DispatchBooleanValueEvent(
+      extension_event_names::kBluetoothOnAvailabilityChanged,
       present);
 }
 
 void ExtensionBluetoothEventRouter::AdapterPoweredChanged(
     chromeos::BluetoothAdapter* adapter, bool has_power) {
   DCHECK(adapter == adapter_.get());
-  DispatchEvent(extension_event_names::kBluetoothOnPowerChanged, has_power);
+  DispatchBooleanValueEvent(
+      extension_event_names::kBluetoothOnPowerChanged,
+      has_power);
 }
 
-void ExtensionBluetoothEventRouter::DispatchEvent(
+void ExtensionBluetoothEventRouter::DeviceAdded(
+    chromeos::BluetoothAdapter* adapter, chromeos::BluetoothDevice* device) {
+  if (!send_discovery_events_)
+    return;
+
+  DCHECK(adapter == adapter_.get());
+
+  extensions::api::experimental_bluetooth::Device extension_device;
+  extension_device.name = UTF16ToUTF8(device->GetName());
+  extension_device.address = device->address();
+
+  ListValue args;
+  args.Append(extension_device.ToValue().release());
+  std::string json_args;
+  base::JSONWriter::Write(&args, &json_args);
+
+  profile_->GetExtensionEventRouter()->DispatchEventToRenderers(

[asargent_no_longer_on_chrome, 2012/06/14 18:23:55]

If I have 2 platform apps installed that both have added a listener for this
event, and only one of them calls startDiscovery, does this mean that both apps
will start getting their listener functions fired? If so, for the one that
hasn't called startDiscovery that seems kind of strange.

We also have the DispatchEventToExtension method which might help you avoid that
problem, but then you'll probably need to add bookkeeping for which Extension
objects are currently discovering vs. not.

+      extension_event_names::kBluetoothOnDeviceDiscovered,
+      json_args,
+      NULL,
+      GURL());
+}
+
+void ExtensionBluetoothEventRouter::DispatchBooleanValueEvent(
     const char* event_name, bool value) {
   ListValue args;
   args.Append(Value::CreateBooleanValue(value));
   std::string json_args;
   base::JSONWriter::Write(&args, &json_args);
 
   profile_->GetExtensionEventRouter()->DispatchEventToRenderers(
       event_name, json_args, NULL, GURL());
 }
 
 }  // namespace chromeos