[Chromoting] Replace some bare pointers with scoped_ptrs, and improve naming.

remoting/host/server_log_entry.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 "remoting/host/server_log_entry.h"
 
 #include "base/sys_info.h"
 #include "remoting/base/constants.h"
 #include "remoting/protocol/session.h"
 #include "third_party/libjingle/source/talk/xmllite/xmlelement.h"
@@ skipping 45 matching lines @@
 ServerLogEntry::~ServerLogEntry() {
 }
 
 // static
 scoped_ptr<buzz::XmlElement> ServerLogEntry::MakeStanza() {
   return scoped_ptr<buzz::XmlElement>(
       new XmlElement(QName(kChromotingXmlNamespace, kLogCommand)));
 }
 
 // static
-ServerLogEntry* ServerLogEntry::MakeSessionStateChange(bool connected) {
-  ServerLogEntry* entry = new ServerLogEntry();
+scoped_ptr<ServerLogEntry> ServerLogEntry::MakeForSessionStateChange(
+    bool connected) {
+  scoped_ptr<ServerLogEntry> entry(new ServerLogEntry());
   entry->Set(kKeyRole, kValueRoleHost);
   entry->Set(kKeyEventName, kValueEventNameSessionState);
   entry->Set(kKeySessionState, GetValueSessionState(connected));
-  return entry;
+  return entry.Pass();
 }
 
 // static
-ServerLogEntry* ServerLogEntry::MakeForHeartbeat() {
-  ServerLogEntry* entry = new ServerLogEntry();
+scoped_ptr<ServerLogEntry> ServerLogEntry::MakeForHeartbeat() {
+  scoped_ptr<ServerLogEntry> entry(new ServerLogEntry());
   entry->Set(kKeyRole, kValueRoleHost);
   entry->Set(kKeyEventName, kValueEventNameHeartbeat);
-  return entry;
+  return entry.Pass();
 }
 
 void ServerLogEntry::AddHostFields() {
 #if defined(OS_WIN)
   Set(kKeyOsName, kValueOsNameWindows);
 #elif defined(OS_MACOSX)
   Set(kKeyOsName, kValueOsNameMac);
 #elif defined(OS_CHROMEOS)
   Set(kKeyOsName, kValueOsNameChromeOS);
 #elif defined(OS_LINUX)
@@ skipping 52 matching lines @@
 // static
 const char* ServerLogEntry::GetValueSessionState(bool connected) {
   return connected ? kValueSessionStateConnected : kValueSessionStateClosed;
 }
 
 void ServerLogEntry::Set(const std::string& key, const std::string& value) {
   values_map_[key] = value;
 }
 
 }  // namespace remoting