GN: Fix ordering of public and all dep configs

GN: Fix ordering of public and all dep configs.

Previously GN iterated through the deps and added the all_dependent followed by the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed by all of the public_configs. This patch brings the ordering in line with the documentation and adds a test.

BUG=474000
Committed: https://crrev.com/47e6f464f5f85e3ad0b4bc21b570735e2fcfba7d
Cr-Commit-Position: refs/heads/master@{#405257}

[brettw, 2016-07-12 22:46:06]

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

[brettw, 2016-07-12 22:46:24]

The CQ bit was checked by brettw@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 22:48:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[brettw, 2016-07-12 22:49:33]

Description was changed from

==========
GN: Fix ordering of public and all dep configs in

Previously GN iterated through the deps and added the all_dependent followed by
the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed
by all of the public_configs. This patch brings the ordering in line with the
documentation and adds a test.

BUG=474000
==========

to

==========
GN: Fix ordering of public and all dep configs.

Previously GN iterated through the deps and added the all_dependent followed by
the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed
by all of the public_configs. This patch brings the ordering in line with the
documentation and adds a test.

BUG=474000
==========

[commit-bot: I haz the power, 2016-07-13 02:11:35]

The CQ bit was unchecked by commit-bot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-13 02:11:35]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)

[Dirk Pranke, 2016-07-13 17:22:33]

lgtm

[brettw, 2016-07-13 19:39:59]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-07-13 19:40:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 20:10:53]

Description was changed from

==========
GN: Fix ordering of public and all dep configs.

Previously GN iterated through the deps and added the all_dependent followed by
the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed
by all of the public_configs. This patch brings the ordering in line with the
documentation and adds a test.

BUG=474000
==========

to

==========
GN: Fix ordering of public and all dep configs.

Previously GN iterated through the deps and added the all_dependent followed by
the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed
by all of the public_configs. This patch brings the ordering in line with the
documentation and adds a test.

BUG=474000
==========

[commit-bot: I haz the power, 2016-07-13 20:10:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-13 20:11:01]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 20:12:16]

Description was changed from

==========
GN: Fix ordering of public and all dep configs.

Previously GN iterated through the deps and added the all_dependent followed by
the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed
by all of the public_configs. This patch brings the ordering in line with the
documentation and adds a test.

BUG=474000
==========

to

==========
GN: Fix ordering of public and all dep configs.

Previously GN iterated through the deps and added the all_dependent followed by
the public configs for each target.

But the ordering is documented to be all of the all_dependent_configs, followed
by all of the public_configs. This patch brings the ordering in line with the
documentation and adds a test.

BUG=474000

Committed: https://crrev.com/47e6f464f5f85e3ad0b4bc21b570735e2fcfba7d
Cr-Commit-Position: refs/heads/master@{#405257}
==========

[commit-bot: I haz the power, 2016-07-13 20:12:17]

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