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

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

Issue 1809093003: Moving SwitchOutputDevice out of OutputDevice interface, eliminating OutputDevice (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Getting rid of OutputDevice interface Created 4 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/webmediaplayer_ms.h" 5 #include "content/renderer/media/webmediaplayer_ms.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <limits> 8 #include <limits>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 if (audio_renderer_.get()) 229 if (audio_renderer_.get())
230 audio_renderer_->SetVolume(volume_ * volume_multiplier_); 230 audio_renderer_->SetVolume(volume_ * volume_multiplier_);
231 } 231 }
232 232
233 void WebMediaPlayerMS::setSinkId( 233 void WebMediaPlayerMS::setSinkId(
234 const blink::WebString& sink_id, 234 const blink::WebString& sink_id,
235 const blink::WebSecurityOrigin& security_origin, 235 const blink::WebSecurityOrigin& security_origin,
236 blink::WebSetSinkIdCallbacks* web_callback) { 236 blink::WebSetSinkIdCallbacks* web_callback) {
237 DVLOG(1) << __FUNCTION__; 237 DVLOG(1) << __FUNCTION__;
238 DCHECK(thread_checker_.CalledOnValidThread()); 238 DCHECK(thread_checker_.CalledOnValidThread());
239 const media::SwitchOutputDeviceCB callback = 239 const media::OutputDeviceStatusCB callback =
240 media::ConvertToSwitchOutputDeviceCB(web_callback); 240 media::ConvertToOutputDeviceStatusCB(web_callback);
241 if (audio_renderer_ && audio_renderer_->GetOutputDevice()) { 241 if (audio_renderer_) {
242 audio_renderer_->GetOutputDevice()->SwitchOutputDevice( 242 audio_renderer_->SwitchOutputDevice(sink_id.utf8(), security_origin,
243 sink_id.utf8(), security_origin, callback); 243 callback);
244 } else { 244 } else {
245 callback.Run(media::OUTPUT_DEVICE_STATUS_ERROR_INTERNAL); 245 callback.Run(media::OUTPUT_DEVICE_STATUS_ERROR_INTERNAL);
246 } 246 }
247 } 247 }
248 248
249 void WebMediaPlayerMS::setPreload(WebMediaPlayer::Preload preload) { 249 void WebMediaPlayerMS::setPreload(WebMediaPlayer::Preload preload) {
250 DCHECK(thread_checker_.CalledOnValidThread()); 250 DCHECK(thread_checker_.CalledOnValidThread());
251 } 251 }
252 252
253 bool WebMediaPlayerMS::hasVideo() const { 253 bool WebMediaPlayerMS::hasVideo() const {
(...skipping 290 matching lines...) Expand 10 before | Expand all | Expand 10 after
544 media::SkCanvasVideoRenderer* WebMediaPlayerMS::GetSkCanvasVideoRenderer() { 544 media::SkCanvasVideoRenderer* WebMediaPlayerMS::GetSkCanvasVideoRenderer() {
545 return &video_renderer_; 545 return &video_renderer_;
546 } 546 }
547 547
548 void WebMediaPlayerMS::ResetCanvasCache() { 548 void WebMediaPlayerMS::ResetCanvasCache() {
549 DCHECK(thread_checker_.CalledOnValidThread()); 549 DCHECK(thread_checker_.CalledOnValidThread());
550 video_renderer_.ResetCache(); 550 video_renderer_.ResetCache();
551 } 551 }
552 552
553 } // namespace content 553 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698