Code cleaning: Uses scoped_ptr<> to express ownership rather than writing ownership in comments.

chromeos/dbus/ibus/ibus_panel_service.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 "chromeos/dbus/ibus/ibus_panel_service.h"
 
 #include <string>
 #include "base/bind.h"
 #include "base/callback.h"
 #include "chromeos/dbus/ibus/ibus_constants.h"
@@ skipping 195 matching lines @@
                    << method_call->ToString();
       return;
     }
     bool visible = false;
     if (!reader.PopBool(&visible)) {
       LOG(WARNING) << "UpdateLookupTable called with incorrect parameters: "
                    << method_call->ToString();
       return;
     }
     candidate_window_handler_->UpdateLookupTable(table, visible);
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles HideLookupTable method call from ibus-daemon.
   void HideLookupTable(dbus::MethodCall* method_call,
                        dbus::ExportedObject::ResponseSender response_sender) {
     if (!candidate_window_handler_)
       return;
 
     candidate_window_handler_->HideLookupTable();
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles UpdateAuxiliaryText method call from ibus-daemon.
   void UpdateAuxiliaryText(
       dbus::MethodCall* method_call,
       dbus::ExportedObject::ResponseSender response_sender) {
     if (!candidate_window_handler_)
       return;
 
     dbus::MessageReader reader(method_call);
     std::string text;
     if (!ibus::PopStringFromIBusText(&reader, &text)) {
       LOG(WARNING) << "UpdateAuxiliaryText called with incorrect parameters: "
                    << method_call->ToString();
       return;
     }
     bool visible = false;
     if (!reader.PopBool(&visible)) {
       LOG(WARNING) << "UpdateAuxiliaryText called with incorrect parameters: "
                    << method_call->ToString();
       return;
     }
     candidate_window_handler_->UpdateAuxiliaryText(text, visible);
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles HideAuxiliaryText method call from ibus-daemon.
   void HideAuxiliaryText(dbus::MethodCall* method_call,
                          dbus::ExportedObject::ResponseSender response_sender) {
     if (!candidate_window_handler_)
       return;
 
     candidate_window_handler_->HideAuxiliaryText();
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles UpdatePreeditText method call from ibus-daemon.
   void UpdatePreeditText(dbus::MethodCall* method_call,
                          dbus::ExportedObject::ResponseSender response_sender) {
     if (!candidate_window_handler_)
       return;
 
     dbus::MessageReader reader(method_call);
     std::string text;
     if (!ibus::PopStringFromIBusText(&reader, &text)) {
       LOG(WARNING) << "UpdatePreeditText called with incorrect parameters: "
                    << method_call->ToString();
       return;
     }
     uint32 cursor_pos = 0;
     if (!reader.PopUint32(&cursor_pos)) {
       LOG(WARNING) << "UpdatePreeditText called with incorrect parameters: "
                    << method_call->ToString();
       return;
     }
     bool visible = false;
     if (!reader.PopBool(&visible)) {
       LOG(WARNING) << "UpdatePreeditText called with incorrect parameters: "
                    << method_call->ToString();
       return;
     }
     candidate_window_handler_->UpdatePreeditText(text, cursor_pos, visible);
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles HidePreeditText method call from ibus-daemon.
   void HidePreeditText(dbus::MethodCall* method_call,
                        dbus::ExportedObject::ResponseSender response_sender) {
     if (!candidate_window_handler_)
       return;
 
     candidate_window_handler_->HidePreeditText();
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles RegisterProperties method call from ibus-daemon.
   void RegisterProperties(
       dbus::MethodCall* method_call,
       dbus::ExportedObject::ResponseSender response_sender) {
     if (!property_handler_)
       return;
 
     dbus::MessageReader reader(method_call);
     ibus::IBusPropertyList properties;
     if (!ibus::PopIBusPropertyList(&reader, &properties)) {
       DLOG(WARNING) << "RegisterProperties called with incorrect parameters:"
                     << method_call->ToString();
       return;
     }
     property_handler_->RegisterProperties(properties);
 
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Handles UpdateProperty method call from ibus-daemon.
   void UpdateProperty(dbus::MethodCall* method_call,
                       dbus::ExportedObject::ResponseSender response_sender) {
     if (!property_handler_)
       return;
 
     dbus::MessageReader reader(method_call);
     ibus::IBusProperty property;
     if (!ibus::PopIBusProperty(&reader, &property)) {
       DLOG(WARNING) << "RegisterProperties called with incorrect parameters:"
                     << method_call->ToString();
       return;
     }
     property_handler_->UpdateProperty(property);
 
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   void SetCursorLocation(const ibus::Rect& cursor_location,
                          const ibus::Rect& composition_head) {
     // TODO(nona): implement this function.
   }
 
   // Handles FocusIn, FocusOut, StateChanged method calls from IBus, and ignores
   // them.
   void NoOperation(dbus::MethodCall* method_call,
                    dbus::ExportedObject::ResponseSender response_sender) {
     if (!property_handler_)
       return;
 
-    dbus::Response* response = dbus::Response::FromMethodCall(method_call);
-    response_sender.Run(response);
+    response_sender.Run(dbus::Response::FromMethodCall(method_call));
   }
 
   // Called when the method call is exported.
   void OnMethodExported(const std::string& interface_name,
                         const std::string& method_name,
                         bool success) {
     LOG_IF(WARNING, !success) << "Failed to export "
                               << interface_name << "." << method_name;
   }
 
@@ skipping 82 matching lines @@
     IBusInputContextClient* input_context) {
   if (type == REAL_DBUS_CLIENT_IMPLEMENTATION) {
     return new IBusPanelServiceImpl(bus, input_context);
   } else {
     return new IBusPanelServiceDaemonlessImpl();
   }
 }
 
 }  // namespace ibus
 }  // namespace chromeos