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

Side by Side Diff: webrtc/media/base/videoengine_unittest.h

Issue 1670153003: Introduce struct MediaConfig, with construction-time settings. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 10 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
OLDNEW
1 /* 1 /*
2 * libjingle 2 * libjingle
3 * Copyright 2014 Google Inc. 3 * Copyright 2014 Google Inc.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are met: 6 * modification, are permitted provided that the following conditions are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright notice, 8 * 1. Redistributions of source code must retain the above copyright notice,
9 * this list of conditions and the following disclaimer. 9 * this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright notice, 10 * 2. Redistributions in binary form must reproduce the above copyright notice,
(...skipping 24 matching lines...) Expand all
35 #include "webrtc/base/gunit.h" 35 #include "webrtc/base/gunit.h"
36 #include "webrtc/base/timeutils.h" 36 #include "webrtc/base/timeutils.h"
37 #include "webrtc/call.h" 37 #include "webrtc/call.h"
38 #include "webrtc/media/base/fakenetworkinterface.h" 38 #include "webrtc/media/base/fakenetworkinterface.h"
39 #include "webrtc/media/base/fakevideocapturer.h" 39 #include "webrtc/media/base/fakevideocapturer.h"
40 #include "webrtc/media/base/fakevideorenderer.h" 40 #include "webrtc/media/base/fakevideorenderer.h"
41 #include "webrtc/media/base/mediachannel.h" 41 #include "webrtc/media/base/mediachannel.h"
42 #include "webrtc/media/base/streamparams.h" 42 #include "webrtc/media/base/streamparams.h"
43 #include "webrtc/media/webrtc/fakewebrtccall.h" 43 #include "webrtc/media/webrtc/fakewebrtccall.h"
44 44
45 // TODO(nisse): Improper include? Move declaration of MediaConfig elsewhere?
46 #include "talk/app/webrtc/mediacontroller.h"
47
45 #define EXPECT_FRAME_WAIT(c, w, h, t) \ 48 #define EXPECT_FRAME_WAIT(c, w, h, t) \
46 EXPECT_EQ_WAIT((c), renderer_.num_rendered_frames(), (t)); \ 49 EXPECT_EQ_WAIT((c), renderer_.num_rendered_frames(), (t)); \
47 EXPECT_EQ((w), renderer_.width()); \ 50 EXPECT_EQ((w), renderer_.width()); \
48 EXPECT_EQ((h), renderer_.height()); \ 51 EXPECT_EQ((h), renderer_.height()); \
49 EXPECT_EQ(0, renderer_.errors()); \ 52 EXPECT_EQ(0, renderer_.errors()); \
50 53
51 #define EXPECT_FRAME_ON_RENDERER_WAIT(r, c, w, h, t) \ 54 #define EXPECT_FRAME_ON_RENDERER_WAIT(r, c, w, h, t) \
52 EXPECT_EQ_WAIT((c), (r).num_rendered_frames(), (t)); \ 55 EXPECT_EQ_WAIT((c), (r).num_rendered_frames(), (t)); \
53 EXPECT_EQ((w), (r).width()); \ 56 EXPECT_EQ((w), (r).width()); \
54 EXPECT_EQ((h), (r).height()); \ 57 EXPECT_EQ((h), (r).height()); \
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 virtual cricket::VideoCodec DefaultCodec() = 0; 139 virtual cricket::VideoCodec DefaultCodec() = 0;
137 140
138 virtual cricket::StreamParams DefaultSendStreamParams() { 141 virtual cricket::StreamParams DefaultSendStreamParams() {
139 return cricket::StreamParams::CreateLegacy(kSsrc); 142 return cricket::StreamParams::CreateLegacy(kSsrc);
140 } 143 }
141 144
142 virtual void SetUp() { 145 virtual void SetUp() {
143 cricket::Device device("test", "device"); 146 cricket::Device device("test", "device");
144 engine_.Init(); 147 engine_.Init();
145 channel_.reset( 148 channel_.reset(
146 engine_.CreateChannel(call_.get(), cricket::VideoOptions())); 149 engine_.CreateChannel(call_.get(),
150 webrtc::MediaConfig(),
151 cricket::VideoOptions()));
147 EXPECT_TRUE(channel_.get() != NULL); 152 EXPECT_TRUE(channel_.get() != NULL);
148 network_interface_.SetDestination(channel_.get()); 153 network_interface_.SetDestination(channel_.get());
149 channel_->SetInterface(&network_interface_); 154 channel_->SetInterface(&network_interface_);
150 media_error_ = cricket::VideoMediaChannel::ERROR_NONE; 155 media_error_ = cricket::VideoMediaChannel::ERROR_NONE;
151 cricket::VideoRecvParameters parameters; 156 cricket::VideoRecvParameters parameters;
152 parameters.codecs = engine_.codecs(); 157 parameters.codecs = engine_.codecs();
153 channel_->SetRecvParameters(parameters); 158 channel_->SetRecvParameters(parameters);
154 EXPECT_TRUE(channel_->AddSendStream(DefaultSendStreamParams())); 159 EXPECT_TRUE(channel_->AddSendStream(DefaultSendStreamParams()));
155 video_capturer_.reset(CreateFakeVideoCapturer()); 160 video_capturer_.reset(CreateFakeVideoCapturer());
156 cricket::VideoFormat format(640, 480, 161 cricket::VideoFormat format(640, 480,
(...skipping 1057 matching lines...) Expand 10 before | Expand all | Expand 10 after
1214 rtc::scoped_ptr<C> channel_; 1219 rtc::scoped_ptr<C> channel_;
1215 cricket::FakeNetworkInterface network_interface_; 1220 cricket::FakeNetworkInterface network_interface_;
1216 cricket::FakeVideoRenderer renderer_; 1221 cricket::FakeVideoRenderer renderer_;
1217 cricket::VideoMediaChannel::Error media_error_; 1222 cricket::VideoMediaChannel::Error media_error_;
1218 1223
1219 // Used by test cases where 2 streams are run on the same channel. 1224 // Used by test cases where 2 streams are run on the same channel.
1220 cricket::FakeVideoRenderer renderer2_; 1225 cricket::FakeVideoRenderer renderer2_;
1221 }; 1226 };
1222 1227
1223 #endif // WEBRTC_MEDIA_BASE_VIDEOENGINE_UNITTEST_H_ NOLINT 1228 #endif // WEBRTC_MEDIA_BASE_VIDEOENGINE_UNITTEST_H_ NOLINT
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698