Add iOS support to the chrome/ resource gyp/gypi files

Add iOS support to the chrome/ resource gyp/gypi files

Excludes resources that aren't used on iOS.

BUG=None


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

[stuartmorgan, 2012-10-17 11:37:39]

Let me know if you'd rather I find another owner for these kinds of changes; I
just figured you're used to reviewing iOS gyp changes at this point :)

[Nico, 2012-10-17 17:06:28]

LGTM

I don't mind reviewing stuff like this. Gives me at least some idea about what's
happening in chrome/ios land :-)

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp
File chrome/chrome_resources.gyp (left):

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp#o...
chrome/chrome_resources.gyp:57: 'variables': {
You have about:sync-internals on ios?

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp
File chrome/chrome_resources.gyp (right):

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp#n...
chrome/chrome_resources.gyp:113: 'grit_grd_file':
'renderer/resources/renderer_resources.grd',
Do you need these? You don't have a renderer process after all.

[stuartmorgan, 2012-10-17 20:35:24]

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp
File chrome/chrome_resources.gyp (left):

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp#o...
chrome/chrome_resources.gyp:57: 'variables': {
On 2012/10/17 17:06:28, Nico wrote:
> You have about:sync-internals on ios?

We do, yes. We recently got WebUI working, and expect to bring more (e.g.,
net_internals) online over time.

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp
File chrome/chrome_resources.gyp (right):

https://codereview.chromium.org/11189031/diff/1/chrome/chrome_resources.gyp#n...
chrome/chrome_resources.gyp:113: 'grit_grd_file':
'renderer/resources/renderer_resources.grd',
On 2012/10/17 17:06:28, Nico wrote:
> Do you need these? You don't have a renderer process after all.

Temporarily, yes; we use a network error page resource that didn't get moved out
of this grd when we moved the error page code out of chrome/renderer, which was
an oversight that I found while doing this CL. I already have an internal bug
filed about it, and we'll fix it soon, but I didn't want it to block merging our
fork of this file on that. Once that's in I'll exclude this too.

[commit-bot: I haz the power, 2012-10-17 20:36:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stuartmorgan@chromium.org/11189031/1

[commit-bot: I haz the power, 2012-10-17 21:57:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stuartmorgan@chromium.org/11189031/15001

[commit-bot: I haz the power, 2012-10-17 23:58:08]

Change committed as 162575