Fix group datadeps

Fix group datadeps

NOTRY=true

Committed: https://crrev.com/1b668a2e9177b3482ddfbc9bd3714e980b7cc5d1
Cr-Commit-Position: refs/heads/master@{#293952}

[cjhopman, 2014-09-08 01:03:47]

cjhopman@chromium.org changed reviewers:
+ brettw@chromium.org

[cjhopman, 2014-09-08 01:03:47]

brettw: *

[tfarina, 2014-09-08 01:56:35]

Is there a test covering this?

[brettw, 2014-09-08 20:05:48]

I am trying to write tests for the bugs that come up now, since the original
code didn't have many. Can you make a group writer unit test (you can copy the
structure and setup from the ninja_action_target_writer_unittest) and add deps
and data deps?

[cjhopman, 2014-09-08 23:27:07]

On 2014/09/08 20:05:48, brettw wrote:
> I am trying to write tests for the bugs that come up now, since the original
> code didn't have many. Can you make a group writer unit test (you can copy the
> structure and setup from the ninja_action_target_writer_unittest) and add deps
> and data deps?

I've added a simple test for this.

[brettw, 2014-09-08 23:29:03]

Please add the new file to the GYP build

LGTM

[cjhopman, 2014-09-08 23:30:50]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-09-08 23:35:47]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/cjhopman@chromium.org/551703002/60001

[commit-bot: I haz the power, 2014-09-09 06:24:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-09 06:25:00]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[cjhopman, 2014-09-09 16:23:11]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-09-09 16:30:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/cjhopman@chromium.org/551703002/60001

[commit-bot: I haz the power, 2014-09-09 16:52:26]

Committed patchset #4 (id:60001) as 60acb3fc288f204009218788b1df6fcdf1a93821

[commit-bot: I haz the power, 2014-09-10 03:53:49]

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