Remove Net.AuthGenerateToken histograms from histograms.xml

Remove Net.AuthGenerateToken histograms from histograms.xml

These were removed from the code in 2011 as seen in https://crrev.com/70740

Committed: https://crrev.com/18fd2272929ce63818c4bc8c486dea994830c940
Cr-Commit-Position: refs/heads/master@{#293996}

[cbentzel, 2014-09-09 17:23:12]

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

[asanka, 2014-09-09 17:56:48]

lgtm

[cbentzel, 2014-09-09 18:01:39]

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

[cbentzel, 2014-09-09 18:01:40]

+asvitkine

[Alexei Svitkine (slow), 2014-09-09 18:17:09]

lgtm % comment

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

https://codereview.chromium.org/553083002/diff/1/tools/metrics/histograms/his...
tools/metrics/histograms/histograms.xml:14113: -  <owner>Please list the
metric's owners. Add more owner tags as needed.</owner>
Please restore these lines, since we have scripts check for these.

[cbentzel, 2014-09-09 18:25:32]

On 2014/09/09 18:17:09, Alexei Svitkine wrote:
> lgtm % comment
> 
>
https://codereview.chromium.org/553083002/diff/1/tools/metrics/histograms/his...
> File tools/metrics/histograms/histograms.xml (left):
> 
>
https://codereview.chromium.org/553083002/diff/1/tools/metrics/histograms/his...
> tools/metrics/histograms/histograms.xml:14113: -  <owner>Please list the
> metric's owners. Add more owner tags as needed.</owner>
> Please restore these lines, since we have scripts check for these.

Ah - you still want those even for obsolete Histograms?

[Alexei Svitkine (slow), 2014-09-09 18:28:09]

Yes, please.

On Tue, Sep 9, 2014 at 2:25 PM, <cbentzel@chromium.org> wrote:

> On 2014/09/09 18:17:09, Alexei Svitkine wrote:
>
>> lgtm % comment
>>
>
>
> https://codereview.chromium.org/553083002/diff/1/tools/
> metrics/histograms/histograms.xml
>
>> File tools/metrics/histograms/histograms.xml (left):
>>
>
>
> https://codereview.chromium.org/553083002/diff/1/tools/
> metrics/histograms/histograms.xml#oldcode14113
>
>> tools/metrics/histograms/histograms.xml:14113: -  <owner>Please list the
>> metric's owners. Add more owner tags as needed.</owner>
>> Please restore these lines, since we have scripts check for these.
>>
>
> Ah - you still want those even for obsolete Histograms?
>
> https://codereview.chromium.org/553083002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[cbentzel, 2014-09-09 18:31:24]

On 2014/09/09 18:28:09, Alexei Svitkine wrote:
> Yes, please.

Done, thanks.

> 
> On Tue, Sep 9, 2014 at 2:25 PM, <mailto:cbentzel@chromium.org> wrote:
> 
> > On 2014/09/09 18:17:09, Alexei Svitkine wrote:
> >
> >> lgtm % comment
> >>
> >
> >
> > https://codereview.chromium.org/553083002/diff/1/tools/
> > metrics/histograms/histograms.xml
> >
> >> File tools/metrics/histograms/histograms.xml (left):
> >>
> >
> >
> > https://codereview.chromium.org/553083002/diff/1/tools/
> > metrics/histograms/histograms.xml#oldcode14113
> >
> >> tools/metrics/histograms/histograms.xml:14113: -  <owner>Please list the
> >> metric's owners. Add more owner tags as needed.</owner>
> >> Please restore these lines, since we have scripts check for these.
> >>
> >
> > Ah - you still want those even for obsolete Histograms?
> >
> > https://codereview.chromium.org/553083002/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

[cbentzel, 2014-09-09 18:31:30]

The CQ bit was checked by cbentzel@chromium.org

[commit-bot: I haz the power, 2014-09-09 18:39:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/cbentzel@chromium.org/553083002/20001

[commit-bot: I haz the power, 2014-09-09 20:07:57]

Committed patchset #2 (id:20001) as a9ad2916b5f54b51cdec603bbc32263477924071

[commit-bot: I haz the power, 2014-09-10 03:55:12]

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