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

Side by Side Diff: content/renderer/render_frame_impl.cc

Issue 865683004: media: Make |web_encrypted_media_client_| a scoped_ptr. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/render_frame_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 618 matching lines...) Expand 10 before | Expand all | Expand 10 after
629 proxy_routing_id_(MSG_ROUTING_NONE), 629 proxy_routing_id_(MSG_ROUTING_NONE),
630 #if defined(ENABLE_PLUGINS) 630 #if defined(ENABLE_PLUGINS)
631 plugin_power_saver_helper_(NULL), 631 plugin_power_saver_helper_(NULL),
632 #endif 632 #endif
633 cookie_jar_(this), 633 cookie_jar_(this),
634 selection_text_offset_(0), 634 selection_text_offset_(0),
635 selection_range_(gfx::Range::InvalidRange()), 635 selection_range_(gfx::Range::InvalidRange()),
636 handling_select_range_(false), 636 handling_select_range_(false),
637 notification_permission_dispatcher_(NULL), 637 notification_permission_dispatcher_(NULL),
638 web_user_media_client_(NULL), 638 web_user_media_client_(NULL),
639 web_encrypted_media_client_(NULL),
640 midi_dispatcher_(NULL), 639 midi_dispatcher_(NULL),
641 #if defined(OS_ANDROID) 640 #if defined(OS_ANDROID)
642 media_player_manager_(NULL), 641 media_player_manager_(NULL),
643 #endif 642 #endif
644 #if defined(ENABLE_BROWSER_CDMS) 643 #if defined(ENABLE_BROWSER_CDMS)
645 cdm_manager_(NULL), 644 cdm_manager_(NULL),
646 #endif 645 #endif
647 #if defined(VIDEO_HOLE) 646 #if defined(VIDEO_HOLE)
648 contains_media_player_(false), 647 contains_media_player_(false),
649 #endif 648 #endif
(...skipping 2698 matching lines...) Expand 10 before | Expand all | Expand 10 after
3348 if (!web_encrypted_media_client_) { 3347 if (!web_encrypted_media_client_) {
3349 #if defined(ENABLE_PEPPER_CDMS) 3348 #if defined(ENABLE_PEPPER_CDMS)
3350 scoped_ptr<media::CdmFactory> cdm_factory( 3349 scoped_ptr<media::CdmFactory> cdm_factory(
3351 new RenderCdmFactory(base::Bind(PepperCdmWrapperImpl::Create, frame_))); 3350 new RenderCdmFactory(base::Bind(PepperCdmWrapperImpl::Create, frame_)));
3352 #elif defined(ENABLE_BROWSER_CDMS) 3351 #elif defined(ENABLE_BROWSER_CDMS)
3353 scoped_ptr<media::CdmFactory> cdm_factory( 3352 scoped_ptr<media::CdmFactory> cdm_factory(
3354 new RenderCdmFactory(GetCdmManager())); 3353 new RenderCdmFactory(GetCdmManager()));
3355 #else 3354 #else
3356 scoped_ptr<media::CdmFactory> cdm_factory(new RenderCdmFactory()); 3355 scoped_ptr<media::CdmFactory> cdm_factory(new RenderCdmFactory());
3357 #endif 3356 #endif
3358 web_encrypted_media_client_ = 3357 web_encrypted_media_client_.reset(
3359 new media::WebEncryptedMediaClientImpl(cdm_factory.Pass()); 3358 new media::WebEncryptedMediaClientImpl(cdm_factory.Pass()));
3360 } 3359 }
3361 return web_encrypted_media_client_; 3360 return web_encrypted_media_client_.get();
3362 } 3361 }
3363 3362
3364 blink::WebMIDIClient* RenderFrameImpl::webMIDIClient() { 3363 blink::WebMIDIClient* RenderFrameImpl::webMIDIClient() {
3365 if (!midi_dispatcher_) 3364 if (!midi_dispatcher_)
3366 midi_dispatcher_ = new MidiDispatcher(this); 3365 midi_dispatcher_ = new MidiDispatcher(this);
3367 return midi_dispatcher_; 3366 return midi_dispatcher_;
3368 } 3367 }
3369 3368
3370 bool RenderFrameImpl::willCheckAndDispatchMessageEvent( 3369 bool RenderFrameImpl::willCheckAndDispatchMessageEvent(
3371 blink::WebLocalFrame* source_frame, 3370 blink::WebLocalFrame* source_frame,
(...skipping 1031 matching lines...) Expand 10 before | Expand all | Expand 10 after
4403 4402
4404 #if defined(ENABLE_BROWSER_CDMS) 4403 #if defined(ENABLE_BROWSER_CDMS)
4405 RendererCdmManager* RenderFrameImpl::GetCdmManager() { 4404 RendererCdmManager* RenderFrameImpl::GetCdmManager() {
4406 if (!cdm_manager_) 4405 if (!cdm_manager_)
4407 cdm_manager_ = new RendererCdmManager(this); 4406 cdm_manager_ = new RendererCdmManager(this);
4408 return cdm_manager_; 4407 return cdm_manager_;
4409 } 4408 }
4410 #endif // defined(ENABLE_BROWSER_CDMS) 4409 #endif // defined(ENABLE_BROWSER_CDMS)
4411 4410
4412 } // namespace content 4411 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698