Remove Sparse Histograms from Blink Platform API.

Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012
Committed: https://crrev.com/5119494478729af6b78cc4525136a4422acadf3b
Cr-Commit-Position: refs/heads/master@{#372830}

[dtapuska, 2016-02-01 18:39:00]

Description was changed from

==========
Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012
==========

to

==========
Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012
==========

[dtapuska, 2016-02-01 18:39:00]

dtapuska@chromium.org changed reviewers:
+ creis@chromium.org, esprehn@chromium.org

[dtapuska, 2016-02-01 20:38:31]

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

[Charlie Reis, 2016-02-01 21:24:40]

content/ LGTM.

[Alexei Svitkine (slow), 2016-02-01 21:30:31]

https://codereview.chromium.org/1653043002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/1653043002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/animation/css/CSSAnimations.cpp:497:
propertyHistogram.sample(UseCounter::mapCSSPropertyIdToCSSSampleIdForHistogram(id));
Perhaps there should be a macro that wraps the extra boilerplate?

[esprehn, 2016-02-01 23:49:27]

On 2016/02/01 at 21:30:31, asvitkine wrote:
>
https://codereview.chromium.org/1653043002/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/animation/css/CSSAnimations.cpp (right):
> 
>
https://codereview.chromium.org/1653043002/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/animation/css/CSSAnimations.cpp:497:
propertyHistogram.sample(UseCounter::mapCSSPropertyIdToCSSSampleIdForHistogram(id));
> Perhaps there should be a macro that wraps the extra boilerplate?

We've been trying to remove most custom macros so the code is more obvious. Lets
start here, if we find it's too painful we can add them back.

[esprehn, 2016-02-01 23:49:56]

The CQ bit was checked by esprehn@chromium.org

[esprehn, 2016-02-01 23:49:57]

lgtm

[commit-bot: I haz the power, 2016-02-01 23:52:50]

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

[commit-bot: I haz the power, 2016-02-02 00:21:41]

Description was changed from

==========
Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012
==========

to

==========
Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012
==========

[commit-bot: I haz the power, 2016-02-02 00:21:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-02 00:22:40]

Description was changed from

==========
Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012
==========

to

==========
Remove Sparse Histograms from Blink Platform API.

Use the newly created API (in https://codereview.chromium.org/1647883004/)
to create sparse histograms and remove the API from the blink platform.

BUG=583012

Committed: https://crrev.com/5119494478729af6b78cc4525136a4422acadf3b
Cr-Commit-Position: refs/heads/master@{#372830}
==========

[commit-bot: I haz the power, 2016-02-02 00:22:40]

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