Remove base::SharedMemory::ShareToProcess.

extensions/browser/user_script_loader.cc

Index: extensions/browser/user_script_loader.cc
diff --git a/extensions/browser/user_script_loader.cc b/extensions/browser/user_script_loader.cc
index c20f96a6385963bf57d01a3d9c350646bdd811cb..3eaa3ffc37577f26ef9263a0108e056d2e9d6f13 100644
--- a/extensions/browser/user_script_loader.cc
+++ b/extensions/browser/user_script_loader.cc
@@ -416,9 +416,10 @@ void UserScriptLoader::SendUpdate(content::RenderProcessHost* process,
   if (!handle)
     return;
 
-  base::SharedMemoryHandle handle_for_process;
-  if (!shared_memory->ShareToProcess(handle, &handle_for_process))
-    return;  // This can legitimately fail if the renderer asserts at startup.
+  base::SharedMemoryHandle handle_for_process =
+      shared_memory->handle().Duplicate();
+  if (!handle_for_process.IsValid())
+    return;
 
   if (base::SharedMemory::IsHandleValid(handle_for_process)) {

[Nico, 2017/05/02 20:08:00]

Is this just an alias for handle_for_process.isValid() now?

[erikchen, 2017/05/02 21:33:10]

Yes. We could do a whole other refactor to make that change - I'm avoiding doing
so, since it's not actually needed for the changes I'm *actually* trying to
make.

     process->Send(new ExtensionMsg_UpdateUserScripts(