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

Side by Side Diff: webrtc/base/physicalsocketserver_unittest.cc

Issue 2111043004: Enabling IPv6 socket recv timestamp test, and making less flaky. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 5 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/base/socket.h » ('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 2004 The WebRTC Project Authors. All rights reserved. 2 * Copyright 2004 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
(...skipping 368 matching lines...) Expand 10 before | Expand all | Expand 10 after
379 TEST_F(PhysicalSocketTest, TestGetSetOptionsIPv4) { 379 TEST_F(PhysicalSocketTest, TestGetSetOptionsIPv4) {
380 SocketTest::TestGetSetOptionsIPv4(); 380 SocketTest::TestGetSetOptionsIPv4();
381 } 381 }
382 382
383 TEST_F(PhysicalSocketTest, TestGetSetOptionsIPv6) { 383 TEST_F(PhysicalSocketTest, TestGetSetOptionsIPv6) {
384 SocketTest::TestGetSetOptionsIPv6(); 384 SocketTest::TestGetSetOptionsIPv6();
385 } 385 }
386 386
387 #if defined(WEBRTC_POSIX) 387 #if defined(WEBRTC_POSIX)
388 388
389 // We don't get recv timestamps on Mac.
389 #if !defined(WEBRTC_MAC) 390 #if !defined(WEBRTC_MAC)
390 TEST_F(PhysicalSocketTest, TestSocketRecvTimestampIPv4) { 391 TEST_F(PhysicalSocketTest, TestSocketRecvTimestampIPv4) {
391 SocketTest::TestSocketRecvTimestamp(); 392 SocketTest::TestSocketRecvTimestampIPv4();
392 } 393 }
393 394
394 #if defined(WEBRTC_LINUX) 395 TEST_F(PhysicalSocketTest, TestSocketRecvTimestampIPv6) {
395 #define MAYBE_TestSocketRecvTimestampIPv6 DISABLED_TestSocketRecvTimestampIPv6 396 SocketTest::TestSocketRecvTimestampIPv6();
396 #else
397 #define MAYBE_TestSocketRecvTimestampIPv6 TestSocketRecvTimestampIPv6
398 #endif
399 TEST_F(PhysicalSocketTest, MAYBE_TestSocketRecvTimestampIPv6) {
400 SocketTest::TestSocketRecvTimestamp();
401 } 397 }
402 #endif 398 #endif
403 399
404 class PosixSignalDeliveryTest : public testing::Test { 400 class PosixSignalDeliveryTest : public testing::Test {
405 public: 401 public:
406 static void RecordSignal(int signum) { 402 static void RecordSignal(int signum) {
407 signals_received_.push_back(signum); 403 signals_received_.push_back(signum);
408 signaled_thread_ = Thread::Current(); 404 signaled_thread_ = Thread::Current();
409 } 405 }
410 406
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
518 thread->Start(runnable.get()); 514 thread->Start(runnable.get());
519 EXPECT_TRUE(ss_->Wait(1500, true)); 515 EXPECT_TRUE(ss_->Wait(1500, true));
520 EXPECT_TRUE(ExpectSignal(SIGTERM)); 516 EXPECT_TRUE(ExpectSignal(SIGTERM));
521 EXPECT_EQ(Thread::Current(), signaled_thread_); 517 EXPECT_EQ(Thread::Current(), signaled_thread_);
522 EXPECT_TRUE(ExpectNone()); 518 EXPECT_TRUE(ExpectNone());
523 } 519 }
524 520
525 #endif 521 #endif
526 522
527 } // namespace rtc 523 } // namespace rtc
OLDNEW
« no previous file with comments | « no previous file | webrtc/base/socket.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698