Remove logging::LogAtLevel and rewrite its sole caller.

dbus/object_proxy.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 "dbus/object_proxy.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 553 matching lines @@
     const base::StringPiece& error_message) const {
   if (ignore_service_unknown_errors_ &&
       (error_name == kErrorServiceUnknown || error_name == kErrorObjectUnknown))
     return;
   logging::LogSeverity severity = logging::LOG_ERROR;
   // "UnknownObject" indicates that an object or service is no longer available,
   // e.g. a Shill network service has gone out of range. Treat these as warnings
   // not errors.
   if (error_name == kErrorObjectUnknown)
     severity = logging::LOG_WARNING;
-  std::ostringstream msg;
-  msg << "Failed to call method: " << interface_name << "." << method_name
-      << ": object_path= " << object_path_.value()
-      << ": " << error_name << ": " << error_message;
-  logging::LogAtLevel(severity, msg.str());
+  LOG_AT_LEVEL(severity) << "Failed to call method: " << interface_name << "."

[danakj, 2016/01/19 21:10:19]

How about we delete the method in base and just make this code use LOG().

[jbroman, 2016/01/19 21:28:11]

Done.

+                         << method_name
+                         << ": object_path= " << object_path_.value() << ": "
+                         << error_name << ": " << error_message;
 }
 
 void ObjectProxy::OnCallMethodError(const std::string& interface_name,
                                     const std::string& method_name,
                                     ResponseCallback response_callback,
                                     ErrorResponse* error_response) {
   if (error_response) {
     // Error message may contain the error message as string.
     MessageReader reader(error_response);
     std::string error_message;
@@ skipping 114 matching lines @@
     bool service_is_available) {
   bus_->AssertOnOriginThread();
 
   std::vector<WaitForServiceToBeAvailableCallback> callbacks;
   callbacks.swap(wait_for_service_to_be_available_callbacks_);
   for (size_t i = 0; i < callbacks.size(); ++i)
     callbacks[i].Run(service_is_available);
 }
 
 }  // namespace dbus