device/bluetooth:Implement BluetoothMediaEndpointServiceProvider delegate and media-related overrid…

chromeos/dbus/bluetooth_media_transport_client.h

Index: chromeos/dbus/bluetooth_media_transport_client.h
diff --git a/chromeos/dbus/bluetooth_media_transport_client.h b/chromeos/dbus/bluetooth_media_transport_client.h
index 9ed0af23010ee9ab48d820bd49aee7ef46eb0b7a..5237d777e401fe452f068d133f0dcf77c99ae007 100644
--- a/chromeos/dbus/bluetooth_media_transport_client.h
+++ b/chromeos/dbus/bluetooth_media_transport_client.h
@@ -73,6 +73,24 @@ class CHROMEOS_EXPORT BluetoothMediaTransportClient : public DBusClient {
         const std::string& property_name) {}
   };
 
+  // Constants used to indicate exceptional error conditions.

[armansito, 2015/02/10 00:17:01]

All of these should go to service_constants.h. though I'm not sure why you need
to declare all of the Property constants here in the header?

[Miao, 2015/02/10 22:15:21]

Comment added! Right, these will go to service_constants.h eventually, but
before that, I think it might be more centralized to put them here. In this
case, I can ensure that the reference in endpoint service provider will be
updated as well. And it makes sense to me that all property constants are under
transport namespace.

+  static const char kNoResponseError[];
+  static const char kUnexpectedResponse[];
+
+  // Constants used for the names of Media Transport's properties.
+  static const char kDeviceProperty[];
+  static const char kUUIDProperty[];
+  static const char kCodecProperty[];
+  static const char kConfigurationProperty[];
+  static const char kStateProperty[];
+  static const char kDelayProperty[];
+  static const char kVolumeProperty[];
+
+  // All possible states of a valid media transport object.
+  static const char kStateIdle[];
+  static const char kStatePending[];
+  static const char kStateActive[];
+
   ~BluetoothMediaTransportClient() override;
 
   // The ErrorCallback is used by media transport API methods to indicate
@@ -115,10 +133,6 @@ class CHROMEOS_EXPORT BluetoothMediaTransportClient : public DBusClient {
 
   static BluetoothMediaTransportClient* Create();
 
-  // Constants used to indicate exceptional error conditions.
-  static const char kNoResponseError[];
-  static const char kUnexpectedResponse[];
-
  protected:
   BluetoothMediaTransportClient();