Pass correct --parent-window value to native messaging hosts

Pass correct --parent-window value to native messaging hosts

--parent-window is used to pass HWND of the chrome window when launching
native messaging hosts. It was brokend with Aura because

BUG=330777
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244445

[miket_OOO, 2014-01-07 17:15:11]

lgtm

https://codereview.chromium.org/115683005/diff/570001/chrome/test/data/native...
File chrome/test/data/native_messaging/native_hosts/echo.py (right):

https://codereview.chromium.org/115683005/diff/570001/chrome/test/data/native...
chrome/test/data/native_messaging/native_hosts/echo.py:26: caller_url = None;
Begone, semicolon!

https://codereview.chromium.org/115683005/diff/570001/chrome/test/data/native...
chrome/test/data/native_messaging/native_hosts/echo.py:34: if
arg.startswith('--parent-window='):
Not a big deal, but it makes me sad when I see manual parsing of command-line
arguments when wonderful libraries like
http://docs.python.org/2/library/argparse.html#module-argparse exist. OK to keep
as-is -- or maybe add a TODO for next time?

[Sergey Ulanov, 2014-01-07 20:56:10]

https://codereview.chromium.org/115683005/diff/570001/chrome/test/data/native...
File chrome/test/data/native_messaging/native_hosts/echo.py (right):

https://codereview.chromium.org/115683005/diff/570001/chrome/test/data/native...
chrome/test/data/native_messaging/native_hosts/echo.py:26: caller_url = None;
On 2014/01/07 17:15:11, miket wrote:
> Begone, semicolon!

Done.

https://codereview.chromium.org/115683005/diff/570001/chrome/test/data/native...
chrome/test/data/native_messaging/native_hosts/echo.py:34: if
arg.startswith('--parent-window='):
On 2014/01/07 17:15:11, miket wrote:
> Not a big deal, but it makes me sad when I see manual parsing of command-line
> arguments when wonderful libraries like
> http://docs.python.org/2/library/argparse.html#module-argparse exist. OK to
keep
> as-is -- or maybe add a TODO for next time?

Added TODO. FWIW argparse is available only starting Python 2.7 and we are still
using 2.6 on Windows. This script could use optparse (which is available in 2.6,
deprecated in 2.7), but it doesn't seem to worth spending time on it right now
given that we would want to switch to argparse in the future.

[miket_OOO, 2014-01-07 20:58:39]

> Added TODO. FWIW argparse is available only starting Python 2.7 and we are
still
> using 2.6 on Windows. This script could use optparse (which is available in
2.6,
> deprecated in 2.7), but it doesn't seem to worth spending time on it right now
> given that we would want to switch to argparse in the future.

That's reasonable. Thanks!

[commit-bot: I haz the power, 2014-01-12 08:46:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/115683005/990001

[commit-bot: I haz the power, 2014-01-12 18:21:07]

Change committed as 244445