Remove CertCacheTrial (DiskBasedCertCache).

Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
Committed: https://crrev.com/f9f21c101b486ff42c813fe48971c9909fac6a8e
Cr-Commit-Position: refs/heads/master@{#371313}

[martijnc, 2016-01-17 21:53:39]

Description was changed from

==========
Remove CertCacheTrial (DiskBasedCertCache).

BUG=522312
==========

to

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
==========

[martijnc, 2016-01-17 21:53:39]

martijn@martijnc.be changed reviewers:
+ mmenke@chromium.org

[martijnc, 2016-01-18 21:26:33]

This is all the CertCacheTrial related code I could find. Can you take a look?
Thanks!

(I'm not sure if you are the best reviewer for this but the original reviewers
don't appear to be active anymore.)

[mmenke, 2016-01-19 05:33:12]

mmenke@chromium.org changed reviewers:
+ rsleevi@chromium.org

[mmenke, 2016-01-19 05:33:12]

I don't know this code.

[+rsleevi]:  Ryan, you commented on the bug.  Can you review, or should it go to
a cache person?

[Ryan Sleevi, 2016-01-19 19:20:55]

On 2016/01/19 05:33:12, mmenke wrote:
> I don't know this code.
> 
> [+rsleevi]:  Ryan, you commented on the bug.  Can you review, or should it go
to
> a cache person?

I can review; I'll keep this in my queue and try to get around to it later
today.

[Ryan Sleevi, 2016-01-22 23:57:36]

Sorry it took a bit longer than I expected.

This LGTM. One thing I was double checking on is whether we needed to explicitly
doom the cert: entries associated here, but on evaluation, it should be fine and
they'll DOOM off the LRU or if the user clears history.

So LG, thanks for doing this.

[martijnc, 2016-01-23 12:23:26]

martijn@martijnc.be changed reviewers:
+ isherman@chromium.org

[martijnc, 2016-01-23 12:26:03]

Thank you for taking a look.

+isherman: Can you review the histogram.xml changes?

[mmenke, 2016-01-25 17:48:59]

Description was changed from

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
==========

to

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
==========

[mmenke, 2016-01-25 17:48:59]

mmenke@chromium.org changed reviewers:
- mmenke@chromium.org

[Ilya Sherman, 2016-01-25 19:36:42]

histograms.xml lgtm -- thanks!

[martijnc, 2016-01-25 19:38:22]

The CQ bit was checked by martijn@martijnc.be

[commit-bot: I haz the power, 2016-01-25 19:38:57]

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

[commit-bot: I haz the power, 2016-01-25 21:11:11]

Description was changed from

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
==========

to

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
==========

[commit-bot: I haz the power, 2016-01-25 21:11:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-25 21:12:39]

Description was changed from

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312
==========

to

==========
Remove CertCacheTrial (DiskBasedCertCache).

Removes CertCacheTrial related code that was added in:
https://codereview.chromium.org/329733002
https://codereview.chromium.org/356953003

BUG=522312

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

[commit-bot: I haz the power, 2016-01-25 21:12:40]

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