Mandate unique_ptr for base::IDMap in IDMapOwnPointer mode.

content/child/indexed_db/indexed_db_dispatcher.h

Index: content/child/indexed_db/indexed_db_dispatcher.h
diff --git a/content/child/indexed_db/indexed_db_dispatcher.h b/content/child/indexed_db/indexed_db_dispatcher.h
index 3d75d37bc6dc1ba54beb815020c0774b9306c01e..c5ae7d0c925629cd0ac28e6ecf2612e54781fb24 100644
--- a/content/child/indexed_db/indexed_db_dispatcher.h
+++ b/content/child/indexed_db/indexed_db_dispatcher.h
@@ -131,9 +131,9 @@ class CONTENT_EXPORT IndexedDBDispatcher : public WorkerThread::Observer {
 
   template <typename T>
   void init_params(T* params, blink::WebIDBCallbacks* callbacks_ptr) {

[danakj, 2016/11/18 00:15:33]

could this be a unique_ptr?

[rlanday, 2016/11/18 20:40:41]

It's unclear to me that this function is used anywhere, I can comment it out and
still build all the targets buildable on Linux. I also can't find any calls
using code search. I guess I'll change it to unique_ptr here and put up a
separate changeset to remove it.

-    std::unique_ptr<blink::WebIDBCallbacks> callbacks(callbacks_ptr);
     params->ipc_thread_id = CurrentWorkerId();
-    params->ipc_callbacks_id = pending_callbacks_.Add(callbacks.release());
+    params->ipc_callbacks_id = pending_callbacks_.Add(
+        std::unique_ptr<blink::WebIDBCallbacks>(callbacks_ptr));
   }
 
   // IDBCallback message handlers.