Use scoped_ptr<> to pass ownership in more places.

remoting/host/log_to_server.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/log_to_server.h"
 
 #include "base/bind.h"
 #include "base/message_loop_proxy.h"
 #include "remoting/base/constants.h"
 #include "remoting/host/chromoting_host.h"
@@ skipping 79 matching lines @@
     return;
   }
   if (pending_entries_.empty()) {
     return;
   }
   // Make one stanza containing all the pending entries.
   scoped_ptr<XmlElement> stanza(new XmlElement(QName(
       kChromotingXmlNamespace, kLogCommand)));
   while (!pending_entries_.empty()) {
     ServerLogEntry& entry = pending_entries_.front();
-    stanza->AddElement(entry.ToStanza());

[Alpha Left Google, 2012/03/20 18:56:40]

This old code seems to cause a crash.

[Sergey Ulanov, 2012/03/20 19:08:12]

Not sure what you mean here. How does it cause a crash?

+    stanza->AddElement(entry.ToStanza().release());
     pending_entries_.pop_front();
   }
   // Send the stanza to the server.
   scoped_ptr<IqRequest> req = iq_sender_->SendIq(
       buzz::STR_SET, kChromotingBotJid, stanza.Pass(),
       IqSender::ReplyCallback());
   // We ignore any response, so let the IqRequest be destroyed.
   return;
 }
 
 }  // namespace remoting