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

Unified Diff: components/test_runner/mock_web_user_media_client.cc

Issue 1391363002: Extend components/test_runner's generated WebMediaStream to have a Chrome VideoTrack and Source (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/test_runner/mock_web_user_media_client.h ('k') | components/test_runner/test_runner.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/test_runner/mock_web_user_media_client.cc
diff --git a/components/test_runner/mock_web_user_media_client.cc b/components/test_runner/mock_web_user_media_client.cc
index f30fac9924beb9f87cb5df90cdd323c0e7a15871..7cff4c3096df63c2f1ccbac945451b40f0a0915b 100644
--- a/components/test_runner/mock_web_user_media_client.cc
+++ b/components/test_runner/mock_web_user_media_client.cc
@@ -18,7 +18,6 @@
#include "third_party/WebKit/public/platform/WebVector.h"
#include "third_party/WebKit/public/web/WebDocument.h"
#include "third_party/WebKit/public/web/WebMediaDevicesRequest.h"
-#include "third_party/WebKit/public/web/WebMediaStreamRegistry.h"
#include "third_party/WebKit/public/web/WebUserMediaRequest.h"
using blink::WebMediaConstraints;
@@ -161,34 +160,34 @@ void MockWebUserMediaClient::requestUserMedia(
return;
}
- const size_t zero = 0;
- const size_t one = 1;
- WebVector<WebMediaStreamTrack> audio_tracks(request.audio() ? one : zero);
- WebVector<WebMediaStreamTrack> video_tracks(request.video() ? one : zero);
+ WebMediaStream stream;
+ stream.initialize(WebVector<WebMediaStreamTrack>(),
+ WebVector<WebMediaStreamTrack>());
+ stream.setExtraData(new MockExtraData());
if (request.audio()) {
WebMediaStreamSource source;
source.initialize("MockAudioDevice#1",
WebMediaStreamSource::TypeAudio,
"Mock audio device",
- false /* remote */, true /* readonly */);
- audio_tracks[0].initialize(source);
+ false /* remote */,
+ true /* readonly */);
+ WebMediaStreamTrack web_track;
+ web_track.initialize(source);
+ stream.addTrack(web_track);
}
- if (request.video()) {
+ if (request.video() && !delegate_->AddMediaStreamSourceAndTrack(&stream)) {
WebMediaStreamSource source;
source.initialize("MockVideoDevice#1",
WebMediaStreamSource::TypeVideo,
"Mock video device",
false /* remote */, true /* readonly */);
- video_tracks[0].initialize(source);
+ WebMediaStreamTrack web_track;
+ web_track.initialize(source);
+ stream.addTrack(web_track);
}
- WebMediaStream stream;
- stream.initialize(audio_tracks, video_tracks);
-
- stream.setExtraData(new MockExtraData());
-
delegate_->PostTask(new UserMediaRequestTask(this, request, stream));
}
« no previous file with comments | « components/test_runner/mock_web_user_media_client.h ('k') | components/test_runner/test_runner.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698