Pass ID of the calling extension to the native messaging host.

chrome/browser/extensions/api/messaging/native_process_launcher.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_MESSAGING_NATIVE_PROCESS_LAUNCHER_H_
 #define CHROME_BROWSER_EXTENSIONS_API_MESSAGING_NATIVE_PROCESS_LAUNCHER_H_
 
 #include "base/process.h"
 #include "chrome/browser/extensions/api/messaging/native_message_process_host.h"
 
+class CommandLine;
 class GURL;
 
 namespace base {
 class FilePath;
 }
 
 namespace extensions {
 
 class NativeMessagingHostManifest;
 
@@ skipping 24 matching lines @@
  protected:
   // The following two methods are platform specific and are implemented in
   // platform-specific .cc files.
 
   // Loads manifest for the native messaging host |name|.
   static scoped_ptr<NativeMessagingHostManifest> FindAndLoadManifest(
       const std::string& native_host_name,
       std::string* error_message);
 
   // Launches native messaging process.
   static bool LaunchNativeProcess(

[Matt Perry, 2013/03/05 18:19:24]

I think it would be better to have this method accept a path and origin_url.
Otherwise, the CommandLine parameter is opaque, and it's unclear what arguments
you should pass.

[Sergey Ulanov, 2013/03/05 19:03:05]

This method is protected and supposed to be called by NativeProcessLauncherImpl
only. I think it's better to pass command line here is because we always want to
use the same command line on all platforms so the code that creates command line
should be shared between platforms.

[Matt Perry, 2013/03/05 19:14:25]

Oh OK, that makes sense.

nit: fix the variable name here

-      const base::FilePath& path,
+      const CommandLine& path,
       base::PlatformFile* read_file,
       base::PlatformFile* write_file);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NativeProcessLauncher);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_MESSAGING_NATIVE_PROCESS_LAUNCHER_H_