Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1092)

Unified Diff: content/renderer/media/webrtc_uma_histograms.cc

Issue 2909103002: Deprecate NonThreadSafe in content/renderer/media in favor of SequenceChecker. (Closed)
Patch Set: fix compile Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/media/webrtc_uma_histograms.cc
diff --git a/content/renderer/media/webrtc_uma_histograms.cc b/content/renderer/media/webrtc_uma_histograms.cc
index e2db071d5e66aa31f4f29dd038660c8fbc60da22..26a004418c1b1d548f3ee57396831790c4edf21c 100644
--- a/content/renderer/media/webrtc_uma_histograms.cc
+++ b/content/renderer/media/webrtc_uma_histograms.cc
@@ -26,6 +26,7 @@ void UpdateWebRTCMethodCount(JavaScriptAPIName api_name) {
}
PerSessionWebRTCAPIMetrics::~PerSessionWebRTCAPIMetrics() {
+ DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
}
// static
@@ -34,12 +35,12 @@ PerSessionWebRTCAPIMetrics* PerSessionWebRTCAPIMetrics::GetInstance() {
}
void PerSessionWebRTCAPIMetrics::IncrementStreamCounter() {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
++num_streams_;
}
void PerSessionWebRTCAPIMetrics::DecrementStreamCounter() {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
if (--num_streams_ == 0) {
ResetUsage();
}
@@ -56,7 +57,7 @@ void PerSessionWebRTCAPIMetrics::LogUsage(JavaScriptAPIName api_name) {
}
void PerSessionWebRTCAPIMetrics::LogUsageOnlyOnce(JavaScriptAPIName api_name) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
if (!has_used_api_[api_name]) {
has_used_api_[api_name] = true;
LogUsage(api_name);

Powered by Google App Engine
This is Rietveld 408576698