Remove control character cookie UMA stats.

Remove control character cookie UMA stats.

BUG=238041
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224583

[jww, 2013-09-20 16:37:00]

We are done collecting the UMA stats on cookie parsing for
http://crbug.com/238041 so this is a CL to remove them.

[Ilya Sherman, 2013-09-20 17:42:45]

https://codereview.chromium.org/23503084/diff/1/tools/metrics/histograms/hist...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/23503084/diff/1/tools/metrics/histograms/hist...
tools/metrics/histograms/histograms.xml:1393: -</histogram>
Please mark the histograms as <obsolete> rather than removing them outright. 
Among other things, this helps prevent folks from accidentally re-using these
same histogram names for unrelated logging down the line.

[jww, 2013-09-20 17:56:26]

https://codereview.chromium.org/23503084/diff/1/tools/metrics/histograms/hist...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/23503084/diff/1/tools/metrics/histograms/hist...
tools/metrics/histograms/histograms.xml:1393: -</histogram>
On 2013/09/20 17:42:46, Ilya Sherman wrote:
> Please mark the histograms as <obsolete> rather than removing them outright. 
> Among other things, this helps prevent folks from accidentally re-using these
> same histogram names for unrelated logging down the line.

Done.

[Ilya Sherman, 2013-09-20 18:03:07]

Thanks, histograms.xml lgtm w/ nit:

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
tools/metrics/histograms/histograms.xml:19085: +  </obsolete>
Nit: No need to mark enums as obsolete, especially for potentially reusable ones
like this one.

[jww, 2013-09-20 18:08:56]

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
tools/metrics/histograms/histograms.xml:19085: +  </obsolete>
On 2013/09/20 18:03:08, Ilya Sherman wrote:
> Nit: No need to mark enums as obsolete, especially for potentially reusable
ones
> like this one.
Okay. What about the much more specific one below, ParsedCookieStatus? I assume
I should leave it marked obsolete?

[Ilya Sherman, 2013-09-20 18:10:45]

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
tools/metrics/histograms/histograms.xml:19085: +  </obsolete>
On 2013/09/20 18:08:57, jww wrote:
> On 2013/09/20 18:03:08, Ilya Sherman wrote:
> > Nit: No need to mark enums as obsolete, especially for potentially reusable
> ones
> > like this one.
> Okay. What about the much more specific one below, ParsedCookieStatus? I
assume
> I should leave it marked obsolete?

Either way is fine.  <obsolete> only has an observable impact for <histogram>s
on the dashboard, but if you think it's useful documentation for the enum, I'm
fine with adding it there as well.

[jww, 2013-09-20 18:12:45]

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/23503084/diff/4001/tools/metrics/histograms/h...
tools/metrics/histograms/histograms.xml:19085: +  </obsolete>
On 2013/09/20 18:10:45, Ilya Sherman wrote:
> On 2013/09/20 18:08:57, jww wrote:
> > On 2013/09/20 18:03:08, Ilya Sherman wrote:
> > > Nit: No need to mark enums as obsolete, especially for potentially
reusable
> > ones
> > > like this one.
> > Okay. What about the much more specific one below, ParsedCookieStatus? I
> assume
> > I should leave it marked obsolete?
> 
> Either way is fine.  <obsolete> only has an observable impact for <histogram>s
> on the dashboard, but if you think it's useful documentation for the enum, I'm
> fine with adding it there as well.
Cool, thanks! I'm going to leave it marked obsolete since it's so incredibly
specific.

[erikwright (departed), 2013-09-20 19:46:00]

LGTM.

[commit-bot: I haz the power, 2013-09-20 19:55:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jww@chromium.org/23503084/11001

[commit-bot: I haz the power, 2013-09-20 23:37:46]

Retried try job too often on win7_aura for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-09-21 00:03:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jww@chromium.org/23503084/11001

[commit-bot: I haz the power, 2013-09-21 16:12:03]

Change committed as 224583