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

Side by Side Diff: content/renderer/media/media_stream_impl.cc

Issue 10177008: Shift media_stream::StreamOptions to align with the new getUserMedia spec (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 7 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/media_stream_impl.h" 5 #include "content/renderer/media/media_stream_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 void MediaStreamImpl::requestUserMedia( 161 void MediaStreamImpl::requestUserMedia(
162 const WebKit::WebUserMediaRequest& user_media_request, 162 const WebKit::WebUserMediaRequest& user_media_request,
163 const WebKit::WebVector<WebKit::WebMediaStreamSource>& audio_sources, 163 const WebKit::WebVector<WebKit::WebMediaStreamSource>& audio_sources,
164 const WebKit::WebVector<WebKit::WebMediaStreamSource>& video_sources) { 164 const WebKit::WebVector<WebKit::WebMediaStreamSource>& video_sources) {
165 DCHECK(CalledOnValidThread()); 165 DCHECK(CalledOnValidThread());
166 DCHECK(!user_media_request.isNull()); 166 DCHECK(!user_media_request.isNull());
167 167
168 int request_id = next_request_id_++; 168 int request_id = next_request_id_++;
169 169
170 bool audio = user_media_request.audio(); 170 bool audio = user_media_request.audio();
171 media_stream::StreamOptions::VideoOption video_option = 171 bool video = user_media_request.video();
172 media_stream::StreamOptions::kNoCamera;
173 if (user_media_request.video())
174 video_option = media_stream::StreamOptions::kFacingBoth;
175 172
176 std::string security_origin = UTF16ToUTF8( 173 std::string security_origin = UTF16ToUTF8(
177 user_media_request.securityOrigin().toString()); 174 user_media_request.securityOrigin().toString());
178 175
179 DVLOG(1) << "MediaStreamImpl::generateStream(" << request_id << ", [ " 176 DVLOG(1) << "MediaStreamImpl::generateStream(" << request_id << ", [ "
180 << (audio ? "audio" : "") 177 << (audio ? "audio" : "")
181 << (user_media_request.video() ? " video" : "") << "], " 178 << (user_media_request.video() ? " video" : "") << "], "
182 << security_origin << ")"; 179 << security_origin << ")";
183 180
184 user_media_requests_.insert( 181 user_media_requests_.insert(
185 std::pair<int, WebKit::WebUserMediaRequest>( 182 std::pair<int, WebKit::WebUserMediaRequest>(
186 request_id, user_media_request)); 183 request_id, user_media_request));
187 184
188 media_stream_dispatcher_->GenerateStream( 185 media_stream_dispatcher_->GenerateStream(
189 request_id, 186 request_id,
190 AsWeakPtr(), 187 AsWeakPtr(),
191 media_stream::StreamOptions(audio, video_option), 188 media_stream::StreamOptions(audio, video),
192 security_origin); 189 security_origin);
193 } 190 }
194 191
195 void MediaStreamImpl::cancelUserMediaRequest( 192 void MediaStreamImpl::cancelUserMediaRequest(
196 const WebKit::WebUserMediaRequest& user_media_request) { 193 const WebKit::WebUserMediaRequest& user_media_request) {
197 DCHECK(CalledOnValidThread()); 194 DCHECK(CalledOnValidThread());
198 // TODO(grunell): Implement. 195 // TODO(grunell): Implement.
199 NOTIMPLEMENTED(); 196 NOTIMPLEMENTED();
200 } 197 }
201 198
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
505 return rtc_video_decoder; 502 return rtc_video_decoder;
506 } 503 }
507 504
508 MediaStreamImpl::VideoRendererWrapper::VideoRendererWrapper( 505 MediaStreamImpl::VideoRendererWrapper::VideoRendererWrapper(
509 RTCVideoDecoder* decoder) 506 RTCVideoDecoder* decoder)
510 : rtc_video_decoder_(decoder) { 507 : rtc_video_decoder_(decoder) {
511 } 508 }
512 509
513 MediaStreamImpl::VideoRendererWrapper::~VideoRendererWrapper() { 510 MediaStreamImpl::VideoRendererWrapper::~VideoRendererWrapper() {
514 } 511 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698