Add net public_deps in content/public/common:common_sources, and the followon public_deps in conten…

Add net public_deps in content/public/common:common_sources, and the followon public_deps in content/public/browser:browser_sources.

BUG=none

Review-Url: https://codereview.chromium.org/2797093002
Cr-Commit-Position: refs/heads/master@{#462244}
Committed: https://chromium.googlesource.com/chromium/src/+/ac7d5f39fa14c9b0f980a498c5ced6a79b77d113

[mattm, 2017-04-05 03:18:21]

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

[commit-bot: I haz the power, 2017-04-05 03:18:39]

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

[mattm, 2017-04-05 03:20:25]

mattm@chromium.org changed reviewers:
+ piman@chromium.org

[mattm, 2017-04-05 03:20:25]

content/public/common/referrer.h includes net headers, therefore //net should be
in content/public/common:common_sources public_deps.


content/public/browser/navigation_controller.h includes
content/public/common/referrer.h,
therefore //content/public/common:common_sources should be in
content/public/browser:browser_sources public_deps.



Example error (assumes you change net/cert/x509_certificate.h to #include
"third_party/boringssl/src/include/openssl/base.h"):

In file included from ../../net/cert/x509_certificate.h:25:0,
                 from ../../net/ssl/ssl_config.h:12,
                 from ../../net/ssl/ssl_info.h:20,
                 from ../../net/http/http_response_info.h:14,
                 from ../../net/base/net_error_details.h:9,
                 from ../../net/url_request/url_request.h:26,
                 from ../../content/public/common/referrer.h:10,
                 from ../../content/public/browser/navigation_controller.h:24,
                 from ../../content/public/browser/web_contents_observer.h:15,
                 from ../../components/pdf/browser/pdf_web_contents_helper.h:13,
                 from ../../components/pdf/browser/pdf_web_contents_helper.cc:5:
../../third_party/boringssl/src/include/openssl/base.h:68:33: fatal error:
openssl/opensslconf.h: No such file or directory
 #include <openssl/opensslconf.h>

[commit-bot: I haz the power, 2017-04-05 04:27:24]

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

[commit-bot: I haz the power, 2017-04-05 04:27:25]

Dry run: This issue passed the CQ dry run.

[piman, 2017-04-05 19:15:56]

lgtm

[mattm, 2017-04-05 22:13:28]

The CQ bit was checked by mattm@chromium.org

[commit-bot: I haz the power, 2017-04-05 22:14:27]

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, 2017-04-05 22:23:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491430408812070, "parent_rev":
"d927e98fb228fb205e54152165e238432372df85", "commit_rev":
"ac7d5f39fa14c9b0f980a498c5ced6a79b77d113"}

[commit-bot: I haz the power, 2017-04-05 22:24:46]

Description was changed from

==========
Add net public_deps in content/public/common:common_sources, and the followon
public_deps in content/public/browser:browser_sources.

BUG=none
==========

to

==========
Add net public_deps in content/public/common:common_sources, and the followon
public_deps in content/public/browser:browser_sources.

BUG=none

Review-Url: https://codereview.chromium.org/2797093002
Cr-Commit-Position: refs/heads/master@{#462244}
Committed:
https://chromium.googlesource.com/chromium/src/+/ac7d5f39fa14c9b0f980a498c5ce...
==========

[commit-bot: I haz the power, 2017-04-05 22:24:47]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ac7d5f39fa14c9b0f980a498c5ce...