From cadac62e5c2b9f5fef59ce0326bb2cd79ffbe622 Mon Sep 17 00:00:00 2001 From: "rsleevi@chromium.org" <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Date: Tue, 11 Jun 2013 16:46:36 +0000 Subject: Call scoped_refptr<T>::get() rather than relying on implicit "operator T*" This upates calls to bound temporary objects to also use get(). While it has the same semantic equivalence to the existing code, this generally represents a dangerous pattern - indeed, part of the whole motivation for this change is to make this anti-pattern very visible to authors. This change simply updates all of the call sites, to allow the "operator T*" to be removed and preventing new instances. The existing instances will then be reviewed for "suspicious" changes and updated to use/pass scoped_refptr<T> rather than T*, as appropriate. BUG=110610 TBR=darin Review URL: https://codereview.chromium.org/15984016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205560 0039d316-1c4b-4281-b951-d872f2087c98 --- content/browser/gpu/shader_disk_cache.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'content/browser/gpu/shader_disk_cache.cc') diff --git a/content/browser/gpu/shader_disk_cache.cc b/content/browser/gpu/shader_disk_cache.cc index 3e07572..a9ad9b8 100644 --- a/content/browser/gpu/shader_disk_cache.cc +++ b/content/browser/gpu/shader_disk_cache.cc @@ -529,7 +529,7 @@ void ShaderDiskCache::Init() { cache_path_.Append(kGpuCachePath), max_cache_size_, true, - BrowserThread::GetMessageLoopProxyForThread(BrowserThread::CACHE), + BrowserThread::GetMessageLoopProxyForThread(BrowserThread::CACHE).get(), NULL, &backend_, base::Bind(&ShaderDiskCache::CacheCreatedCallback, this)); -- cgit v1.1