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

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

Index: chrome/browser/extensions/api/messaging/native_process_launcher.h
diff --git a/chrome/browser/extensions/api/messaging/native_process_launcher.h b/chrome/browser/extensions/api/messaging/native_process_launcher.h
index f1b6483b9d15a813653691661bfcd0896341ab39..0924f13ae39b143bfb9002512147165443a7602a 100644
--- a/chrome/browser/extensions/api/messaging/native_process_launcher.h
+++ b/chrome/browser/extensions/api/messaging/native_process_launcher.h
@@ -8,6 +8,7 @@
 #include "base/process.h"
 #include "chrome/browser/extensions/api/messaging/native_message_process_host.h"
 
+class CommandLine;
 class GURL;
 
 namespace base {
@@ -53,7 +54,7 @@ class NativeProcessLauncher {
 
   // 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);