Returning scoped_ptr instead of raw pointer in SpdySessionPoolInfoToValue() in net/

net/spdy/spdy_session.h

 // 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.
 
 #ifndef NET_SPDY_SPDY_SESSION_H_
 #define NET_SPDY_SPDY_SESSION_H_
 
 #include <deque>
 #include <map>
 #include <set>
@@ skipping 379 matching lines @@
   // STATE_GOING_AWAY. After this function, DcheckGoingAway() will
   // pass. May be called multiple times.
   void StartGoingAway(SpdyStreamId last_good_stream_id, Error status);
 
   // Must be called only when going away (i.e., DcheckGoingAway()
   // passes). If there are no more active streams and the session
   // isn't closed yet, close it.
   void MaybeFinishGoingAway();
 
   // Retrieves information on the current state of the SPDY session as a
   // Value.  Caller takes possession of the returned value.

[eroman, 2015/05/25 17:50:46]

Remove comment on ownership

[payal.pandey, 2015/05/26 06:07:35]

Done.

-  base::Value* GetInfoAsValue() const;
+  scoped_ptr<base::Value> GetInfoAsValue() const;
 
   // Indicates whether the session is being reused after having successfully
   // used to send/receive data in the past or if the underlying socket was idle
   // before being used for a SPDY session.
   bool IsReused() const;
 
   // Returns true if the underlying transport socket ever had any reads or
   // writes.
   bool WasEverUsed() const {
     return connection_->socket()->WasEverUsed();
@@ skipping 771 matching lines @@
   // Used for posting asynchronous IO tasks.  We use this even though
   // SpdySession is refcounted because we don't need to keep the SpdySession
   // alive if the last reference is within a RunnableMethod.  Just revoke the
   // method.
   base::WeakPtrFactory<SpdySession> weak_factory_;
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_SPDY_SESSION_H_