[Mac/GN] Collect a bundle's data_deps when using `gn desc runtime_deps`.

[Mac/GN] Collect a bundle's data_deps when using `gn desc runtime_deps`.

In 903e2b85c71675f83767ac9b1687087382b49f39, GN stopped recursing into bundles
to collect target dependencies. But any explicit data_deps on a bundle should be
included in the runtime_deps output.

BUG=611414
R=brettw@chromium.org,sdefresne@chromium.org

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

[sdefresne, 2016-05-28 13:19:00]

non-owners lgtm

[Dirk Pranke, 2016-05-31 18:58:40]

dpranke@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2016-05-31 18:58:40]

lgtm

[Robert Sesek, 2016-05-31 18:59:03]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2016-05-31 18:59:26]

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

[commit-bot: I haz the power, 2016-05-31 20:11:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-31 20:45:03]

Description was changed from

==========
[Mac/GN] Collect a bundle's data_deps when using `gn desc runtime_deps`.

In 903e2b85c71675f83767ac9b1687087382b49f39, GN stopped recursing into bundles
to collect target dependencies. But any explicit data_deps on a bundle should be
included in the runtime_deps output.

BUG=611414
R=brettw@chromium.org,sdefresne@chromium.org
==========

to

==========
[Mac/GN] Collect a bundle's data_deps when using `gn desc runtime_deps`.

In 903e2b85c71675f83767ac9b1687087382b49f39, GN stopped recursing into bundles
to collect target dependencies. But any explicit data_deps on a bundle should be
included in the runtime_deps output.

BUG=611414
R=brettw@chromium.org,sdefresne@chromium.org

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

[commit-bot: I haz the power, 2016-05-31 20:45:04]

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

[Dirk Pranke, 2016-05-31 20:49:22]

On 2016/05/31 20:45:04, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/be4da4cd67c068e11bd226979faf1f578dae5a3f
> Cr-Commit-Position: refs/heads/master@{#396910}

Bah, I realized after the fact that we probably should've updated the docs for
either `create_bundle` or `data_deps` or both to document this, since it's a bit
counter-intuitive otherwise.

Mind doing that in a follow-up CL?

[Robert Sesek, 2016-06-01 18:35:43]

On 2016/05/31 20:49:22, Dirk Pranke wrote:
> Bah, I realized after the fact that we probably should've updated the docs for
> either `create_bundle` or `data_deps` or both to document this, since it's a
bit
> counter-intuitive otherwise.
> 
> Mind doing that in a follow-up CL?

https://codereview.chromium.org/2023003006