bluetooth: Allow updates on chooser items.

content/public/browser/bluetooth_chooser.h

Index: content/public/browser/bluetooth_chooser.h
diff --git a/content/public/browser/bluetooth_chooser.h b/content/public/browser/bluetooth_chooser.h
index aec3c2f19415c7606f2c6fe578d36aca43334318..9b55a8a4896a8ce92b160133f7db959d19dadeea 100644
--- a/content/public/browser/bluetooth_chooser.h
+++ b/content/public/browser/bluetooth_chooser.h
@@ -68,9 +68,10 @@ class CONTENT_EXPORT BluetoothChooser {
   enum class DiscoveryState { FAILED_TO_START, DISCOVERING, IDLE };
   virtual void ShowDiscoveryState(DiscoveryState state) {}
 
-  // Shows a new device in the chooser.
-  virtual void AddDevice(const std::string& device_id,
-                         const base::string16& device_name) {}
+  // Adds a new device to the chooser or updates the information of an existing
+  // device.
+  virtual void AddOrUpdateDevice(const std::string& device_id,

[juncai, 2016/08/04 21:52:25]

Does it make sense to have a separate function:
UpdateDevice()
instead of doing these two things in one function? This probably will also make
testing easier since we can test them separately.

[ortuno, 2016/08/05 02:16:09]

If we were to separate these two functions, chooser clients would have to keep
track of which devices they have already added which means yet another set of
device_ids. We could expose a HasDevice() function so that clients don't have to
keep track, but that would mean two extra functions HasDevice() and
UpdateDevice() that we have to implement all the way down to java.

[juncai, 2016/08/05 20:44:30]

Acknowledged.

+                                 const base::string16& device_name) {}
 
   // Tells the chooser that a device is no longer available. The chooser should
   // not call DeviceSelected() for a device that's been removed.