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

Side by Side Diff: talk/app/webrtc/videosource_unittest.cc

Issue 1346153002: Remove use of DeviceManager from ChannelManager. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase Created 5 years, 2 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 2012 Google Inc. 3 * Copyright 2012 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 14 matching lines...) Expand all
25 * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 25 * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
26 */ 26 */
27 27
28 #include <string> 28 #include <string>
29 #include <vector> 29 #include <vector>
30 30
31 #include "talk/app/webrtc/remotevideocapturer.h" 31 #include "talk/app/webrtc/remotevideocapturer.h"
32 #include "talk/app/webrtc/test/fakeconstraints.h" 32 #include "talk/app/webrtc/test/fakeconstraints.h"
33 #include "talk/app/webrtc/videosource.h" 33 #include "talk/app/webrtc/videosource.h"
34 #include "talk/media/base/fakemediaengine.h" 34 #include "talk/media/base/fakemediaengine.h"
35 #include "talk/media/base/fakevideocapturer.h"
35 #include "talk/media/base/fakevideorenderer.h" 36 #include "talk/media/base/fakevideorenderer.h"
36 #include "talk/media/devices/fakedevicemanager.h"
37 #include "talk/media/webrtc/webrtcvideoframe.h" 37 #include "talk/media/webrtc/webrtcvideoframe.h"
38 #include "talk/session/media/channelmanager.h" 38 #include "talk/session/media/channelmanager.h"
39 #include "webrtc/base/gunit.h" 39 #include "webrtc/base/gunit.h"
40 40
41 using webrtc::FakeConstraints; 41 using webrtc::FakeConstraints;
42 using webrtc::VideoSource; 42 using webrtc::VideoSource;
43 using webrtc::MediaConstraintsInterface; 43 using webrtc::MediaConstraintsInterface;
44 using webrtc::MediaSourceInterface; 44 using webrtc::MediaSourceInterface;
45 using webrtc::ObserverInterface; 45 using webrtc::ObserverInterface;
46 using webrtc::VideoSourceInterface; 46 using webrtc::VideoSourceInterface;
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 MediaSourceInterface::SourceState state_; 123 MediaSourceInterface::SourceState state_;
124 rtc::scoped_refptr<VideoSourceInterface> source_; 124 rtc::scoped_refptr<VideoSourceInterface> source_;
125 }; 125 };
126 126
127 class VideoSourceTest : public testing::Test { 127 class VideoSourceTest : public testing::Test {
128 protected: 128 protected:
129 VideoSourceTest() 129 VideoSourceTest()
130 : capturer_cleanup_(new TestVideoCapturer()), 130 : capturer_cleanup_(new TestVideoCapturer()),
131 capturer_(capturer_cleanup_.get()), 131 capturer_(capturer_cleanup_.get()),
132 channel_manager_(new cricket::ChannelManager( 132 channel_manager_(new cricket::ChannelManager(
133 new cricket::FakeMediaEngine(), 133 new cricket::FakeMediaEngine(), rtc::Thread::Current())) {
134 new cricket::FakeDeviceManager(), rtc::Thread::Current())) {
135 } 134 }
136 135
137 void SetUp() { 136 void SetUp() {
138 ASSERT_TRUE(channel_manager_->Init()); 137 ASSERT_TRUE(channel_manager_->Init());
139 } 138 }
140 139
141 void CreateVideoSource() { 140 void CreateVideoSource() {
142 CreateVideoSource(NULL); 141 CreateVideoSource(NULL);
143 } 142 }
144 143
(...skipping 409 matching lines...) Expand 10 before | Expand all | Expand 10 after
554 constraints.AddOptional(MediaConstraintsInterface::kMaxFrameRate, 0.5); 553 constraints.AddOptional(MediaConstraintsInterface::kMaxFrameRate, 0.5);
555 554
556 CreateVideoSource(&constraints); 555 CreateVideoSource(&constraints);
557 EXPECT_EQ_WAIT(MediaSourceInterface::kLive, state_observer_->state(), 556 EXPECT_EQ_WAIT(MediaSourceInterface::kLive, state_observer_->state(),
558 kMaxWaitMs); 557 kMaxWaitMs);
559 const cricket::VideoFormat* format = capturer_->GetCaptureFormat(); 558 const cricket::VideoFormat* format = capturer_->GetCaptureFormat();
560 ASSERT_TRUE(format != NULL); 559 ASSERT_TRUE(format != NULL);
561 EXPECT_EQ(30, format->framerate()); 560 EXPECT_EQ(30, format->framerate());
562 } 561 }
563 562
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698