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

Side by Side Diff: webrtc/media/engine/webrtcmediaengine_unittest.cc

Issue 2069643002: Added backwards compatible version of WebRtcMediaEngineFactory::Create. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Added tests. Created 4 years, 6 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/media/engine/webrtcmediaengine.cc ('k') | no next file » | 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) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 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 #include "testing/gtest/include/gtest/gtest.h" 11 #include "testing/gtest/include/gtest/gtest.h"
12 12
13 #include "webrtc/modules/audio_coding/codecs/builtin_audio_decoder_factory.h"
13 #include "webrtc/media/engine/webrtcmediaengine.h" 14 #include "webrtc/media/engine/webrtcmediaengine.h"
14 15
15 using webrtc::RtpExtension; 16 using webrtc::RtpExtension;
16 17
17 namespace cricket { 18 namespace cricket {
18 namespace { 19 namespace {
19 20
20 std::vector<RtpExtension> MakeUniqueExtensions() { 21 std::vector<RtpExtension> MakeUniqueExtensions() {
21 std::vector<RtpExtension> result; 22 std::vector<RtpExtension> result;
22 char name[] = "a"; 23 char name[] = "a";
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 173
173 TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantBwe_3) { 174 TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantBwe_3) {
174 std::vector<RtpExtension> extensions; 175 std::vector<RtpExtension> extensions;
175 extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 2)); 176 extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 2));
176 extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 14)); 177 extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 14));
177 std::vector<webrtc::RtpExtension> filtered = 178 std::vector<webrtc::RtpExtension> filtered =
178 FilterRtpExtensions(extensions, SupportedExtensions2, true); 179 FilterRtpExtensions(extensions, SupportedExtensions2, true);
179 EXPECT_EQ(1, filtered.size()); 180 EXPECT_EQ(1, filtered.size());
180 EXPECT_EQ(RtpExtension::kTimestampOffsetUri, filtered[0].uri); 181 EXPECT_EQ(RtpExtension::kTimestampOffsetUri, filtered[0].uri);
181 } 182 }
183
184 TEST(WebRtcMediaEngineFactoryTest, CreateOldApi) {
185 std::unique_ptr<MediaEngineInterface> engine(
186 WebRtcMediaEngineFactory::Create(nullptr, nullptr, nullptr));
187 EXPECT_TRUE(engine);
188 }
189
190 TEST(WebRtcMediaEngineFactoryTest, CreateWithBuiltinDecoders) {
191 std::unique_ptr<MediaEngineInterface> engine(WebRtcMediaEngineFactory::Create(
192 nullptr, webrtc::CreateBuiltinAudioDecoderFactory(), nullptr, nullptr));
193 EXPECT_TRUE(engine);
194 }
195
182 } // namespace cricket 196 } // namespace cricket
OLDNEW
« no previous file with comments | « webrtc/media/engine/webrtcmediaengine.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698