chromeos/dbus: Add verbose log to media-related clients and service provider.

chromeos/dbus/bluetooth_media_client.cc

 // Copyright 2014 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 "chromeos/dbus/bluetooth_media_client.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "base/observer_list.h"
@@ skipping 85 matching lines @@
   void RemoveObserver(BluetoothMediaClient::Observer* observer) override {
     DCHECK(observer);
     observers_.RemoveObserver(observer);
   }
 
   void RegisterEndpoint(const dbus::ObjectPath& object_path,
                         const dbus::ObjectPath& endpoint_path,
                         const EndpointProperties& properties,
                         const base::Closure& callback,
                         const ErrorCallback& error_callback) override {
+    VLOG(1) << "RegisterEndpoint: endpoint:" << endpoint_path.value();

[armansito, 2015/02/28 00:50:28]

nit: Add space after "endpoint: ", otherwise the object path will print
immediately after the colon. Also two colons make look a bit weird, I'd do:

"RegisterEndpoint - endpoint: " << ...

[Miao, 2015/02/28 02:19:32]

Done.

+
     dbus::MethodCall method_call(kBluetoothMediaInterface, kRegisterEndpoint);
 
     dbus::MessageWriter writer(&method_call);
     dbus::MessageWriter array_writer(nullptr);
     dbus::MessageWriter dict_entry_writer(nullptr);
 
     // Send the path to the endpoint.
     writer.AppendObjectPath(endpoint_path);
 
     writer.OpenArray("{sv}", &array_writer);
@@ skipping 32 matching lines @@
         base::Bind(&BluetoothMediaClientImpl::OnSuccess,
                    weak_ptr_factory_.GetWeakPtr(), callback),
         base::Bind(&BluetoothMediaClientImpl::OnError,
                    weak_ptr_factory_.GetWeakPtr(), error_callback));
   }
 
   void UnregisterEndpoint(const dbus::ObjectPath& object_path,
                           const dbus::ObjectPath& endpoint_path,
                           const base::Closure& callback,
                           const ErrorCallback& error_callback) override {
+    VLOG(1) << "UnregisterEndpoint: endpoint:" << endpoint_path.value();

[armansito, 2015/02/28 00:50:28]

nit: same as above.

[Miao, 2015/02/28 02:19:32]

Done.

+
     dbus::MethodCall method_call(kBluetoothMediaInterface, kUnregisterEndpoint);
 
     // Send the path to the endpoint.
     dbus::MessageWriter writer(&method_call);
     writer.AppendObjectPath(endpoint_path);
 
     // Get Object Proxy based on the service name and the service path and call
     // RegisterEndpoint medthod.
     scoped_refptr<dbus::ObjectProxy> object_proxy(
         object_manager_->GetObjectProxy(object_path));
@@ skipping 54 matching lines @@
 }
 
 BluetoothMediaClient::~BluetoothMediaClient() {
 }
 
 BluetoothMediaClient* BluetoothMediaClient::Create() {
   return new BluetoothMediaClientImpl();
 }
 
 }  // namespace chromeos