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

Side by Side Diff: webrtc/test/call_test.cc

Issue 2774463003: Don't hardcode MediaType::ANY in FakeNetworkPipe. (Closed)
Patch Set: Update FakeCall::DeliverPacket, for consistency with Call::DeliverRtp. 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 | « webrtc/test/call_test.h ('k') | webrtc/test/direct_transport.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 (c) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 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 440 matching lines...) Expand 10 before | Expand all | Expand 10 after
451 451
452 Call::Config BaseTest::GetReceiverCallConfig() { 452 Call::Config BaseTest::GetReceiverCallConfig() {
453 return Call::Config(&event_log_); 453 return Call::Config(&event_log_);
454 } 454 }
455 455
456 void BaseTest::OnCallsCreated(Call* sender_call, Call* receiver_call) { 456 void BaseTest::OnCallsCreated(Call* sender_call, Call* receiver_call) {
457 } 457 }
458 458
459 test::PacketTransport* BaseTest::CreateSendTransport(Call* sender_call) { 459 test::PacketTransport* BaseTest::CreateSendTransport(Call* sender_call) {
460 return new PacketTransport(sender_call, this, test::PacketTransport::kSender, 460 return new PacketTransport(sender_call, this, test::PacketTransport::kSender,
461 MediaType::VIDEO,
461 FakeNetworkPipe::Config()); 462 FakeNetworkPipe::Config());
462 } 463 }
463 464
464 test::PacketTransport* BaseTest::CreateReceiveTransport() { 465 test::PacketTransport* BaseTest::CreateReceiveTransport() {
465 return new PacketTransport(nullptr, this, test::PacketTransport::kReceiver, 466 return new PacketTransport(nullptr, this, test::PacketTransport::kReceiver,
467 MediaType::VIDEO,
466 FakeNetworkPipe::Config()); 468 FakeNetworkPipe::Config());
467 } 469 }
468 470
469 size_t BaseTest::GetNumVideoStreams() const { 471 size_t BaseTest::GetNumVideoStreams() const {
470 return 1; 472 return 1;
471 } 473 }
472 474
473 size_t BaseTest::GetNumAudioStreams() const { 475 size_t BaseTest::GetNumAudioStreams() const {
474 return 0; 476 return 0;
475 } 477 }
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
523 525
524 EndToEndTest::EndToEndTest(unsigned int timeout_ms) : BaseTest(timeout_ms) { 526 EndToEndTest::EndToEndTest(unsigned int timeout_ms) : BaseTest(timeout_ms) {
525 } 527 }
526 528
527 bool EndToEndTest::ShouldCreateReceivers() const { 529 bool EndToEndTest::ShouldCreateReceivers() const {
528 return true; 530 return true;
529 } 531 }
530 532
531 } // namespace test 533 } // namespace test
532 } // namespace webrtc 534 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/test/call_test.h ('k') | webrtc/test/direct_transport.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698