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

Side by Side Diff: webrtc/modules/audio_processing/level_estimator_impl.cc

Issue 1424663003: Lock scheme #8: Introduced the new locking scheme (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@add_threadcheckers_CL
Patch Set: Created 5 years, 1 month 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #include "webrtc/modules/audio_processing/level_estimator_impl.h" 11 #include "webrtc/modules/audio_processing/level_estimator_impl.h"
12 12
13 #include "webrtc/modules/audio_processing/audio_buffer.h" 13 #include "webrtc/modules/audio_processing/audio_buffer.h"
14 #include "webrtc/modules/audio_processing/include/audio_processing.h" 14 #include "webrtc/modules/audio_processing/include/audio_processing.h"
15 #include "webrtc/modules/audio_processing/rms_level.h" 15 #include "webrtc/modules/audio_processing/rms_level.h"
16 #include "webrtc/system_wrappers/interface/critical_section_wrapper.h" 16 #include "webrtc/system_wrappers/interface/critical_section_wrapper.h"
17 17
18 namespace webrtc { 18 namespace webrtc {
19 19
20 LevelEstimatorImpl::LevelEstimatorImpl(const AudioProcessing* apm, 20 LevelEstimatorImpl::LevelEstimatorImpl(const AudioProcessing* apm)
21 CriticalSectionWrapper* crit) 21 : ProcessingComponent() {}
22 : ProcessingComponent(),
23 crit_(crit) {}
24 22
25 LevelEstimatorImpl::~LevelEstimatorImpl() {} 23 LevelEstimatorImpl::~LevelEstimatorImpl() {}
26 24
27 int LevelEstimatorImpl::ProcessStream(AudioBuffer* audio) { 25 int LevelEstimatorImpl::ProcessStream(AudioBuffer* audio) {
28 if (!is_component_enabled()) { 26 if (!is_component_enabled()) {
29 return AudioProcessing::kNoError; 27 return AudioProcessing::kNoError;
30 } 28 }
31 29
32 RMSLevel* rms_level = static_cast<RMSLevel*>(handle(0)); 30 RMSLevel* rms_level = static_cast<RMSLevel*>(handle(0));
33 for (int i = 0; i < audio->num_channels(); ++i) { 31 for (int i = 0; i < audio->num_channels(); ++i) {
34 rms_level->Process(audio->channels_const()[i], 32 rms_level->Process(audio->channels_const()[i],
35 audio->num_frames()); 33 audio->num_frames());
36 } 34 }
37 35
38 return AudioProcessing::kNoError; 36 return AudioProcessing::kNoError;
39 } 37 }
40 38
41 int LevelEstimatorImpl::Enable(bool enable) { 39 int LevelEstimatorImpl::Enable(bool enable) {
42 CriticalSectionScoped crit_scoped(crit_);
43 return EnableComponent(enable); 40 return EnableComponent(enable);
44 } 41 }
45 42
46 bool LevelEstimatorImpl::is_enabled() const { 43 bool LevelEstimatorImpl::is_enabled() const {
47 return is_component_enabled(); 44 return is_component_enabled();
48 } 45 }
49 46
50 int LevelEstimatorImpl::RMS() { 47 int LevelEstimatorImpl::RMS() {
51 if (!is_component_enabled()) { 48 if (!is_component_enabled()) {
52 return AudioProcessing::kNotEnabledError; 49 return AudioProcessing::kNotEnabledError;
(...skipping 24 matching lines...) Expand all
77 74
78 int LevelEstimatorImpl::num_handles_required() const { 75 int LevelEstimatorImpl::num_handles_required() const {
79 return 1; 76 return 1;
80 } 77 }
81 78
82 int LevelEstimatorImpl::GetHandleError(void* /*handle*/) const { 79 int LevelEstimatorImpl::GetHandleError(void* /*handle*/) const {
83 return AudioProcessing::kUnspecifiedError; 80 return AudioProcessing::kUnspecifiedError;
84 } 81 }
85 82
86 } // namespace webrtc 83 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698