Returning scoped_ptr in SdchInfoToValue() instead of raw pointers in net/base

net/base/sdch_manager.h

 // Copyright (c) 2011 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.
 
 // This file contains the SdchManager class and the DictionarySet
 // nested class.  The manager is responsible for storing all
 // SdchDictionarys, and provides access to them through DictionarySet
 // objects. A DictionarySet is an object whose lifetime is under the
 // control of the consumer. It is a reference to a set of
 // dictionaries, and guarantees that none of those dictionaries will
@@ skipping 166 matching lines @@
                            std::string* client_hash, std::string* server_hash);
 
   // For Latency testing only, we need to know if we've succeeded in doing a
   // round trip before starting our comparative tests. If ever we encounter
   // problems with SDCH, we opt-out of the test unless/until we perform a
   // complete SDCH decoding.
   bool AllowLatencyExperiment(const GURL& url) const;
 
   void SetAllowLatencyExperiment(const GURL& url, bool enable);
 
-  base::Value* SdchInfoToValue() const;
+  scoped_ptr<base::Value> SdchInfoToValue() const;
 
   // Add an SDCH dictionary to our list of availible
   // dictionaries. This addition will fail if addition is illegal
   // (data in the dictionary is not acceptable from the
   // dictionary_url; dictionary already added, etc.).
   // If |server_hash| is non-null, returns the server hash that may be
   // used as an argument to GetDictionarySetByHash.
   // Returns SDCH_OK if the addition was successfull, and corresponding error
   // code otherwise.
   SdchProblemCode AddSdchDictionary(const std::string& dictionary_text,
@@ skipping 59 matching lines @@
   base::ThreadChecker thread_checker_;
 
   base::WeakPtrFactory<SdchManager> factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SdchManager);
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_SDCH_MANAGER_H_