Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these are for infrequent

Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283
Committed: https://crrev.com/53805324c0fa904d796cc0b333868c591f2c5f2c
Cr-Commit-Position: refs/heads/master@{#11099}

[åsapersson, 2015-12-16 13:25:41]

Patchset #2 (id:20001) has been deleted

[åsapersson, 2015-12-16 14:25:11]

Description was changed from

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE to indicate these are
for infrequent updates.

BUG=webrtc:5283
==========

to

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

BUG=webrtc:5283
==========

[åsapersson, 2015-12-16 14:25:11]

asapersson@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org, pbos@webrtc.org

[hlundin-webrtc, 2015-12-16 16:28:59]

lgtm for webrtc/modules/audio_* and webrtc/modules/bitrate_controller

We should probably reconsider the UMA logging in
webrtc/modules/audio_coding/neteq/statistics_calculator.cc; it may not qualify
as sparse.

BTW: I think you need more reviewers to cover all required OWNERS.

[åsapersson, 2015-12-16 16:35:08]

asapersson@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[pbos-webrtc, 2015-12-18 14:55:52]

lgtm but please put why in the description (that the implementation is slow due
to names changing) and that it's going away, so that sparse shouldn't be used

[åsapersson, 2015-12-18 14:59:39]

Description was changed from

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

BUG=webrtc:5283
==========

to

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be later
removed.

BUG=webrtc:5283
==========

[åsapersson, 2015-12-18 15:00:03]

Description was changed from

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be later
removed.

BUG=webrtc:5283
==========

to

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283
==========

[åsapersson, 2015-12-18 15:05:18]

On 2015/12/18 14:55:52, pbos-webrtc wrote:
> lgtm but please put why in the description (that the implementation is slow
due
> to names changing) and that it's going away, so that sparse shouldn't be used

Done.

Magnus, can you take a look at webrtc/system_wrappers/include/metrics.h

[mflodman, 2015-12-21 08:25:53]

LGTM

[åsapersson, 2015-12-21 08:30:44]

The CQ bit was checked by asapersson@webrtc.org

[commit-bot: I haz the power, 2015-12-21 08:30:50]

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

[commit-bot: I haz the power, 2015-12-21 09:46:24]

Description was changed from

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283
==========

to

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283
==========

[commit-bot: I haz the power, 2015-12-21 09:46:25]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-12-21 09:46:28]

Description was changed from

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283
==========

to

==========
Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these
are for infrequent updates.

This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283

Committed: https://crrev.com/53805324c0fa904d796cc0b333868c591f2c5f2c
Cr-Commit-Position: refs/heads/master@{#11099}
==========

[commit-bot: I haz the power, 2015-12-21 09:46:44]

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