Reland dd7edfa67: Switch Mac over to BoringSSL from NSS.

Reland dd7edfa67: Switch Mac over to BoringSSL from NSS.

This is a reland of dd7edfa67 which was reverted in 80a9a88f4c for build
failures. It also includes a BoringSSL roll and build changes to avoid adding a
static initializer.

This is a much much larger change than the diff suggests. If it breaks
something, please revert first and ask questions later.

BUG=338885
TBR=brettw

Committed: https://crrev.com/71c13e6ac01d5cdc49ca40e8e9301d5142d85018
Cr-Commit-Position: refs/heads/master@{#294589}

[davidben, 2014-09-12 00:31:27]

davidben@chromium.org changed reviewers:
+ brettw@chromium.org, rsleevi@chromium.org

[davidben, 2014-09-12 00:31:28]

Will give this another go tomorrow morning once all the try jobs clear. The
static initializer should be gone. And, try bots willing, there should be some
64-bit Mac in there, it won't explode immediately next time.

agl: FYI

https://codereview.chromium.org/568643003/diff/1/crypto/openssl_util.cc
File crypto/openssl_util.cc (left):

https://codereview.chromium.org/568643003/diff/1/crypto/openssl_util.cc#oldco...
crypto/openssl_util.cc:53: OpenSSL_add_all_algorithms();
Removing this line because it's #define-d to SSL_library_init. Pretty sure it
was redundant in upstream OpenSSL too.

[Ryan Sleevi, 2014-09-12 00:36:27]

lgtm

https://codereview.chromium.org/568643003/diff/1/third_party/boringssl/BUILD.gn
File third_party/boringssl/BUILD.gn (right):

https://codereview.chromium.org/568643003/diff/1/third_party/boringssl/BUILD....
third_party/boringssl/BUILD.gn:36: "BORINGSSL_NO_STATIC_INITIALIZER",
Just to confirm I'm reading https://boringssl-review.googlesource.com/#/c/1770/
right, this _only_ affects the compilation, and is not visible in any of the
headers, right? (Seems to be)

[davidben, 2014-09-12 00:40:00]

https://codereview.chromium.org/568643003/diff/1/third_party/boringssl/BUILD.gn
File third_party/boringssl/BUILD.gn (right):

https://codereview.chromium.org/568643003/diff/1/third_party/boringssl/BUILD....
third_party/boringssl/BUILD.gn:36: "BORINGSSL_NO_STATIC_INITIALIZER",
On 2014/09/12 00:36:27, Ryan Sleevi wrote:
> Just to confirm I'm reading
https://boringssl-review.googlesource.com/#/c/1770/
> right, this _only_ affects the compilation, and is not visible in any of the
> headers, right? (Seems to be)

Yup. The only reference to that #define in a header file is in a comment.

[davidben, 2014-09-12 14:20:54]

(Moving brettw to TBR since the build/ changes are exactly the same as the
original CL.)

[davidben, 2014-09-12 14:22:57]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2014-09-12 14:23:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/568643003/1

[commit-bot: I haz the power, 2014-09-12 15:12:25]

Committed patchset #1 (id:1) as 60f7406eba53ce0cca97f89b23e3a15a977b7cb3

[commit-bot: I haz the power, 2014-09-12 15:15:13]

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