Move Chromoting strings to string_resources.grd.

Move Chromoting strings to string_resources.grd.

Previously all strings were in messages.json files. Moving them to GRD 
file will allow to share them with native components. Also added script 
that I used to convert translations - to make it easier to review this 
CL and just in case we need it again in the future.

[Jamie, 2012-10-15 04:06:37]

This CL seems to get rid of all the JSON files without adding a script to
recreate them from the XTB files. Am I missing something?

http://codereview.chromium.org/11150008/diff/2016/remoting/remoting.gyp
File remoting/remoting.gyp (right):

http://codereview.chromium.org/11150008/diff/2016/remoting/remoting.gyp#newco...
remoting/remoting.gyp:74: '<(webapp_locale_dir)/ar/messages.json',
This will fail for an official build because the ".official" has been lost in
this patch. I'm not convinced that the old approach is the best way to separate
out the official vs. unofficial web-apps, but for this CL I think we need to
preserve the current behaviour.

http://codereview.chromium.org/11150008/diff/2016/remoting/remoting.gyp#newco...
remoting/remoting.gyp:1150: 'remoting_resources',
This change seems unrelated to anything else in this CL. What is it needed for?

[Sergey Ulanov, 2012-10-15 18:57:21]

> This CL seems to get rid of all the JSON files without adding a script to
> recreate them from the XTB files. Am I missing something?

I'm adding support for messages.json directly in grit:
http://codereview.chromium.org/11152002/ . The grd file itself specifies that
messages.json files should be generated.

http://codereview.chromium.org/11150008/diff/2016/remoting/remoting.gyp
File remoting/remoting.gyp (right):

http://codereview.chromium.org/11150008/diff/2016/remoting/remoting.gyp#newco...
remoting/remoting.gyp:74: '<(webapp_locale_dir)/ar/messages.json',
On 2012/10/15 04:06:38, Jamie wrote:
> This will fail for an official build because the ".official" has been lost in
> this patch. I'm not convinced that the old approach is the best way to
separate
> out the official vs. unofficial web-apps, but for this CL I think we need to
> preserve the current behaviour.

Now the messages.json files are generated from grd file and they are always
stored in the location specified by webapp_locale_dir, which is the same for
both official and chromium builds.

http://codereview.chromium.org/11150008/diff/2016/remoting/remoting.gyp#newco...
remoting/remoting.gyp:1150: 'remoting_resources',
On 2012/10/15 04:06:38, Jamie wrote:
> This change seems unrelated to anything else in this CL. What is it needed
for?

It is related. remoting_resources target now generates messages.json files that
are used by the webapp.

[Jamie, 2012-10-15 20:13:23]

Okay, that makes sense. So this CL is dependent on that one then? That being the
case, I'll hold of okaying this one until the other has landed and you've had a
chance to test with it in place.

[Sergey Ulanov, 2012-10-16 20:19:41]

On 2012/10/15 20:13:23, Jamie wrote:
> Okay, that makes sense. So this CL is dependent on that one then? That being
the
> case, I'll hold of okaying this one until the other has landed and you've had
a
> chance to test with it in place.

I've landed and rolled grit changes. PTAL.

[Sergey Ulanov, 2012-10-18 18:44:15]

Jamie: ping

[Jamie, 2012-10-18 19:35:51]

On 2012/10/18 18:44:15, sergeyu wrote:
> Jamie: ping

lgtm, although I haven't looked at json_to_grd in detail. If it's important
enough to warrant a thorough review, please ask someone who knows Python better
than I do.

Also, I still need to land a few translations for M23. Can you wait to commit
this for a few more days, otherwise I won't easily be able to merge them.

[Sergey Ulanov, 2012-10-18 20:12:44]

On 2012/10/18 19:35:51, Jamie wrote:
> On 2012/10/18 18:44:15, sergeyu wrote:
> > Jamie: ping
> 
> lgtm, although I haven't looked at json_to_grd in detail. If it's important
> enough to warrant a thorough review, please ask someone who knows Python
better
> than I do.

I added it here just in case we need it later and in case you wanted to look
inside. But it's not very likely we will need to use it again. So I'll remove it
before landing this CL.

> 
> Also, I still need to land a few translations for M23. Can you wait to commit
> this for a few more days, otherwise I won't easily be able to merge them.

SGTM

[commit-bot: I haz the power, 2012-10-30 20:49:02]

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

[commit-bot: I haz the power, 2012-10-30 22:34:55]

Change committed as 165041