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

net/spdy/spdy_session_pool.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_POOL_H_
 #define NET_SPDY_SPDY_SESSION_POOL_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 101 matching lines @@
 
   // Close only the currently existing SpdySessions that are idle.
   // Let any new ones created while this method is running continue to
   // live.
   void CloseCurrentIdleSessions();
 
   // Close all SpdySessions, including any new ones created in the process of
   // closing the current ones.
   void CloseAllSessions();
 
   // Creates a Value summary of the state of the spdy session pool. The caller

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

Remove ownership comment

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

Done.

   // responsible for deleting the returned value.
-  base::Value* SpdySessionPoolInfoToValue() const;
+  scoped_ptr<base::Value> SpdySessionPoolInfoToValue() const;
 
   base::WeakPtr<HttpServerProperties> http_server_properties() {
     return http_server_properties_;
   }
 
   // NetworkChangeNotifier::IPAddressObserver methods:
 
   // We flush all idle sessions and release references to the active ones so
   // they won't get re-used.  The active ones will either complete successfully
   // or error out due to the IP address change.
@@ skipping 85 matching lines @@
   // This SPDY proxy is allowed to push resources from origins that are
   // different from those of their associated streams.
   HostPortPair trusted_spdy_proxy_;
 
   DISALLOW_COPY_AND_ASSIGN(SpdySessionPool);
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_SPDY_SESSION_POOL_H_