Add version info resource to the it2me and me2me native messaging hosts.

Add version info resource to the it2me and me2me native messaging hosts.

BUG=338952, 325567
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251921

[weitao, 2014-02-18 20:11:24]

https://codereview.chromium.org/171163003/diff/1/remoting/remoting_host.gypi
File remoting/remoting_host.gypi (right):

https://codereview.chromium.org/171163003/diff/1/remoting/remoting_host.gypi#...
remoting/remoting_host.gypi:592: 'SubSystem': '1', # /SUBSYSTEM:CONSOLE
The default value is '1'. But we should be explicit about it because the native
messaging hosts need to be console apps.

https://codereview.chromium.org/171163003/diff/1/remoting/resources/remoting_...
File remoting/resources/remoting_strings.grd (right):

https://codereview.chromium.org/171163003/diff/1/remoting/resources/remoting_...
remoting/resources/remoting_strings.grd:684: Native messaging host process
Is this a proper description? Is casing correct?

https://codereview.chromium.org/171163003/diff/1/remoting/resources/remoting_...
remoting/resources/remoting_strings.grd:687: Remote assistance host process
Is this a proper description? is casing correct?

[Sergey Ulanov, 2014-02-18 22:27:27]

lgtm

https://codereview.chromium.org/171163003/diff/1/remoting/resources/remoting_...
File remoting/resources/remoting_strings.grd (right):

https://codereview.chromium.org/171163003/diff/1/remoting/resources/remoting_...
remoting/resources/remoting_strings.grd:684: Native messaging host process
On 2014/02/18 20:11:24, weitaosu wrote:
> Is this a proper description? Is casing correct?

I suggest "Native messaging host".
Don't need 'process' as this is a description for a file, not process.

https://codereview.chromium.org/171163003/diff/1/remoting/resources/remoting_...
remoting/resources/remoting_strings.grd:687: Remote assistance host process
On 2014/02/18 20:11:24, weitaosu wrote:
> Is this a proper description? is casing correct?

Maybe 'Native messaging host for remote assistance' ?

[weitao, 2014-02-18 22:40:46]

The CQ bit was checked by weitaosu@chromium.org

[commit-bot: I haz the power, 2014-02-18 22:42:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/weitaosu@chromium.org/171163003/70001

[commit-bot: I haz the power, 2014-02-19 02:06:13]

Change committed as 251921