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

Side by Side Diff: webrtc/system_wrappers/source/condition_variable_unittest.cc

Issue 1888593004: Delete all use of tick_util.h. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase. Created 4 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 unified diff | Download patch
« no previous file with comments | « webrtc/system_wrappers/source/clock.cc ('k') | webrtc/system_wrappers/source/tick_util.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) 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 // TODO(tommi): Remove completely. As is there is still some code for Windows 11 // TODO(tommi): Remove completely. As is there is still some code for Windows
12 // that relies on ConditionVariableEventWin, but code has been removed on other 12 // that relies on ConditionVariableEventWin, but code has been removed on other
13 // platforms. 13 // platforms.
14 #if defined(WEBRTC_WIN) 14 #if defined(WEBRTC_WIN)
15 15
16 #include "webrtc/system_wrappers/source/condition_variable_event_win.h" 16 #include "webrtc/system_wrappers/source/condition_variable_event_win.h"
17 17
18 #include "testing/gtest/include/gtest/gtest.h" 18 #include "testing/gtest/include/gtest/gtest.h"
19 #include "webrtc/base/platform_thread.h" 19 #include "webrtc/base/platform_thread.h"
20 #include "webrtc/base/timeutils.h"
20 #include "webrtc/system_wrappers/include/critical_section_wrapper.h" 21 #include "webrtc/system_wrappers/include/critical_section_wrapper.h"
21 #include "webrtc/system_wrappers/include/tick_util.h"
22 #include "webrtc/system_wrappers/include/trace.h" 22 #include "webrtc/system_wrappers/include/trace.h"
23 23
24 namespace webrtc { 24 namespace webrtc {
25 25
26 namespace { 26 namespace {
27 27
28 const int kLongWaitMs = 100 * 1000; // A long time in testing terms 28 const int kLongWaitMs = 100 * 1000; // A long time in testing terms
29 const int kShortWaitMs = 2 * 1000; // Long enough for process switches to happen 29 const int kShortWaitMs = 2 * 1000; // Long enough for process switches to happen
30 const int kVeryShortWaitMs = 20; // Used when we want a timeout 30 const int kVeryShortWaitMs = 20; // Used when we want a timeout
31 31
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 ASSERT_TRUE(baton_.Grab(kShortWaitMs)); 184 ASSERT_TRUE(baton_.Grab(kShortWaitMs));
185 } 185 }
186 EXPECT_EQ(2 * kNumberOfRounds, baton_.PassCount()); 186 EXPECT_EQ(2 * kNumberOfRounds, baton_.PassCount());
187 } 187 }
188 188
189 TEST(CondVarWaitTest, WaitingWaits) { 189 TEST(CondVarWaitTest, WaitingWaits) {
190 CRITICAL_SECTION crit_sect; 190 CRITICAL_SECTION crit_sect;
191 InitializeCriticalSection(&crit_sect); 191 InitializeCriticalSection(&crit_sect);
192 ConditionVariableEventWin cond_var; 192 ConditionVariableEventWin cond_var;
193 EnterCriticalSection(&crit_sect); 193 EnterCriticalSection(&crit_sect);
194 int64_t start_ms = TickTime::MillisecondTimestamp(); 194 int64_t start_ms = rtc::TimeMillis();
195 EXPECT_FALSE(cond_var.SleepCS(&crit_sect, kVeryShortWaitMs)); 195 EXPECT_FALSE(cond_var.SleepCS(&crit_sect, kVeryShortWaitMs));
196 int64_t end_ms = TickTime::MillisecondTimestamp(); 196 int64_t end_ms = rtc::TimeMillis();
197 EXPECT_LE(start_ms + kVeryShortWaitMs, end_ms) 197 EXPECT_LE(start_ms + kVeryShortWaitMs, end_ms)
198 << "actual elapsed:" << end_ms - start_ms; 198 << "actual elapsed:" << end_ms - start_ms;
199 LeaveCriticalSection(&crit_sect); 199 LeaveCriticalSection(&crit_sect);
200 DeleteCriticalSection(&crit_sect); 200 DeleteCriticalSection(&crit_sect);
201 } 201 }
202 202
203 } // anonymous namespace 203 } // anonymous namespace
204 204
205 } // namespace webrtc 205 } // namespace webrtc
206 206
207 #endif // defined(WEBRTC_WIN) 207 #endif // defined(WEBRTC_WIN)
OLDNEW
« no previous file with comments | « webrtc/system_wrappers/source/clock.cc ('k') | webrtc/system_wrappers/source/tick_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698