First attempt at cleaning up chrome_resources.gyp.

First attempt at cleaning up chrome_resources.gyp.

This first prefix action_names with 'generate' to differentiate them
from the target names.

This also collapses theme_resources_gen into theme_resources as there is
no reason to separate them.

It also removes a second declaration of grit_out_dir variable. We don't need to declare it twice in the same file.

BUG=None
TEST=None
R=thestig@chromium.org
TBR=brettw

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

[tfarina, 2014-05-17 04:04:36]

Lei, I'm trying to make more reason of this file. To make it more hackable.

I have been seeing a lot of mistakes made at other top-level directories (ash,
components, extensions and ui) because they tried to copy the logic in here and
there was many subtle details that was not get right.

In a follow up I'll try to come with better target names. I just didn't come to
anything about them yet (i.e., packed_extra_resources, packed_resources, etc...,
these are not very meaningful names imo :/).

[Lei Zhang, 2014-05-19 22:10:20]

lgtm

+oshima in case he has any concerns/comments

[tfarina, 2014-05-19 22:14:53]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-05-19 22:15:54]

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

[oshima, 2014-05-19 22:23:37]

lgtm

[commit-bot: I haz the power, 2014-05-20 01:43:04]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-05-20 06:15:30]

Change committed as 271579