Remove raw DictionaryValue::Set in //chrome

chrome/browser/prerender/prerender_history.h

Index: chrome/browser/prerender/prerender_history.h
diff --git a/chrome/browser/prerender/prerender_history.h b/chrome/browser/prerender/prerender_history.h
index 1ace14df5790cb7bb8e7acf93abf3ab3dbd4e8de..8418fb3687fec3904ad4d4b493bea7e6f776228c 100644
--- a/chrome/browser/prerender/prerender_history.h
+++ b/chrome/browser/prerender/prerender_history.h
@@ -68,7 +68,7 @@ class PrerenderHistory : public base::NonThreadSafe {
   void Clear();
 
   // Retrieves the entries as a value which can be displayed.
-  base::Value* GetEntriesAsValue() const;
+  std::unique_ptr<base::Value> GetEntriesAsValue() const;

[Nico, 2017/05/17 15:43:10]

Huh, having a function starting Get... transfer ownership is kind of surprising.
Maybe CopyEntriesAsValue() would be better? (but optional, not for this CL, etc)

[jdoerrie, 2017/05/17 16:23:54]

Done: https://codereview.chromium.org/2893653002

 
  private:
   std::list<Entry> entries_;