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

Side by Side Diff: components/html_viewer/html_document.cc

Issue 1133033003: Eliminate MediaPlayer & MediaPlayerClient abstractions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: added new path for createMediaPlayer Created 5 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 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 "components/html_viewer/html_document.h" 5 #include "components/html_viewer/html_document.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/message_loop/message_loop_proxy.h" 10 #include "base/message_loop/message_loop_proxy.h"
(...skipping 221 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 compositor_thread_, surface.Pass(), gpu_service.Pass())); 232 compositor_thread_, surface.Pass(), gpu_service.Pass()));
233 } 233 }
234 234
235 blink::WebLayerTreeView* HTMLDocument::layerTreeView() { 235 blink::WebLayerTreeView* HTMLDocument::layerTreeView() {
236 return web_layer_tree_view_impl_.get(); 236 return web_layer_tree_view_impl_.get();
237 } 237 }
238 238
239 blink::WebMediaPlayer* HTMLDocument::createMediaPlayer( 239 blink::WebMediaPlayer* HTMLDocument::createMediaPlayer(
240 blink::WebLocalFrame* frame, 240 blink::WebLocalFrame* frame,
241 const blink::WebURL& url, 241 const blink::WebURL& url,
242 blink::WebMediaPlayerClient* client) { 242 blink::WebMediaPlayerClient* client,
243 return createMediaPlayer(frame, url, client, nullptr); 243 blink::WebContentDecryptionModule* initial_cdm) {
244 return createMediaPlayer(frame, url, client, nullptr, nullptr);
ddorwin 2015/05/26 21:43:07 need to pass initial_cdm
Srirama 2015/05/27 14:48:50 Acknowledged.
244 } 245 }
245 246
246 blink::WebMediaPlayer* HTMLDocument::createMediaPlayer( 247 blink::WebMediaPlayer* HTMLDocument::createMediaPlayer(
247 blink::WebLocalFrame* frame, 248 blink::WebLocalFrame* frame,
248 const blink::WebURL& url, 249 const blink::WebURL& url,
249 blink::WebMediaPlayerClient* client, 250 blink::WebMediaPlayerClient* client,
251 blink::WebMediaPlayerEncryptedMediaClient* encrypted_client,
250 blink::WebContentDecryptionModule* initial_cdm) { 252 blink::WebContentDecryptionModule* initial_cdm) {
251 blink::WebMediaPlayer* player = 253 blink::WebMediaPlayer* player =
252 web_media_player_factory_ 254 web_media_player_factory_
253 ? web_media_player_factory_->CreateMediaPlayer( 255 ? web_media_player_factory_->CreateMediaPlayer(
254 frame, url, client, GetMediaPermission(), GetCdmFactory(), 256 frame, url, client, GetMediaPermission(), GetCdmFactory(),
255 initial_cdm, shell_) 257 initial_cdm, shell_)
256 : nullptr; 258 : nullptr;
257 return player; 259 return player;
258 } 260 }
259 261
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
376 return media_permission_.get(); 378 return media_permission_.get();
377 } 379 }
378 380
379 media::CdmFactory* HTMLDocument::GetCdmFactory() { 381 media::CdmFactory* HTMLDocument::GetCdmFactory() {
380 if (!cdm_factory_) 382 if (!cdm_factory_)
381 cdm_factory_.reset(new media::DefaultCdmFactory()); 383 cdm_factory_.reset(new media::DefaultCdmFactory());
382 return cdm_factory_.get(); 384 return cdm_factory_.get();
383 } 385 }
384 386
385 } // namespace html_viewer 387 } // namespace html_viewer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698