Finalize new UseCounter histogram names

Finalize new UseCounter histogram names

The new UseCounter histograms are now working well and the data we're
getting looks solid.  It's time to make them official as the primary
histograms to track.

BUG=676837
Review-Url: https://codereview.chromium.org/2614213002
Cr-Commit-Position: refs/heads/master@{#442115}
Committed: https://chromium.googlesource.com/chromium/src/+/68f6cfe0bc29b69e8de3e77a21809021ba4c11ee

[Rick Byers, 2017-01-06 21:08:20]

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

[commit-bot: I haz the power, 2017-01-06 21:08:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Rick Byers, 2017-01-06 21:10:04]

rbyers@chromium.org changed reviewers:
+ rkaplow@chromium.org

[Rick Byers, 2017-01-06 21:10:04]

rkaplow: PTAL

[rkaplow, 2017-01-06 21:20:44]

lgtm

[Rick Byers, 2017-01-06 21:22:32]

The CQ bit was unchecked by rbyers@chromium.org

[Rick Byers, 2017-01-06 21:22:38]

The CQ bit was checked by rbyers@chromium.org

[commit-bot: I haz the power, 2017-01-06 21:23:01]

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-01-06 22:05:33]

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

[commit-bot: I haz the power, 2017-01-06 22:05:34]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Rick Byers, 2017-01-06 22:52:11]

The CQ bit was checked by rbyers@chromium.org

[commit-bot: I haz the power, 2017-01-06 22:52:47]

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-01-07 00:44:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483743131803850, "parent_rev":
"e6a9f01dbfbc97e63b90de01b329a1b1f8fe122f", "commit_rev":
"68f6cfe0bc29b69e8de3e77a21809021ba4c11ee"}

[commit-bot: I haz the power, 2017-01-07 00:45:29]

Description was changed from

==========
Finalize new UseCounter histogram names

The new UseCounter histograms are now working well and the data we're
getting looks solid.  It's time to make them official as the primary
histograms to track.

BUG=676837
==========

to

==========
Finalize new UseCounter histogram names

The new UseCounter histograms are now working well and the data we're
getting looks solid.  It's time to make them official as the primary
histograms to track.

BUG=676837

Review-Url: https://codereview.chromium.org/2614213002
Cr-Commit-Position: refs/heads/master@{#442115}
Committed:
https://chromium.googlesource.com/chromium/src/+/68f6cfe0bc29b69e8de3e77a2180...
==========

[commit-bot: I haz the power, 2017-01-07 00:45:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/68f6cfe0bc29b69e8de3e77a2180...

[tyoshino (SeeGerritForStatus), 2017-03-27 05:54:07]

tyoshino@chromium.org changed reviewers:
+ tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2017-03-27 05:54:07]

https://codereview.chromium.org/2614213002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/frame/UseCounter.cpp (right):

https://codereview.chromium.org/2614213002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/frame/UseCounter.cpp:1291: * WebCore.UseCounter
This should be also updated?

[Rick Byers, 2017-03-27 16:24:47]

https://codereview.chromium.org/2614213002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/frame/UseCounter.cpp (right):

https://codereview.chromium.org/2614213002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/frame/UseCounter.cpp:1291: * WebCore.UseCounter
On 2017/03/27 05:54:07, tyoshino wrote:
> This should be also updated?

Yep, sorry I missed that.  I'm pretty close to being able to delete that code
entirely though (should happen in the next month or so -
https://crbug.com/676837).  Worth me landing a separate fix for anyway do you
think?

[tyoshino (SeeGerritForStatus), 2017-03-28 06:18:27]

On 2017/03/27 16:24:47, Rick Byers wrote:
>
https://codereview.chromium.org/2614213002/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/frame/UseCounter.cpp (right):
> 
>
https://codereview.chromium.org/2614213002/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/frame/UseCounter.cpp:1291: * WebCore.UseCounter
> On 2017/03/27 05:54:07, tyoshino wrote:
> > This should be also updated?
> 
> Yep, sorry I missed that.  I'm pretty close to being able to delete that code
> entirely though (should happen in the next month or so -
> https://crbug.com/676837).  Worth me landing a separate fix for anyway do you
> think?

Oh, ok. It's totally up to you. I just notified you of that since I noticed it.