Refactor inspector protocol compiles.

Refactor inspector protocol compiles.

This removes the duplication of outputs from core/inspector:protocol_sources to
the compiled version in core:core_generated. A new target is created
core/inspector:protocol that uses get_target_outputs on :protocol_sources to
avoid duplicating the sources list.

To make this easier to implement, the core_config_add and core_config_remove
configs list were moved to core.gni so it can be shared with multiple core
build files.

The inspector_protocol.gni file was changed from listing the sources to
defining a template to run the script. This elimiates some duplicated code
between the two users of this script.

BUG=

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

[brettw, 2016-08-30 22:59:41]

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

[brettw, 2016-08-30 22:59:41]

The context is that the core:core_generate target should no longer list all of
the generated sources (which requires multiple places to keep them in sync).
This is the first (and biggest) step of that.

[brettw, 2016-08-30 22:59:47]

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

[commit-bot: I haz the power, 2016-08-30 23:00:14]

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-08-30 23:37:11]

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

[commit-bot: I haz the power, 2016-08-30 23:37:12]

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

[brettw, 2016-08-31 04:10:24]

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

[commit-bot: I haz the power, 2016-08-31 04: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-08-31 06:18:22]

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

[commit-bot: I haz the power, 2016-08-31 06:18:23]

Dry run: This issue passed the CQ dry run.

[brettw, 2016-08-31 21:39:05]

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

[brettw, 2016-08-31 21:39:06]

-> dpranke

[Dirk Pranke, 2016-08-31 22:01:05]

lgtm

[brettw, 2016-08-31 22:12:18]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-08-31 22:12:59]

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-08-31 22:18:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-31 22:20:05]

Description was changed from

==========
Refactor inspector protocol compiles.

This removes the duplication of outputs from core/inspector:protocol_sources to
the compiled version in core:core_generated. A new target is created
core/inspector:protocol that uses get_target_outputs on :protocol_sources to
avoid duplicating the sources list.

To make this easier to implement, the core_config_add and core_config_remove
configs list were moved to core.gni so it can be shared with multiple core
build files.

The inspector_protocol.gni file was changed from listing the sources to
defining a template to run the script. This elimiates some duplicated code
between the two users of this script.

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

to

==========
Refactor inspector protocol compiles.

This removes the duplication of outputs from core/inspector:protocol_sources to
the compiled version in core:core_generated. A new target is created
core/inspector:protocol that uses get_target_outputs on :protocol_sources to
avoid duplicating the sources list.

To make this easier to implement, the core_config_add and core_config_remove
configs list were moved to core.gni so it can be shared with multiple core
build files.

The inspector_protocol.gni file was changed from listing the sources to
defining a template to run the script. This elimiates some duplicated code
between the two users of this script.

BUG=

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

[commit-bot: I haz the power, 2016-08-31 22:20:06]

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