Fix tracing_resources grit/generated file usage

Fix tracing_resources grit/generated file usage

This CL fixes two problems in tracing_resources.gyp:

(1) Multiple targets generate files that are then consumed by other targets but
    don't set themselves as hard dependencies.
(2) The target processing the grd file doesn't have the resource_ids file as an
    input, so it is not rerun when that file changes.

These problems were discovered in the course of shifting the location of some
resources in the tree.

Committed: https://crrev.com/cff7ec49d42148d19e67c614bcfff8e36f9592c9
Cr-Commit-Position: refs/heads/master@{#362786}

[blundell, 2015-11-30 12:56:11]

blundell@chromium.org changed reviewers:
+ dsinclair@chromium.org

[blundell, 2015-12-01 20:27:23]

ping

[dsinclair, 2015-12-01 23:47:21]

I'm ooo until next week.

Dan

On Tue, Dec 1, 2015, 15:27  <blundell@chromium.org> wrote:

> ping
>
> https://codereview.chromium.org/1485633002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[blundell, 2015-12-02 06:58:19]

blundell@chromium.org changed reviewers:
+ nduca@chromium.org, oysteine@chromium.org, simonhatch@chromium.org

[blundell, 2015-12-02 06:58:19]

Thanks (something some people do that works well is to reflect their OOO status
in their Chromium nickname).

Adding the other OWNERS of //content/browser/tracing in a shotgun approach --
whoever gets to it first! It's IMO a pretty straightforward fix.

[nduca, 2015-12-02 17:37:40]

lgtm

[blundell, 2015-12-02 19:27:43]

The CQ bit was checked by blundell@chromium.org

[commit-bot: I haz the power, 2015-12-02 19:29:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1485633002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1485633002/1

[commit-bot: I haz the power, 2015-12-02 20:44:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-02 20:45:55]

Description was changed from

==========
Fix tracing_resources grit/generated file usage

This CL fixes two problems in tracing_resources.gyp:

(1) Multiple targets generate files that are then consumed by other targets but
    don't set themselves as hard dependencies.
(2) The target processing the grd file doesn't have the resource_ids file as an
    input, so it is not rerun when that file changes.

These problems were discovered in the course of shifting the location of some
resources in the tree.
==========

to

==========
Fix tracing_resources grit/generated file usage

This CL fixes two problems in tracing_resources.gyp:

(1) Multiple targets generate files that are then consumed by other targets but
    don't set themselves as hard dependencies.
(2) The target processing the grd file doesn't have the resource_ids file as an
    input, so it is not rerun when that file changes.

These problems were discovered in the course of shifting the location of some
resources in the tree.

Committed: https://crrev.com/cff7ec49d42148d19e67c614bcfff8e36f9592c9
Cr-Commit-Position: refs/heads/master@{#362786}
==========

[commit-bot: I haz the power, 2015-12-02 20:45:56]

Patchset 1 (id:??) landed as
https://crrev.com/cff7ec49d42148d19e67c614bcfff8e36f9592c9
Cr-Commit-Position: refs/heads/master@{#362786}