Try #2: Convert android to use X509CertificateBytes instead of X509CertificateOpenSSL.

Try #2: Convert android to use X509CertificateBytes instead of X509CertificateOpenSSL.

BUG=671420
TBR=davidben@chromium.org

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

[mattm, 2017-05-01 23:51:12]

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

[commit-bot: I haz the power, 2017-05-01 23:51:53]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mattm, 2017-05-01 23:52:41]

Description was changed from

==========
Convert android to use X509CertificateBytes instead of X509CertificateOpenSSL.

BUG=671420
==========

to

==========
Try #2: Convert android to use X509CertificateBytes instead of
X509CertificateOpenSSL.

BUG=671420
==========

[mattm, 2017-05-01 23:54:30]

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

[mattm, 2017-05-01 23:54:31]

The change is even smaller this time due to your changes
https://codereview.chromium.org/2824713002 and
https://codereview.chromium.org/2816363002.

Perf bots should hopefully be happy this time (see
https://bugs.chromium.org/p/chromium/issues/detail?id=711634#c16)

[commit-bot: I haz the power, 2017-05-02 01:10:41]

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

[commit-bot: I haz the power, 2017-05-02 01:10:41]

Dry run: This issue passed the CQ dry run.

[mattm, 2017-05-02 01:11:35]

Description was changed from

==========
Try #2: Convert android to use X509CertificateBytes instead of
X509CertificateOpenSSL.

BUG=671420
==========

to

==========
Try #2: Convert android to use X509CertificateBytes instead of
X509CertificateOpenSSL.

BUG=671420
TBR=davidben@chromium.org
==========

[mattm, 2017-05-02 01:12:45]

(TBRing since this is just a reland of
https://codereview.chromium.org/2786173003 minus some changes that are no longer
necessary)

[mattm, 2017-05-02 01:15:07]

The CQ bit was checked by mattm@chromium.org

[commit-bot: I haz the power, 2017-05-02 01:15:44]

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-05-02 01:20:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493687707066630,
"parent_rev": "37235f0ea4c66711ae8cc694978bfb98657dd0c6", "commit_rev":
"e05a6fdeff3d39f64b4af1d1468251602e9ccc6b"}

[commit-bot: I haz the power, 2017-05-02 01:20:18]

Description was changed from

==========
Try #2: Convert android to use X509CertificateBytes instead of
X509CertificateOpenSSL.

BUG=671420
TBR=davidben@chromium.org
==========

to

==========
Try #2: Convert android to use X509CertificateBytes instead of
X509CertificateOpenSSL.

BUG=671420
TBR=davidben@chromium.org

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

[commit-bot: I haz the power, 2017-05-02 01:20:19]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/e05a6fdeff3d39f64b4af1d14682...

[davidben, 2017-05-02 06:08:18]

lgtm