diff options
author | dcheng <dcheng@chromium.org> | 2014-10-06 21:14:54 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-07 04:15:17 +0000 |
commit | 165763c20c375201e93c5663f6b1fc0834503747 (patch) | |
tree | c4428a19dd281c2b9c9a8af817f664302cd6db88 /content/browser/renderer_host | |
parent | c53c0289d86a263c3446b3c89779612502682a01 (diff) | |
download | chromium_src-165763c20c375201e93c5663f6b1fc0834503747.zip chromium_src-165763c20c375201e93c5663f6b1fc0834503747.tar.gz chromium_src-165763c20c375201e93c5663f6b1fc0834503747.tar.bz2 |
Remove implicit conversions from scoped_refptr to T* in content/
This patch was generated by running the rewrite_scoped_refptr clang tool
on an Android build.
BUG=110610
Review URL: https://codereview.chromium.org/628333003
Cr-Commit-Position: refs/heads/master@{#298364}
Diffstat (limited to 'content/browser/renderer_host')
4 files changed, 18 insertions, 18 deletions
diff --git a/content/browser/renderer_host/compositor_impl_android.cc b/content/browser/renderer_host/compositor_impl_android.cc index e7a1d3c..0d6ee68 100644 --- a/content/browser/renderer_host/compositor_impl_android.cc +++ b/content/browser/renderer_host/compositor_impl_android.cc @@ -364,11 +364,11 @@ ui::SystemUIResourceManager& CompositorImpl::GetSystemUIResourceManager() { } void CompositorImpl::SetRootLayer(scoped_refptr<cc::Layer> root_layer) { - if (subroot_layer_) { + if (subroot_layer_.get()) { subroot_layer_->RemoveFromParent(); subroot_layer_ = NULL; } - if (root_layer) { + if (root_layer.get()) { subroot_layer_ = root_layer; root_layer_->AddChild(root_layer); } @@ -515,7 +515,7 @@ CreateGpuProcessViewContext( const scoped_refptr<GpuChannelHost>& gpu_channel_host, const blink::WebGraphicsContext3D::Attributes attributes, int surface_id) { - DCHECK(gpu_channel_host); + DCHECK(gpu_channel_host.get()); GURL url("chrome://gpu/Compositor::createContext3D"); static const size_t kBytesPerPixel = 4; @@ -578,7 +578,7 @@ void CompositorImpl::CreateOutputSurface(bool fallback) { BrowserGpuChannelHostFactory* factory = BrowserGpuChannelHostFactory::instance(); scoped_refptr<GpuChannelHost> gpu_channel_host = factory->GetGpuChannel(); - if (gpu_channel_host && !gpu_channel_host->IsLost()) { + if (gpu_channel_host.get() && !gpu_channel_host->IsLost()) { context_provider = ContextProviderCommandBuffer::Create( CreateGpuProcessViewContext(gpu_channel_host, attrs, surface_id_), "BrowserCompositor"); diff --git a/content/browser/renderer_host/image_transport_factory_android.cc b/content/browser/renderer_host/image_transport_factory_android.cc index 919b709..17664a8 100644 --- a/content/browser/renderer_host/image_transport_factory_android.cc +++ b/content/browser/renderer_host/image_transport_factory_android.cc @@ -55,7 +55,7 @@ CmdBufferImageTransportFactory::CmdBufferImageTransportFactory() { BrowserGpuChannelHostFactory::instance(); scoped_refptr<GpuChannelHost> gpu_channel_host(factory->EstablishGpuChannelSync( CAUSE_FOR_GPU_LAUNCH_WEBGRAPHICSCONTEXT3DCOMMANDBUFFERIMPL_INITIALIZE)); - DCHECK(gpu_channel_host); + DCHECK(gpu_channel_host.get()); blink::WebGraphicsContext3D::Attributes attrs; attrs.shareResources = true; diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc index 76a7717..8984a79 100644 --- a/content/browser/renderer_host/render_process_host_impl.cc +++ b/content/browser/renderer_host/render_process_host_impl.cc @@ -810,11 +810,11 @@ void RenderProcessHostImpl::CreateMessageFilters() { channel_->AddFilter(new FontCacheDispatcher()); #elif defined(OS_ANDROID) browser_demuxer_android_ = new BrowserDemuxerAndroid(); - AddFilter(browser_demuxer_android_); + AddFilter(browser_demuxer_android_.get()); #endif #if defined(ENABLE_BROWSER_CDMS) browser_cdm_manager_ = new BrowserCdmManager(GetID(), NULL); - AddFilter(browser_cdm_manager_); + AddFilter(browser_cdm_manager_.get()); #endif SocketStreamDispatcherHost::GetRequestContextCallback diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc index 9f04f99..89efe4e 100644 --- a/content/browser/renderer_host/render_widget_host_view_android.cc +++ b/content/browser/renderer_host/render_widget_host_view_android.cc @@ -418,7 +418,7 @@ void RenderWidgetHostViewAndroid::GetScaledContentBitmap( scoped_refptr<cc::DelegatedRendererLayer> RenderWidgetHostViewAndroid::CreateDelegatedLayerForFrameProvider() const { - DCHECK(frame_provider_); + DCHECK(frame_provider_.get()); scoped_refptr<cc::DelegatedRendererLayer> delegated_layer = cc::DelegatedRendererLayer::Create(frame_provider_); @@ -432,7 +432,7 @@ RenderWidgetHostViewAndroid::CreateDelegatedLayerForFrameProvider() const { bool RenderWidgetHostViewAndroid::HasValidFrame() const { if (!content_view_core_) return false; - if (!layer_) + if (!layer_.get()) return false; if (texture_size_in_layer_.IsEmpty()) @@ -504,7 +504,7 @@ void RenderWidgetHostViewAndroid::Show() { return; is_showing_ = true; - if (layer_) + if (layer_.get()) layer_->SetHideLayerAndSubtree(false); frame_evictor_->SetVisible(true); @@ -516,7 +516,7 @@ void RenderWidgetHostViewAndroid::Hide() { return; is_showing_ = false; - if (layer_ && locks_on_frame_count_ == 0) + if (layer_.get() && locks_on_frame_count_ == 0) layer_->SetHideLayerAndSubtree(true); frame_evictor_->SetVisible(false); @@ -556,7 +556,7 @@ void RenderWidgetHostViewAndroid::UnlockCompositingSurface() { last_frame_info_.reset(); } - if (!is_showing_ && layer_) + if (!is_showing_ && layer_.get()) layer_->SetHideLayerAndSubtree(true); } } @@ -855,7 +855,7 @@ void RenderWidgetHostViewAndroid::CopyFromCompositingSurface( ui::WindowAndroidCompositor* compositor = content_view_core_->GetWindowAndroid()->GetCompositor(); DCHECK(compositor); - DCHECK(frame_provider_); + DCHECK(frame_provider_.get()); scoped_refptr<cc::DelegatedRendererLayer> delegated_layer = CreateDelegatedLayerForFrameProvider(); delegated_layer->SetHideLayerAndSubtree(true); @@ -914,7 +914,7 @@ void RenderWidgetHostViewAndroid::SendDelegatedFrameAck( void RenderWidgetHostViewAndroid::SendReturnedDelegatedResources( uint32 output_surface_id) { - DCHECK(resource_collection_); + DCHECK(resource_collection_.get()); cc::CompositorFrameAck ack; resource_collection_->TakeUnusedResourcesForChildCompositor(&ack.resources); @@ -976,7 +976,7 @@ void RenderWidgetHostViewAndroid::SwapDelegatedFrame( resource_collection_ = new cc::DelegatedFrameResourceCollection; resource_collection_->SetClient(this); } - if (!frame_provider_ || + if (!frame_provider_.get() || texture_size_in_layer_ != frame_provider_->frame_size()) { RemoveLayers(); frame_provider_ = new cc::DelegatedFrameProvider( @@ -1036,7 +1036,7 @@ void RenderWidgetHostViewAndroid::InternalSwapCompositorFrame( return; } - if (layer_ && layer_->layer_tree_host()) { + if (layer_.get() && layer_->layer_tree_host()) { for (size_t i = 0; i < frame->metadata.latency_info.size(); i++) { scoped_ptr<cc::SwapPromise> swap_promise( new cc::LatencyInfoSwapPromise(frame->metadata.latency_info[i])); @@ -1120,7 +1120,7 @@ void RenderWidgetHostViewAndroid::SynchronousFrameMetadata( } void RenderWidgetHostViewAndroid::SetOverlayVideoMode(bool enabled) { - if (layer_) + if (layer_.get()) layer_->SetContentsOpaque(!enabled); } @@ -1511,7 +1511,7 @@ SkColor RenderWidgetHostViewAndroid::GetCachedBackgroundColor() const { void RenderWidgetHostViewAndroid::DidOverscroll( const DidOverscrollParams& params) { - if (!content_view_core_ || !layer_ || !is_showing_) + if (!content_view_core_ || !layer_.get() || !is_showing_) return; const float device_scale_factor = content_view_core_->GetDpiScale(); |