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

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

Issue 2767963002: Revert of Remove |remote| and |readonly| members of MediaStreamTrack. (Closed)
Patch Set: rebase Created 3 years, 9 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 "content/renderer/media/webrtc/webrtc_media_stream_adapter.h" 5 #include "content/renderer/media/webrtc/webrtc_media_stream_adapter.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <memory> 9 #include <memory>
10 10
(...skipping 30 matching lines...) Expand all
41 adapter_.reset(); 41 adapter_.reset();
42 blink::WebHeap::collectAllGarbageForTesting(); 42 blink::WebHeap::collectAllGarbageForTesting();
43 } 43 }
44 44
45 blink::WebMediaStream CreateBlinkMediaStream(bool audio, bool video) { 45 blink::WebMediaStream CreateBlinkMediaStream(bool audio, bool video) {
46 blink::WebVector<blink::WebMediaStreamTrack> audio_track_vector( 46 blink::WebVector<blink::WebMediaStreamTrack> audio_track_vector(
47 audio ? static_cast<size_t>(1) : 0); 47 audio ? static_cast<size_t>(1) : 0);
48 if (audio) { 48 if (audio) {
49 blink::WebMediaStreamSource audio_source; 49 blink::WebMediaStreamSource audio_source;
50 audio_source.initialize("audio", blink::WebMediaStreamSource::TypeAudio, 50 audio_source.initialize("audio", blink::WebMediaStreamSource::TypeAudio,
51 "audio"); 51 "audio", false /* remote */);
52 ProcessedLocalAudioSource* const source = new ProcessedLocalAudioSource( 52 ProcessedLocalAudioSource* const source = new ProcessedLocalAudioSource(
53 -1 /* consumer_render_frame_id is N/A for non-browser tests */, 53 -1 /* consumer_render_frame_id is N/A for non-browser tests */,
54 StreamDeviceInfo(MEDIA_DEVICE_AUDIO_CAPTURE, "Mock audio device", 54 StreamDeviceInfo(MEDIA_DEVICE_AUDIO_CAPTURE, "Mock audio device",
55 "mock_audio_device_id", 55 "mock_audio_device_id",
56 media::AudioParameters::kAudioCDSampleRate, 56 media::AudioParameters::kAudioCDSampleRate,
57 media::CHANNEL_LAYOUT_STEREO, 57 media::CHANNEL_LAYOUT_STEREO,
58 media::AudioParameters::kAudioCDSampleRate / 50), 58 media::AudioParameters::kAudioCDSampleRate / 50),
59 MockConstraintFactory().CreateWebMediaConstraints(), 59 MockConstraintFactory().CreateWebMediaConstraints(),
60 base::Bind(&WebRtcMediaStreamAdapterTest::OnAudioSourceStarted), 60 base::Bind(&WebRtcMediaStreamAdapterTest::OnAudioSourceStarted),
61 dependency_factory_.get()); 61 dependency_factory_.get());
62 source->SetAllowInvalidRenderFrameIdForTesting(true); 62 source->SetAllowInvalidRenderFrameIdForTesting(true);
63 audio_source.setExtraData(source); // Takes ownership. 63 audio_source.setExtraData(source); // Takes ownership.
64 audio_track_vector[0].initialize(audio_source); 64 audio_track_vector[0].initialize(audio_source);
65 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), 65 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(),
66 Initialize(_, _, -1)); 66 Initialize(_, _, -1));
67 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), 67 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(),
68 SetAutomaticGainControl(true)); 68 SetAutomaticGainControl(true));
69 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), Start()); 69 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), Start());
70 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), Stop()); 70 EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), Stop());
71 CHECK(source->ConnectToTrack(audio_track_vector[0])); 71 CHECK(source->ConnectToTrack(audio_track_vector[0]));
72 } 72 }
73 73
74 blink::WebVector<blink::WebMediaStreamTrack> video_track_vector( 74 blink::WebVector<blink::WebMediaStreamTrack> video_track_vector(
75 video ? static_cast<size_t>(1) : 0); 75 video ? static_cast<size_t>(1) : 0);
76 MediaStreamSource::SourceStoppedCallback dummy_callback; 76 MediaStreamSource::SourceStoppedCallback dummy_callback;
77 if (video) { 77 if (video) {
78 blink::WebMediaStreamSource video_source; 78 blink::WebMediaStreamSource video_source;
79 video_source.initialize("video", blink::WebMediaStreamSource::TypeVideo, 79 video_source.initialize("video", blink::WebMediaStreamSource::TypeVideo,
80 "video"); 80 "video", false /* remote */);
81 MediaStreamVideoSource* native_source = 81 MediaStreamVideoSource* native_source =
82 new MockMediaStreamVideoSource(false); 82 new MockMediaStreamVideoSource(false);
83 video_source.setExtraData(native_source); 83 video_source.setExtraData(native_source);
84 blink::WebMediaConstraints constraints; 84 blink::WebMediaConstraints constraints;
85 constraints.initialize(); 85 constraints.initialize();
86 video_track_vector[0] = MediaStreamVideoTrack::CreateVideoTrack( 86 video_track_vector[0] = MediaStreamVideoTrack::CreateVideoTrack(
87 native_source, constraints, 87 native_source, constraints,
88 MediaStreamVideoSource::ConstraintsCallback(), true); 88 MediaStreamVideoSource::ConstraintsCallback(), true);
89 } 89 }
90 90
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 } 132 }
133 133
134 // Test that we don't crash if a MediaStream is created in Blink with an unknown 134 // Test that we don't crash if a MediaStream is created in Blink with an unknown
135 // audio sources. This can happen if a MediaStream is created with 135 // audio sources. This can happen if a MediaStream is created with
136 // remote audio track. 136 // remote audio track.
137 TEST_F(WebRtcMediaStreamAdapterTest, 137 TEST_F(WebRtcMediaStreamAdapterTest,
138 CreateWebRtcMediaStreamWithoutAudioSource) { 138 CreateWebRtcMediaStreamWithoutAudioSource) {
139 // Create a blink MediaStream description. 139 // Create a blink MediaStream description.
140 blink::WebMediaStreamSource audio_source; 140 blink::WebMediaStreamSource audio_source;
141 audio_source.initialize("audio source", 141 audio_source.initialize("audio source",
142 blink::WebMediaStreamSource::TypeAudio, "something"); 142 blink::WebMediaStreamSource::TypeAudio, "something",
143 false /* remote */);
143 144
144 blink::WebVector<blink::WebMediaStreamTrack> audio_tracks( 145 blink::WebVector<blink::WebMediaStreamTrack> audio_tracks(
145 static_cast<size_t>(1)); 146 static_cast<size_t>(1));
146 audio_tracks[0].initialize(audio_source.id(), audio_source); 147 audio_tracks[0].initialize(audio_source.id(), audio_source);
147 blink::WebVector<blink::WebMediaStreamTrack> video_tracks( 148 blink::WebVector<blink::WebMediaStreamTrack> video_tracks(
148 static_cast<size_t>(0)); 149 static_cast<size_t>(0));
149 150
150 blink::WebMediaStream blink_stream; 151 blink::WebMediaStream blink_stream;
151 blink_stream.initialize("new stream", audio_tracks, video_tracks); 152 blink_stream.initialize("new stream", audio_tracks, video_tracks);
152 blink_stream.setExtraData(new content::MediaStream()); 153 blink_stream.setExtraData(new content::MediaStream());
(...skipping 19 matching lines...) Expand all
172 EXPECT_TRUE(webrtc_stream()->GetVideoTracks().empty()); 173 EXPECT_TRUE(webrtc_stream()->GetVideoTracks().empty());
173 174
174 native_stream->AddTrack(audio_tracks[0]); 175 native_stream->AddTrack(audio_tracks[0]);
175 EXPECT_EQ(1u, webrtc_stream()->GetAudioTracks().size()); 176 EXPECT_EQ(1u, webrtc_stream()->GetAudioTracks().size());
176 177
177 native_stream->AddTrack(video_tracks[0]); 178 native_stream->AddTrack(video_tracks[0]);
178 EXPECT_EQ(1u, webrtc_stream()->GetVideoTracks().size()); 179 EXPECT_EQ(1u, webrtc_stream()->GetVideoTracks().size());
179 } 180 }
180 181
181 } // namespace content 182 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698