Fix branding in chromoting string resources

Move Chromoting strings to string_resources.grd and simplify remoting_resources target.

Previously all strings were in messages.json files. Moving them to GRD 
file will allow to share them with native components. Also simplified 
how resources are generated:
 1. Now <if> is used to handle branded strings instead of generating 
    grd file in build time
 2. Removed resource_ids - ids are now specified in grd files.

BUG=158995
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166127

[Sergey Ulanov, 2012-11-01 23:21:32]

+tony@

[tony, 2012-11-01 23:29:14]

LGTM!  I assume it builds (with no extra compiling) both with branding=Chrome
and without.

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi
File build/grit_action.gypi (right):

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi#newc...
build/grit_action.gypi:19: 'grit_resource_ids%':
'GRIT_DIR/../gritsettings/resource_ids',
Do you need the %? I thought that meant you could override a value from the
command line.

[Sergey Ulanov, 2012-11-01 23:34:38]

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi
File build/grit_action.gypi (right):

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi#newc...
build/grit_action.gypi:19: 'grit_resource_ids%':
'GRIT_DIR/../gritsettings/resource_ids',
On 2012/11/01 23:29:15, tony wrote:
> Do you need the %? I thought that meant you could override a value from the
> command line.

No, grit substitutes GRIT_DIR internally.

[tony, 2012-11-01 23:36:51]

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi
File build/grit_action.gypi (right):

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi#newc...
build/grit_action.gypi:19: 'grit_resource_ids%':
'GRIT_DIR/../gritsettings/resource_ids',
On 2012/11/01 23:34:38, sergeyu wrote:
> On 2012/11/01 23:29:15, tony wrote:
> > Do you need the %? I thought that meant you could override a value from the
> > command line.
> 
> No, grit substitutes GRIT_DIR internally.

Sorry, I was asking about the % in grit_resource_ids%.

[Sergey Ulanov, 2012-11-01 23:41:44]

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi
File build/grit_action.gypi (right):

http://codereview.chromium.org/11275101/diff/6001/build/grit_action.gypi#newc...
build/grit_action.gypi:19: 'grit_resource_ids%':
'GRIT_DIR/../gritsettings/resource_ids',
On 2012/11/01 23:36:51, tony wrote:
> On 2012/11/01 23:34:38, sergeyu wrote:
> > On 2012/11/01 23:29:15, tony wrote:
> > > Do you need the %? I thought that meant you could override a value from
the
> > > command line.
> > 
> > No, grit substitutes GRIT_DIR internally.
> 
> Sorry, I was asking about the % in grit_resource_ids%.

% means don't override the variable if it's already defined. That allows to
define this value in remoting.gyp, while all other gyp files will be using the
default resource_ids.  remoting.gyp sets grit_resource_ids to be empty,
otherwise grit complains when it finds "first_id" in grd file.

[tony, 2012-11-01 23:45:11]

On 2012/11/01 23:41:44, sergeyu wrote:
> % means don't override the variable if it's already defined. That allows to
> define this value in remoting.gyp, while all other gyp files will be using the
> default resource_ids.  remoting.gyp sets grit_resource_ids to be empty,
> otherwise grit complains when it finds "first_id" in grd file.

Ah, I see.  Thanks for explaining.

[commit-bot: I haz the power, 2012-11-01 23:48:53]

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

[commit-bot: I haz the power, 2012-11-02 02:54:03]

Change committed as 165605

[Lei Zhang, 2012-11-02 03:33:39]

On 2012/11/02 02:54:03, I haz the power (commit-bot) wrote:
> Change committed as 165605

Broke the build:

http://build.chromium.org/p/chromium.win/builders/Win%20Builder/builds/2896

http://build.chromium.org/p/chromium.win/builders/Win%20Builder%20%28dbg%29/b...

[Sergey Ulanov, 2012-11-03 00:39:14]

Will need to land a grid bugfix before I can land this CL:
http://codereview.chromium.org/11312072/

[Sergey Ulanov, 2012-11-05 19:28:08]

CL that introduced this bug was reverted
(http://codereview.chromium.org/11150008). So I combined it with this CL and
will land them together.

[commit-bot: I haz the power, 2012-11-06 00:09:52]

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

[commit-bot: I haz the power, 2012-11-06 03:39:24]

Change committed as 166127