Remove base::SharedMemory::ShareToProcess.

extensions/browser/user_script_loader.cc

 // Copyright 2014 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.
 
 #include "extensions/browser/user_script_loader.h"
 
 #include <stddef.h>
 
 #include <set>
 #include <string>
@@ skipping 398 matching lines @@
   if (!ExtensionsBrowserClient::Get()->IsSameContext(
           browser_context_, process->GetBrowserContext()))
     return;
 
   // If the process is being started asynchronously, early return.  We'll end up
   // calling InitUserScripts when it's created which will call this again.
   base::ProcessHandle handle = process->GetHandle();
   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(
         handle_for_process, host_id(), changed_hosts, whitelisted_only));
   }
 }
 
 }  // namespace extensions