Separate sources that depend on printing into a different group.

Separate sources that depend on printing into a different group.

This new group is only included in the build when enable_basic_printing
is true. Builds for which this variable is false (like Chromecast) will
now pass gn check for the targets including these sources.

BUG=

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

[slan, 2015-10-30 19:25:54]

slan@chromium.org changed reviewers:
+ halliwell@chromium.org, sky@chromium.org

[slan, 2015-10-30 19:25:54]

Scott, this is a structural change to separate sources with printing
dependencies from chrome_common_sources. This allows Cast builds to have a clean
gn check.

Please take a look.

+halliwell@ for visibility

[sky, 2015-10-30 22:04:37]

LGTM

[slan, 2015-10-30 22:06:16]

The CQ bit was checked by slan@chromium.org

[commit-bot: I haz the power, 2015-10-30 22:06:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1408973004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1408973004/1

[commit-bot: I haz the power, 2015-10-30 22:18:16]

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

[commit-bot: I haz the power, 2015-10-30 22:18:17]

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...)

[slan, 2015-11-04 00:02:56]

On 2015/10/30 22:18:17, commit-bot: I haz the power wrote:
> 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...)

Chatted with calamity@, author of the CHECK that is failing on
win_chromium_x64_rel_ng trybot. There is no apparent reason for the failure.
Going to kick off a dry-run to test for flakiness / bad bot state. Sorry for the
churn.

[slan, 2015-11-04 00:03:49]

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

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

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1408973004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1408973004/1

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

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

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

Dry run: This issue passed the CQ dry run.

[slan, 2015-11-04 01:38:16]

slan@chromium.org changed reviewers:
+ dcheng@chromium.org

[slan, 2015-11-04 01:38:17]

+dcheng@ for all_messages.h

Daniel, could you please take a look? gn check does not know how to parse
preprocessors, so we add a comment to ignore the file. Thanks!

[slan, 2015-11-05 23:21:09]

On 2015/11/04 01:38:17, slan wrote:
> +dcheng@ for all_messages.h
> 
> Daniel, could you please take a look? gn check does not know how to parse
> preprocessors, so we add a comment to ignore the file. Thanks!

Friendly ping for dcheng@

[dcheng, 2015-11-05 23:25:17]

lgtm

[slan, 2015-11-06 00:47:21]

The CQ bit was checked by slan@chromium.org

[commit-bot: I haz the power, 2015-11-06 00:48:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1408973004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1408973004/1

[commit-bot: I haz the power, 2015-11-06 01:23:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-06 01:24:31]

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