Bluetooth API: add discovering changed events

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"
@@ skipping 65 matching lines @@
 }
 
 void ExtensionBluetoothEventRouter::AdapterPoweredChanged(
     chromeos::BluetoothAdapter* adapter, bool has_power) {
   DCHECK(adapter == adapter_.get());
   DispatchBooleanValueEvent(
       extension_event_names::kBluetoothOnPowerChanged,
       has_power);
 }
 
+void ExtensionBluetoothEventRouter::AdapterDiscoveringChanged(
+    chromeos::BluetoothAdapter* adapter, bool discovering) {
+  DCHECK(adapter == adapter_.get());

[miket_OOO, 2012/07/24 16:58:50]

Sorry I'm late to this design party, but if this must be true, why have the
parameter in the signature at all?

[bryeung, 2012/07/24 17:02:09]

It has to be in the signature because it's an override.

There are cases, in general, where the adapter might not be the same.  But the
extension system only deals with the default adapter, so it should be true for
us.

Thinking about this more, maybe it should be an early NOP return if the adapters
don't match?

+  DispatchBooleanValueEvent(
+      extension_event_names::kBluetoothOnDiscoveringChanged,
+      discovering);
+}
+
 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;
   extensions::api::experimental_bluetooth::BluetoothDeviceToApiDevice(
       *device, &extension_device);
@@ skipping 17 matching lines @@
   std::string json_args;
   base::JSONWriter::Write(&args, &json_args);
 
   // TODO(bryeung): only dispatch the event to interested renderers
   // crbug.com/133179
   profile_->GetExtensionEventRouter()->DispatchEventToRenderers(
       event_name, json_args, NULL, GURL());
 }
 
 }  // namespace chromeos