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

Side by Side Diff: content/renderer/media/webrtc/media_stream_remote_video_source_unittest.cc

Issue 503683003: Remove implicit conversions from scoped_refptr to T* in content/renderer/media/webrtc* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/message_loop/message_loop.h" 5 #include "base/message_loop/message_loop.h"
6 #include "base/run_loop.h" 6 #include "base/run_loop.h"
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "content/child/child_process.h" 8 #include "content/child/child_process.h"
9 #include "content/renderer/media/media_stream_video_track.h" 9 #include "content/renderer/media/media_stream_video_track.h"
10 #include "content/renderer/media/mock_media_stream_video_sink.h" 10 #include "content/renderer/media/mock_media_stream_video_sink.h"
(...skipping 17 matching lines...) Expand all
28 } 28 }
29 using MediaStreamRemoteVideoSource::RenderInterfaceForTest; 29 using MediaStreamRemoteVideoSource::RenderInterfaceForTest;
30 }; 30 };
31 31
32 class MediaStreamRemoteVideoSourceTest 32 class MediaStreamRemoteVideoSourceTest
33 : public ::testing::Test { 33 : public ::testing::Test {
34 public: 34 public:
35 MediaStreamRemoteVideoSourceTest() 35 MediaStreamRemoteVideoSourceTest()
36 : child_process_(new ChildProcess()), 36 : child_process_(new ChildProcess()),
37 mock_factory_(new MockPeerConnectionDependencyFactory()), 37 mock_factory_(new MockPeerConnectionDependencyFactory()),
38 webrtc_video_track_( 38 webrtc_video_track_(mock_factory_->CreateLocalVideoTrack(
39 mock_factory_->CreateLocalVideoTrack( 39 "test",
40 "test", 40 static_cast<cricket::VideoCapturer*>(NULL))),
41 static_cast<cricket::VideoCapturer*>(NULL))), 41 remote_source_(new MediaStreamRemoteVideoSourceUnderTest(
42 remote_source_( 42 webrtc_video_track_.get())),
43 new MediaStreamRemoteVideoSourceUnderTest(webrtc_video_track_)),
44 number_of_successful_constraints_applied_(0), 43 number_of_successful_constraints_applied_(0),
45 number_of_failed_constraints_applied_(0) { 44 number_of_failed_constraints_applied_(0) {
46 webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"), 45 webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
47 blink::WebMediaStreamSource::TypeVideo, 46 blink::WebMediaStreamSource::TypeVideo,
48 base::UTF8ToUTF16("dummy_source_name")); 47 base::UTF8ToUTF16("dummy_source_name"));
49 webkit_source_.setExtraData(remote_source_); 48 webkit_source_.setExtraData(remote_source_);
50 } 49 }
51 50
52 MediaStreamRemoteVideoSourceUnderTest* source() { 51 MediaStreamRemoteVideoSourceUnderTest* source() {
53 return remote_source_; 52 return remote_source_;
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 webkit_source().readyState()); 133 webkit_source().readyState());
135 StopWebRtcTrack(); 134 StopWebRtcTrack();
136 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded, 135 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded,
137 webkit_source().readyState()); 136 webkit_source().readyState());
138 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded, sink.state()); 137 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded, sink.state());
139 138
140 track->RemoveSink(&sink); 139 track->RemoveSink(&sink);
141 } 140 }
142 141
143 } // namespace content 142 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698