Owner for CRL/OCSP histograms

Owner for CRL/OCSP histograms

TBR=isherman

Committed: https://crrev.com/d105ce2fac1dddf3c6794ab0f3f28668e6074430
Cr-Commit-Position: refs/heads/master@{#302043}

[cbentzel, 2014-09-09 17:08:36]

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

[cbentzel, 2014-09-09 17:09:13]

I was thinking about just obsoleting this (beyond the OCSP stapling one) but it
turns out that we are still getting a number of samples for this on iOS as well
as ChromeOS.

[Ryan Sleevi, 2014-09-09 18:29:19]

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

[Ryan Sleevi, 2014-09-09 18:29:19]

These are all going to be obsoleted on iOS soon. felt knows the timing of this -
I think M39? (Felt, this is the EV bits)

Otherwise, we'd only get these numbers when enterprise policy forced OCSP/CRLs
on, which I'm fine removing.

[felt, 2014-09-10 02:18:52]

On 2014/09/09 18:29:19, Ryan Sleevi wrote:
> These are all going to be obsoleted on iOS soon. felt knows the timing of this
-
> I think M39? (Felt, this is the EV bits)
> 
> Otherwise, we'd only get these numbers when enterprise policy forced OCSP/CRLs
> on, which I'm fine removing.

This is being tracked as crbug.com/409055, I think it will land for 39.

[cbentzel, 2014-09-12 19:04:42]

On 2014/09/10 02:18:52, felt wrote:
> On 2014/09/09 18:29:19, Ryan Sleevi wrote:
> > These are all going to be obsoleted on iOS soon. felt knows the timing of
this
> -
> > I think M39? (Felt, this is the EV bits)
> > 
> > Otherwise, we'd only get these numbers when enterprise policy forced
OCSP/CRLs
> > on, which I'm fine removing.
> 
> This is being tracked as crbug.com/409055, I think it will land for 39.

Should we just obsolete these histograms except the OCSP stapling frequency one
then?

[cbentzel, 2014-09-29 15:53:49]

On 2014/09/12 19:04:42, cbentzel wrote:
> On 2014/09/10 02:18:52, felt wrote:
> > On 2014/09/09 18:29:19, Ryan Sleevi wrote:
> > > These are all going to be obsoleted on iOS soon. felt knows the timing of
> this
> > -
> > > I think M39? (Felt, this is the EV bits)
> > > 
> > > Otherwise, we'd only get these numbers when enterprise policy forced
> OCSP/CRLs
> > > on, which I'm fine removing.
> > 
> > This is being tracked as crbug.com/409055, I think it will land for 39.
> 
> Should we just obsolete these histograms except the OCSP stapling frequency
one
> then?

Ping. Any thoughts on this?

[felt, 2014-09-29 16:47:59]

On 2014/09/29 15:53:49, cbentzel wrote:
> On 2014/09/12 19:04:42, cbentzel wrote:
> > On 2014/09/10 02:18:52, felt wrote:
> > > On 2014/09/09 18:29:19, Ryan Sleevi wrote:
> > > > These are all going to be obsoleted on iOS soon. felt knows the timing
of
> > this
> > > -
> > > > I think M39? (Felt, this is the EV bits)
> > > > 
> > > > Otherwise, we'd only get these numbers when enterprise policy forced
> > OCSP/CRLs
> > > > on, which I'm fine removing.
> > > 
> > > This is being tracked as crbug.com/409055, I think it will land for 39.
> > 
> > Should we just obsolete these histograms except the OCSP stapling frequency
> one
> > then?
> 
> Ping. Any thoughts on this?

Deprecating them SGTM

[Ryan Sleevi, 2014-09-29 18:50:13]

Sorry, thought I responded on the internal thread.

Yeah, let's deprecate.

[Ryan Sleevi, 2014-10-27 21:59:41]

Chris, do you want to still own this? Shall we CQ and then deprecate? Just
trying to clear review queue

[cbentzel, 2014-10-29 12:24:21]

On 2014/10/27 21:59:41, Ryan Sleevi wrote:
> Chris, do you want to still own this? Shall we CQ and then deprecate? Just
> trying to clear review queue

I'll still own. CQ this makes sense. Just got a new computer and need to re-set
it up for Chrome builds.

[cbentzel, 2014-10-29 12:24:24]

The CQ bit was checked by cbentzel@chromium.org

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

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

[commit-bot: I haz the power, 2014-10-29 12:25:11]

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

[commit-bot: I haz the power, 2014-10-29 12:25:12]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Ryan Sleevi, 2014-10-30 01:31:33]

lgtm

[Ryan Sleevi, 2014-10-30 01:31:57]

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

[Ryan Sleevi, 2014-10-30 01:32:10]

TBRing this to isherman because it's just updating ownership

[Ryan Sleevi, 2014-10-30 01:32:14]

The CQ bit was checked by rsleevi@chromium.org

[commit-bot: I haz the power, 2014-10-30 01:33:33]

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

[Ilya Sherman, 2014-10-30 01:38:15]

LGTM, thanks.

[commit-bot: I haz the power, 2014-10-30 06:16:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-30 06:17:38]

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