SharedWorker: Clean up SharedWorkerHost

content/browser/shared_worker/shared_worker_host.h

Index: content/browser/shared_worker/shared_worker_host.h
diff --git a/content/browser/shared_worker/shared_worker_host.h b/content/browser/shared_worker/shared_worker_host.h
index 32235254aca259bd06ea7eca889ddbc316e16b6b..f1c394e7651e0ad194652f6efd9dde53291499af 100644
--- a/content/browser/shared_worker/shared_worker_host.h
+++ b/content/browser/shared_worker/shared_worker_host.h
@@ -37,7 +37,7 @@ class SharedWorkerHost {
   ~SharedWorkerHost();
 
   // Sends |message| to the SharedWorker.
-  bool Send(IPC::Message* message);
+  bool Send(std::unique_ptr<IPC::Message> message);

[kinuko, 2016/12/27 15:38:29]

This generally follows IPC::Sender pattern, it's fine but just passing through
ptr is more common

[nhiroki, 2016/12/28 02:20:37]

I reverted this change because I plan to mojofy shared worker anytime soon and
this pattern will be gone regardless of this change.

 
   // Starts the SharedWorker in the renderer process which is associated with
   // |filter_|.