Add missing public_deps for //content/public/app:xxx GN targets.

Add missing public_deps for //content/public/app:xxx GN targets.

The non-component build version of the //content/public/app:both target
provides the public_app_shared_sources as sources, whereas the component
build version of the same target does not.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content/public/app:xxx for the
component build to make the dependency on //content a public_deps entry.

This includes the targets:
//content/public/app:browser
//content/public/app:child
//content/public/app:both

BUG=597830
Committed: https://crrev.com/ae6d61aae5d8b33110c93d60bbe746722f5f966a
Cr-Commit-Position: refs/heads/master@{#390103}

[nyquist, 2016-04-25 23:29:04]

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

[nyquist, 2016-04-25 23:29:04]

dpranke: PTAL

[nyquist, 2016-04-25 23:36:07]

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

[commit-bot: I haz the power, 2016-04-25 23:36:45]

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

[commit-bot: I haz the power, 2016-04-26 00:39:59]

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

[commit-bot: I haz the power, 2016-04-26 00:40:00]

Dry run: This issue passed the CQ dry run.

[nyquist, 2016-04-26 21:27:39]

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

[nyquist, 2016-04-26 21:27:40]

brettw: PTAL
This is related to the discussion on https://codereview.chromium.org/1922613003

[nyquist, 2016-04-26 21:41:00]

Description was changed from

==========
Add missing public_deps for //content GN target.

The non-component build version of the //content target does not provide
//content/public/app:both as a public_deps, which is different than the
component build, which has //content/public/app:both_sources as
public_deps.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content for !is_component_build to
match the one for the component build.

BUG=597830
==========

to

==========
Add missing public_deps for //content/public/app:xxx GN targets.

The non-component build version of the //content/public/app:both target
provides the public_app_shared_sources as sources, whereas the component
build version of the same target does not.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content/public/app:xxx for the
component build to make the dependency on //content a public_deps entry.

This includes the targets:
//content/public/app:browser
//content/public/app:child
//content/public/app:both

BUG=597830
==========

[nyquist, 2016-04-26 21:41:27]

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

[commit-bot: I haz the power, 2016-04-26 21:41:54]

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

[commit-bot: I haz the power, 2016-04-26 22:36:31]

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

[commit-bot: I haz the power, 2016-04-26 22:36:32]

Dry run: This issue passed the CQ dry run.

[brettw, 2016-04-27 05:23:31]

lgtm

[nyquist, 2016-04-27 16:42:04]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2016-04-27 16:42:25]

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

[commit-bot: I haz the power, 2016-04-27 16:46:51]

Description was changed from

==========
Add missing public_deps for //content/public/app:xxx GN targets.

The non-component build version of the //content/public/app:both target
provides the public_app_shared_sources as sources, whereas the component
build version of the same target does not.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content/public/app:xxx for the
component build to make the dependency on //content a public_deps entry.

This includes the targets:
//content/public/app:browser
//content/public/app:child
//content/public/app:both

BUG=597830
==========

to

==========
Add missing public_deps for //content/public/app:xxx GN targets.

The non-component build version of the //content/public/app:both target
provides the public_app_shared_sources as sources, whereas the component
build version of the same target does not.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content/public/app:xxx for the
component build to make the dependency on //content a public_deps entry.

This includes the targets:
//content/public/app:browser
//content/public/app:child
//content/public/app:both

BUG=597830
==========

[commit-bot: I haz the power, 2016-04-27 16:46:53]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-30 17:10:38]

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

[commit-bot: I haz the power, 2016-04-30 17:50:26]

Description was changed from

==========
Add missing public_deps for //content/public/app:xxx GN targets.

The non-component build version of the //content/public/app:both target
provides the public_app_shared_sources as sources, whereas the component
build version of the same target does not.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content/public/app:xxx for the
component build to make the dependency on //content a public_deps entry.

This includes the targets:
//content/public/app:browser
//content/public/app:child
//content/public/app:both

BUG=597830
==========

to

==========
Add missing public_deps for //content/public/app:xxx GN targets.

The non-component build version of the //content/public/app:both target
provides the public_app_shared_sources as sources, whereas the component
build version of the same target does not.

This causes issues for targets outside //content that need to depend on
//content/public/app/content_main.h and other files in
public_app_shared_sources from //content/public/app/BUILD.gn, because
the non-component build has to be special cased.

This CL fixes the public_deps for //content/public/app:xxx for the
component build to make the dependency on //content a public_deps entry.

This includes the targets:
//content/public/app:browser
//content/public/app:child
//content/public/app:both

BUG=597830

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