Returning scoped_ptr<> instead of raw pointer in DnsQuery::CloneWithNewId

net/dns/dns_transaction.cc

Index: net/dns/dns_transaction.cc
diff --git a/net/dns/dns_transaction.cc b/net/dns/dns_transaction.cc
index f60ae26e27d76f43eb98fe536fdd5ddd55724a15..148d248ccb513ba93a677018475640e01658f40c 100644
--- a/net/dns/dns_transaction.cc
+++ b/net/dns/dns_transaction.cc
@@ -708,7 +708,7 @@ class DnsTransactionImpl : public DnsTransaction,
     if (attempts_.empty()) {
       query.reset(new DnsQuery(id, qnames_.front(), qtype_));
     } else {
-      query.reset(attempts_[0]->GetQuery()->CloneWithNewId(id));
+      query = attempts_[0]->GetQuery()->CloneWithNewId(id);
     }
 
     const DnsConfig& config = session_->config();
@@ -759,8 +759,8 @@ class DnsTransactionImpl : public DnsTransaction,
 
     // TODO(szym): Reuse the same id to help the server?
     uint16 id = session_->NextQueryId();
-    scoped_ptr<DnsQuery> query(
-        previous_attempt->GetQuery()->CloneWithNewId(id));
+    scoped_ptr<DnsQuery> query =
+        previous_attempt->GetQuery()->CloneWithNewId(id);
 
     RecordLostPacketsIfAny();
     // Cancel all other attempts, no point waiting on them.