Add bluetooth.getName API.

chrome/browser/chromeos/bluetooth/bluetooth_adapter.h

Index: chrome/browser/chromeos/bluetooth/bluetooth_adapter.h
diff --git a/chrome/browser/chromeos/bluetooth/bluetooth_adapter.h b/chrome/browser/chromeos/bluetooth/bluetooth_adapter.h
index eb3715df4eb60a5004cb980308611adf7becac85..6b8ff2bc13488216e3ec3bd77c5cd31a75842277 100644
--- a/chrome/browser/chromeos/bluetooth/bluetooth_adapter.h
+++ b/chrome/browser/chromeos/bluetooth/bluetooth_adapter.h
@@ -98,7 +98,10 @@ class BluetoothAdapter : public base::RefCounted<BluetoothAdapter>,
 
   // The address of this adapter.  The address format is "XX:XX:XX:XX:XX:XX",
   // where each XX is a hexadecimal number.
-  const std::string& address() const { return address_; }
+  virtual const std::string& address() const;

[Mihai Parparita -not on Chrome, 2012/08/17 20:30:07]

Nit: lowercase names are for basic getters/setters that can be inlined. Virtual
methods are by definition not inlineable, so this should be called GetAddress()
(and the same for name() below)

[keybuk, 2012/08/17 20:31:36]

Is there any particular reason this method has been made virtual?

For testing can't you create the structure with the right member value - I can't
see a reason you'd need to ever "verify that address() or name() are called" ?

[bryeung, 2012/08/20 23:13:19]

Done.

+
+  // The name of the adapter.
+  virtual const std::string& name() const;
 
   // Indicates whether the adapter is actually present on the system, for
   // the default adapter this indicates whether any adapter is present.
@@ -316,6 +319,9 @@ class BluetoothAdapter : public base::RefCounted<BluetoothAdapter>,
   // Address of the adapter.
   std::string address_;
 
+  // Name of the adapter.
+  std::string name_;
+
   // Tracked adapter state, cached locally so we only send change notifications
   // to observers on a genuine change.
   bool powered_;