Update gn_all and gyp_remaining to reflect reality on the win config.

Update gn_all and gyp_remaining to reflect reality on the win config.

R=brettw@chromium.org
BUG=354261
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel

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

[Dirk Pranke, 2015-04-09 03:01:02]

note that this depends on the initial gn_all/gyp_remaining patch for win that
was reverted. It will also need to be merged w/ the chromeos work in
https://codereview.chromium.org/1069903002/ .

[Dirk Pranke, 2015-04-11 00:30:21]

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

[Dirk Pranke, 2015-04-11 00:56:37]

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

[commit-bot: I haz the power, 2015-04-11 00:57:06]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1072653005/40001

[commit-bot: I haz the power, 2015-04-11 01:37:43]

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

[commit-bot: I haz the power, 2015-04-11 01:37:44]

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

[Dirk Pranke, 2015-04-11 01:51:32]

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

[commit-bot: I haz the power, 2015-04-11 01:52:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1072653005/60001

[commit-bot: I haz the power, 2015-04-11 02:01:27]

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

[commit-bot: I haz the power, 2015-04-11 02:01:28]

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

[Dirk Pranke, 2015-04-13 21:28:41]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2015-04-13 21:29:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1072653005/80001

[commit-bot: I haz the power, 2015-04-13 21:29:37]

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

[commit-bot: I haz the power, 2015-04-13 21:29:38]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[brettw, 2015-04-13 21:31:45]

rs lgtm

https://codereview.chromium.org/1072653005/diff/80001/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1072653005/diff/80001/BUILD.gn#newcode11
BUILD.gn:11: import("//build/config/crypto.gni")
Long-term, this structure is unfortunate since it means all of these files have
to be loaded in series. This is fine for now but we should try to have a better
system in the future that avoids making this list longer.

[Dirk Pranke, 2015-04-13 21:33:02]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2015-04-13 21:34:26]

On 2015/04/13 21:31:45, brettw wrote:
> rs lgtm
> 
> https://codereview.chromium.org/1072653005/diff/80001/BUILD.gn
> File BUILD.gn (right):
> 
> https://codereview.chromium.org/1072653005/diff/80001/BUILD.gn#newcode11
> BUILD.gn:11: import("//build/config/crypto.gni")
> Long-term, this structure is unfortunate since it means all of these files
have
> to be loaded in series. This is fine for now but we should try to have a
better
> system in the future that avoids making this list longer.

Yes, agreed. The only option I see at the moment is to add various 'crypto_all'
targets that 
exist unconditionally in the other directories. Which is fine, if somewhat
unfortunate, since 
all of the logic for what "all" builds would no longer be in one place. 

I don't see any other great alternatives, though.

[commit-bot: I haz the power, 2015-04-13 21:34:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1072653005/80001

[commit-bot: I haz the power, 2015-04-13 23:56:12]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-04-13 23:58:07]

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