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

Side by Side Diff: webrtc/modules/audio_conference_mixer/source/time_scheduler.h

Issue 2785673002: Remove more CriticalSectionWrappers. (Closed)
Patch Set: Created 3 years, 8 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 unified diff | Download patch
« no previous file with comments | « no previous file | webrtc/modules/audio_conference_mixer/source/time_scheduler.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2011 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2011 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 // The TimeScheduler class keeps track of periodic events. It is non-drifting 11 // The TimeScheduler class keeps track of periodic events. It is non-drifting
12 // and keeps track of any missed periods so that it is possible to catch up. 12 // and keeps track of any missed periods so that it is possible to catch up.
13 // (compare to a metronome) 13 // (compare to a metronome)
14 #include "webrtc/base/criticalsection.h"
14 15
15 #ifndef WEBRTC_MODULES_AUDIO_CONFERENCE_MIXER_SOURCE_TIME_SCHEDULER_H_ 16 #ifndef WEBRTC_MODULES_AUDIO_CONFERENCE_MIXER_SOURCE_TIME_SCHEDULER_H_
16 #define WEBRTC_MODULES_AUDIO_CONFERENCE_MIXER_SOURCE_TIME_SCHEDULER_H_ 17 #define WEBRTC_MODULES_AUDIO_CONFERENCE_MIXER_SOURCE_TIME_SCHEDULER_H_
17 18
18 namespace webrtc { 19 namespace webrtc {
19 class CriticalSectionWrapper; 20 class CriticalSectionWrapper;
20 class TimeScheduler 21
21 { 22 class TimeScheduler {
22 public: 23 public:
23 TimeScheduler(const int64_t periodicityInMs); 24 TimeScheduler(const int64_t periodicityInMs);
24 ~TimeScheduler(); 25 ~TimeScheduler() = default;
25 26
26 // Signal that a periodic event has been triggered. 27 // Signal that a periodic event has been triggered.
27 int32_t UpdateScheduler(); 28 int32_t UpdateScheduler();
28 29
29 // Set updateTimeInMs to the amount of time until UpdateScheduler() should 30 // Set updateTimeInMs to the amount of time until UpdateScheduler() should
30 // be called. This time will never be negative. 31 // be called. This time will never be negative.
31 int32_t TimeToNextUpdate(int64_t& updateTimeInMS) const; 32 int32_t TimeToNextUpdate(int64_t& updateTimeInMS) const;
32 33
33 private: 34 private:
34 CriticalSectionWrapper* _crit; 35 rtc::CriticalSection _crit;
35 36
36 bool _isStarted; 37 bool _isStarted;
37 int64_t _lastPeriodMark; // In ns 38 int64_t _lastPeriodMark; // In ns
38 39
39 int64_t _periodicityInMs; 40 int64_t _periodicityInMs;
40 int64_t _periodicityInTicks; 41 int64_t _periodicityInTicks;
41 uint32_t _missedPeriods; 42 uint32_t _missedPeriods;
42 }; 43 };
43 } // namespace webrtc 44 } // namespace webrtc
44 45
45 #endif // WEBRTC_MODULES_AUDIO_CONFERENCE_MIXER_SOURCE_TIME_SCHEDULER_H_ 46 #endif // WEBRTC_MODULES_AUDIO_CONFERENCE_MIXER_SOURCE_TIME_SCHEDULER_H_
OLDNEW
« no previous file with comments | « no previous file | webrtc/modules/audio_conference_mixer/source/time_scheduler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698