Enable whitelisting for targets other than libchrome.

Refactor chrome resource targets into chrome_paks.gni

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

BUG=641032
Committed: https://crrev.com/33312c3d3dc09f27325e74644b346123b325891e
Cr-Commit-Position: refs/heads/master@{#418966}

[agrieve, 2016-09-13 01:40:14]

Description was changed from

==========
Enable whitelisting for targets other than libchrome.

BUG=
==========

to

==========
Enable whitelisting for targets other than libchrome.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[agrieve, 2016-09-13 01:41:02]

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

[commit-bot: I haz the power, 2016-09-13 01:41:17]

Dry run: 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-09-13 01:45:51]

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

[commit-bot: I haz the power, 2016-09-13 01:45:51]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[agrieve, 2016-09-13 17:11:11]

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

[commit-bot: I haz the power, 2016-09-13 17:11:51]

Dry run: 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-09-13 17:19:20]

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

[commit-bot: I haz the power, 2016-09-13 17:19:20]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[agrieve, 2016-09-15 02:10:14]

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

[commit-bot: I haz the power, 2016-09-15 02:10:42]

Dry run: 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-09-15 02:13:22]

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

[commit-bot: I haz the power, 2016-09-15 02:13:23]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[agrieve, 2016-09-15 02:18:37]

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

[commit-bot: I haz the power, 2016-09-15 02:18:51]

Dry run: 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-09-15 02:21:23]

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

[commit-bot: I haz the power, 2016-09-15 02:21:24]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[agrieve, 2016-09-15 15:00:16]

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

[commit-bot: I haz the power, 2016-09-15 15:00:29]

Dry run: 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-09-15 16:00:35]

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

[commit-bot: I haz the power, 2016-09-15 16:00:36]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[agrieve, 2016-09-15 19:19:46]

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

[commit-bot: I haz the power, 2016-09-15 19:20:11]

Dry run: 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-09-15 19:25:57]

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

[commit-bot: I haz the power, 2016-09-15 19:25:58]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[agrieve, 2016-09-15 19:39:20]

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

[commit-bot: I haz the power, 2016-09-15 19:39:42]

Dry run: 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-09-15 19:52:43]

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

[commit-bot: I haz the power, 2016-09-15 19:52:44]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[agrieve, 2016-09-15 19:59:29]

Description was changed from

==========
Enable whitelisting for targets other than libchrome.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Refactor chrome resource targets into chrome_paks.gni

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

BUG=641032
==========

[agrieve, 2016-09-15 19:59:29]

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

[agrieve, 2016-09-15 19:59:37]

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

[commit-bot: I haz the power, 2016-09-15 19:59:57]

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

[agrieve, 2016-09-15 19:59:57]

On 2016/09/15 19:59:37, agrieve wrote:
> The CQ bit was checked by mailto:agrieve@chromium.org to run a CQ dry run

One last pak file shuffle :)

[brettw, 2016-09-15 20:18:42]

rs lgtm

[agrieve, 2016-09-15 20:25:30]

The CQ bit was unchecked by agrieve@chromium.org

[agrieve, 2016-09-15 20:25:35]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-09-15 20:25:53]

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-09-15 20:54:43]

Description was changed from

==========
Refactor chrome resource targets into chrome_paks.gni

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

BUG=641032
==========

to

==========
Refactor chrome resource targets into chrome_paks.gni

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

BUG=641032
==========

[commit-bot: I haz the power, 2016-09-15 20:54:45]

Committed patchset #8 (id:40002)

[commit-bot: I haz the power, 2016-09-15 20:57:42]

Description was changed from

==========
Refactor chrome resource targets into chrome_paks.gni

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

BUG=641032
==========

to

==========
Refactor chrome resource targets into chrome_paks.gni

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

BUG=641032

Committed: https://crrev.com/33312c3d3dc09f27325e74644b346123b325891e
Cr-Commit-Position: refs/heads/master@{#418966}
==========

[commit-bot: I haz the power, 2016-09-15 20:57:43]

Patchset 8 (id:??) landed as
https://crrev.com/33312c3d3dc09f27325e74644b346123b325891e
Cr-Commit-Position: refs/heads/master@{#418966}

[miu, 2016-09-15 21:44:54]

A revert of this CL (patchset #8 id:40002) has been created in
https://codereview.chromium.org/2348643002/ by miu@chromium.org.

The reason for reverting is: Broke ninja on Linux x64:

ninja: Entering directory `out/Release'
[1/1] Regenerating ninja files
FAILED: build.ninja 
/mnt/ssd/miu/chromium/src/buildtools/linux64/gn --root=/mnt/ssd/miu/chromium/src
-q gen //out/Release/
ERROR at //chrome/BUILD.gn:1358:5: Undefined identifier.
    public_deps += [ ":default_apps" ]
    ^----------
See //BUILD.gn:211:7: which caused the file to be included.
      "//chrome",
      ^---------
ninja: error: rebuilding 'build.ninja': subcommand failed
.

[agrieve, 2016-09-16 00:17:00]

A revert of this CL (patchset #8 id:40002) has been created in
https://codereview.chromium.org/2340383003/ by agrieve@chromium.org.

The reason for reverting is: will update when fixed.