| Index: content/renderer/media/rtc_peer_connection_handler_unittest.cc
|
| diff --git a/content/renderer/media/rtc_peer_connection_handler_unittest.cc b/content/renderer/media/rtc_peer_connection_handler_unittest.cc
|
| index ec16e84284f7bfeb8aec7e3f1b161fdee8ef54c7..70a52aba82e3a069cca5d41ae3ea98addd59b039 100644
|
| --- a/content/renderer/media/rtc_peer_connection_handler_unittest.cc
|
| +++ b/content/renderer/media/rtc_peer_connection_handler_unittest.cc
|
| @@ -19,18 +19,16 @@
|
| #include "content/child/child_process.h"
|
| #include "content/renderer/media/media_stream.h"
|
| #include "content/renderer/media/media_stream_audio_source.h"
|
| -#include "content/renderer/media/media_stream_audio_track.h"
|
| #include "content/renderer/media/media_stream_source.h"
|
| #include "content/renderer/media/media_stream_video_track.h"
|
| -#include "content/renderer/media/mock_audio_device_factory.h"
|
| -#include "content/renderer/media/mock_constraint_factory.h"
|
| #include "content/renderer/media/mock_data_channel_impl.h"
|
| #include "content/renderer/media/mock_media_stream_video_source.h"
|
| #include "content/renderer/media/mock_peer_connection_impl.h"
|
| #include "content/renderer/media/mock_web_rtc_peer_connection_handler_client.h"
|
| #include "content/renderer/media/peer_connection_tracker.h"
|
| #include "content/renderer/media/webrtc/mock_peer_connection_dependency_factory.h"
|
| -#include "content/renderer/media/webrtc/processed_local_audio_source.h"
|
| +#include "content/renderer/media/webrtc/webrtc_local_audio_track_adapter.h"
|
| +#include "content/renderer/media/webrtc_local_audio_track.h"
|
| #include "testing/gmock/include/gmock/gmock.h"
|
| #include "testing/gtest/include/gtest/gtest.h"
|
| #include "third_party/WebKit/public/platform/WebMediaConstraints.h"
|
| @@ -250,25 +248,12 @@
|
| const std::string& stream_label) {
|
| std::string video_track_label("video-label");
|
| std::string audio_track_label("audio-label");
|
| - blink::WebMediaStreamSource blink_audio_source;
|
| - blink_audio_source.initialize(blink::WebString::fromUTF8(audio_track_label),
|
| - blink::WebMediaStreamSource::TypeAudio,
|
| - blink::WebString::fromUTF8("audio_track"),
|
| - false /* remote */);
|
| - ProcessedLocalAudioSource* const audio_source =
|
| - new ProcessedLocalAudioSource(
|
| - -1 /* consumer_render_frame_id is N/A for non-browser tests */,
|
| - StreamDeviceInfo(MEDIA_DEVICE_AUDIO_CAPTURE, "Mock device",
|
| - "mock_device_id",
|
| - media::AudioParameters::kAudioCDSampleRate,
|
| - media::CHANNEL_LAYOUT_STEREO,
|
| - media::AudioParameters::kAudioCDSampleRate / 100),
|
| - mock_dependency_factory_.get());
|
| - audio_source->SetAllowInvalidRenderFrameIdForTesting(true);
|
| - audio_source->SetSourceConstraints(
|
| - MockConstraintFactory().CreateWebMediaConstraints());
|
| - blink_audio_source.setExtraData(audio_source); // Takes ownership.
|
| -
|
| + blink::WebMediaStreamSource audio_source;
|
| + audio_source.initialize(blink::WebString::fromUTF8(audio_track_label),
|
| + blink::WebMediaStreamSource::TypeAudio,
|
| + blink::WebString::fromUTF8("audio_track"),
|
| + false /* remote */);
|
| + audio_source.setExtraData(new MediaStreamAudioSource());
|
| blink::WebMediaStreamSource video_source;
|
| video_source.initialize(blink::WebString::fromUTF8(video_track_label),
|
| blink::WebMediaStreamSource::TypeVideo,
|
| @@ -280,14 +265,12 @@
|
|
|
| blink::WebVector<blink::WebMediaStreamTrack> audio_tracks(
|
| static_cast<size_t>(1));
|
| - audio_tracks[0].initialize(blink_audio_source.id(), blink_audio_source);
|
| - EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(),
|
| - Initialize(_, _, -1));
|
| - EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(),
|
| - SetAutomaticGainControl(true));
|
| - EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), Start());
|
| - EXPECT_CALL(*mock_audio_device_factory_.mock_capturer_source(), Stop());
|
| - CHECK(audio_source->ConnectToTrack(audio_tracks[0]));
|
| + audio_tracks[0].initialize(audio_source.id(), audio_source);
|
| + scoped_refptr<WebRtcLocalAudioTrackAdapter> adapter(
|
| + WebRtcLocalAudioTrackAdapter::Create(audio_track_label, nullptr));
|
| + std::unique_ptr<WebRtcLocalAudioTrack> native_track(
|
| + new WebRtcLocalAudioTrack(adapter.get()));
|
| + audio_tracks[0].setExtraData(native_track.release());
|
| blink::WebVector<blink::WebMediaStreamTrack> video_tracks(
|
| static_cast<size_t>(1));
|
| blink::WebMediaConstraints video_constraints;
|
| @@ -321,25 +304,12 @@
|
| return stream;
|
| }
|
|
|
| - void StopAllTracks(const blink::WebMediaStream& stream) {
|
| - blink::WebVector<blink::WebMediaStreamTrack> audio_tracks;
|
| - stream.audioTracks(audio_tracks);
|
| - for (const auto& track : audio_tracks)
|
| - MediaStreamAudioTrack::From(track)->Stop();
|
| -
|
| - blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
|
| - stream.videoTracks(video_tracks);
|
| - for (const auto& track : video_tracks)
|
| - MediaStreamVideoTrack::GetVideoTrack(track)->Stop();
|
| - }
|
| -
|
| base::MessageLoop message_loop_;
|
| std::unique_ptr<ChildProcess> child_process_;
|
| std::unique_ptr<MockWebRTCPeerConnectionHandlerClient> mock_client_;
|
| std::unique_ptr<MockPeerConnectionDependencyFactory> mock_dependency_factory_;
|
| std::unique_ptr<NiceMock<MockPeerConnectionTracker>> mock_tracker_;
|
| std::unique_ptr<RTCPeerConnectionHandlerUnderTest> pc_handler_;
|
| - MockAudioDeviceFactory mock_audio_device_factory_;
|
|
|
| // Weak reference to the mocked native peer connection implementation.
|
| MockPeerConnectionImpl* mock_peer_connection_;
|
| @@ -524,8 +494,6 @@
|
|
|
| pc_handler_->removeStream(local_stream);
|
| EXPECT_EQ(0u, mock_peer_connection_->local_streams()->count());
|
| -
|
| - StopAllTracks(local_stream);
|
| }
|
|
|
| TEST_F(RTCPeerConnectionHandlerTest, addStreamWithStoppedAudioAndVideoTrack) {
|
| @@ -555,8 +523,6 @@
|
| EXPECT_EQ(
|
| 1u,
|
| mock_peer_connection_->local_streams()->at(0)->GetVideoTracks().size());
|
| -
|
| - StopAllTracks(local_stream);
|
| }
|
|
|
| TEST_F(RTCPeerConnectionHandlerTest, GetStatsNoSelector) {
|
| @@ -594,8 +560,6 @@
|
| pc_handler_->getStats(request.get());
|
| base::RunLoop().RunUntilIdle();
|
| EXPECT_EQ(1, request->result()->report_count());
|
| -
|
| - StopAllTracks(local_stream);
|
| }
|
|
|
| TEST_F(RTCPeerConnectionHandlerTest, GetStatsWithRemoteSelector) {
|
| @@ -635,8 +599,6 @@
|
| pc_handler_->getStats(request.get());
|
| base::RunLoop().RunUntilIdle();
|
| EXPECT_EQ(0, request->result()->report_count());
|
| -
|
| - StopAllTracks(local_stream);
|
| }
|
|
|
| TEST_F(RTCPeerConnectionHandlerTest, OnSignalingChange) {
|
| @@ -1063,8 +1025,6 @@
|
| std::unique_ptr<blink::WebRTCDTMFSenderHandler> sender(
|
| pc_handler_->createDTMFSender(tracks[0]));
|
| EXPECT_TRUE(sender.get());
|
| -
|
| - StopAllTracks(local_stream);
|
| }
|
|
|
| } // namespace content
|
|
|