Add //ui/* to "gn check"

Add most of //ui/ to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc. as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to define IPC traits, without actually linking in all of IPC (because in this case events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should export the corresponding regular target as a public dep so tests don't have to depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests

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

[brettw, 2015-12-05 01:07:29]

Description was changed from

==========
Add //ui/* to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

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

to

==========
Add //ui/* to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests




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

[brettw, 2015-12-05 01:07:29]

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

[brettw, 2015-12-05 01:07:35]

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

[commit-bot: I haz the power, 2015-12-05 01:08:30]

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

[commit-bot: I haz the power, 2015-12-05 01:18:44]

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

[commit-bot: I haz the power, 2015-12-05 01:18:45]

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

[brettw, 2015-12-07 19:24:13]

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

[commit-bot: I haz the power, 2015-12-07 19:25:52]

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

[commit-bot: I haz the power, 2015-12-07 19:46:43]

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

[commit-bot: I haz the power, 2015-12-07 19:46:43]

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

[brettw, 2015-12-07 20:19:09]

Description was changed from

==========
Add //ui/* to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests




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

to

==========
Add most of //ui/ to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests
==========

[brettw, 2015-12-07 20:19:45]

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

[commit-bot: I haz the power, 2015-12-07 20:20:50]

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

[sky, 2015-12-07 22:32:20]

LGTM

[brettw, 2015-12-07 22:34:49]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2015-12-07 22:34:57]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-12-07 22:40:52]

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

[brettw, 2015-12-08 01:16:16]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2015-12-08 01:16:19]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-12-08 01:29:27]

Description was changed from

==========
Add most of //ui/ to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests
==========

to

==========
Add most of //ui/ to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests
==========

[commit-bot: I haz the power, 2015-12-08 01:29:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-08 01:30:22]

Description was changed from

==========
Add most of //ui/ to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests
==========

to

==========
Add most of //ui/ to "gn check"

Fixes various missing dependencies.

In static builds, the //content target now exports content/public/browser, etc.
as public dependencies.

Add an ipc_param_traits target that allows targets like events/latency_info.h to
define IPC traits, without actually linking in all of IPC (because in this case
events has no use of IPC).

Move some targets to public deps. The rule is that test_support targets should
export the corresponding regular target as a public dep so tests don't have to
depend on both.

Mark a few conditional includes "// nogncheck" where the checker got confused.

Fix manifest for views_unittests

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

[commit-bot: I haz the power, 2015-12-08 01:30:23]

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