Switch remaining scoped_openssl_types uses to BoringSSL scopers.

Switch remaining scoped_openssl_types uses to BoringSSL scopers.

BUG=654143
Committed: https://crrev.com/f165962d15779b07d42d5ecefd304c494a459f1c
Cr-Commit-Position: refs/heads/master@{#424523}

[davidben, 2016-10-11 04:09:13]

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

[commit-bot: I haz the power, 2016-10-11 04:09:27]

Dry run: 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, 2016-10-11 04:57:00]

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

[commit-bot: I haz the power, 2016-10-11 04:57:01]

Dry run: This issue passed the CQ dry run.

[davidben, 2016-10-11 18:58:17]

davidben@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org, sgurun@chromium.org, tedchoc@chromium.org

[davidben, 2016-10-11 18:58:18]

These are all pretty trivial changes.

sgurun for android_webview
tedchoc for chrome/browser/ui/android
rdevlin.cronin for */extensions/*

https://codereview.chromium.org/2408063002/diff/1/chrome/browser/extensions/a...
File
chrome/browser/extensions/api/certificate_provider/certificate_provider_apitest.cc
(right):

https://codereview.chromium.org/2408063002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/certificate_provider/certificate_provider_apitest.cc:84:
RSA* rsa_key = EVP_PKEY_get0_RSA(key->key());
get0 is the same as get1 but doesn't take an unnecessary refcount.

https://codereview.chromium.org/2408063002/diff/1/chrome/common/extensions/ap...
File
chrome/common/extensions/api/networking_private/networking_private_crypto.cc
(right):

https://codereview.chromium.org/2408063002/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/networking_private/networking_private_crypto.cc:168:
RSA* rsa = EVP_PKEY_get0_RSA(private_key->key());
get0 is the same as get1 but doesn't take an unnecessary reference.

[sgurun-gerrit only, 2016-10-11 20:02:52]

On 2016/10/11 18:58:18, davidben wrote:
> These are all pretty trivial changes.
> 
> sgurun for android_webview
> tedchoc for chrome/browser/ui/android
> rdevlin.cronin for */extensions/*
> 
>
https://codereview.chromium.org/2408063002/diff/1/chrome/browser/extensions/a...
> File
>
chrome/browser/extensions/api/certificate_provider/certificate_provider_apitest.cc
> (right):
> 
>
https://codereview.chromium.org/2408063002/diff/1/chrome/browser/extensions/a...
>
chrome/browser/extensions/api/certificate_provider/certificate_provider_apitest.cc:84:
> RSA* rsa_key = EVP_PKEY_get0_RSA(key->key());
> get0 is the same as get1 but doesn't take an unnecessary refcount.
> 
>
https://codereview.chromium.org/2408063002/diff/1/chrome/common/extensions/ap...
> File
> chrome/common/extensions/api/networking_private/networking_private_crypto.cc
> (right):
> 
>
https://codereview.chromium.org/2408063002/diff/1/chrome/common/extensions/ap...
>
chrome/common/extensions/api/networking_private/networking_private_crypto.cc:168:
> RSA* rsa = EVP_PKEY_get0_RSA(private_key->key());
> get0 is the same as get1 but doesn't take an unnecessary reference.

aw lgtm

[Ted C, 2016-10-11 20:05:25]

On 2016/10/11 20:02:52, sgurun wrote:
> On 2016/10/11 18:58:18, davidben wrote:
> > These are all pretty trivial changes.
> > 
> > sgurun for android_webview
> > tedchoc for chrome/browser/ui/android
> > rdevlin.cronin for */extensions/*
> > 
> >
>
https://codereview.chromium.org/2408063002/diff/1/chrome/browser/extensions/a...
> > File
> >
>
chrome/browser/extensions/api/certificate_provider/certificate_provider_apitest.cc
> > (right):
> > 
> >
>
https://codereview.chromium.org/2408063002/diff/1/chrome/browser/extensions/a...
> >
>
chrome/browser/extensions/api/certificate_provider/certificate_provider_apitest.cc:84:
> > RSA* rsa_key = EVP_PKEY_get0_RSA(key->key());
> > get0 is the same as get1 but doesn't take an unnecessary refcount.
> > 
> >
>
https://codereview.chromium.org/2408063002/diff/1/chrome/common/extensions/ap...
> > File
> > chrome/common/extensions/api/networking_private/networking_private_crypto.cc
> > (right):
> > 
> >
>
https://codereview.chromium.org/2408063002/diff/1/chrome/common/extensions/ap...
> >
>
chrome/common/extensions/api/networking_private/networking_private_crypto.cc:168:
> > RSA* rsa = EVP_PKEY_get0_RSA(private_key->key());
> > get0 is the same as get1 but doesn't take an unnecessary reference.
> 
> aw lgtm

chrome/browser/ui/android - lgtm

[Devlin, 2016-10-11 20:10:05]

extensions lgtm

[davidben, 2016-10-11 20:11:20]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2016-10-11 20:11:58]

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, 2016-10-11 20:16:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-11 20:19:11]

Description was changed from

==========
Switch remaining scoped_openssl_types uses to BoringSSL scopers.

BUG=654143
==========

to

==========
Switch remaining scoped_openssl_types uses to BoringSSL scopers.

BUG=654143

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

[commit-bot: I haz the power, 2016-10-11 20:19:13]

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