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

Unified Diff: chrome/browser/thumbnails/thumbnail_service_impl.cc

Issue 16290004: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/thumbnails/thumbnail_service_impl.cc
diff --git a/chrome/browser/thumbnails/thumbnail_service_impl.cc b/chrome/browser/thumbnails/thumbnail_service_impl.cc
index 3b108ac345ca11a96cd6be79bddc759e4f879f5e..21f7227925edeb7253ad9e1f1d3f713835f11d28 100644
--- a/chrome/browser/thumbnails/thumbnail_service_impl.cc
+++ b/chrome/browser/thumbnails/thumbnail_service_impl.cc
@@ -29,7 +29,7 @@ ThumbnailServiceImpl::~ThumbnailServiceImpl() {
bool ThumbnailServiceImpl::SetPageThumbnail(const ThumbnailingContext& context,
const gfx::Image& thumbnail) {
scoped_refptr<history::TopSites> local_ptr(top_sites_);
- if (local_ptr == NULL)
+ if (local_ptr.get() == NULL)
return false;
return local_ptr->SetPageThumbnail(context.url, thumbnail, context.score);
@@ -39,7 +39,7 @@ bool ThumbnailServiceImpl::GetPageThumbnail(
const GURL& url,
scoped_refptr<base::RefCountedMemory>* bytes) {
scoped_refptr<history::TopSites> local_ptr(top_sites_);
- if (local_ptr == NULL)
+ if (local_ptr.get() == NULL)
return false;
return local_ptr->GetPageThumbnail(url, bytes);
@@ -53,7 +53,7 @@ ThumbnailingAlgorithm* ThumbnailServiceImpl::GetThumbnailingAlgorithm()
bool ThumbnailServiceImpl::ShouldAcquirePageThumbnail(const GURL& url) {
scoped_refptr<history::TopSites> local_ptr(top_sites_);
- if (local_ptr == NULL)
+ if (local_ptr.get() == NULL)
return false;
// Skip if the given URL is not appropriate for history.
« no previous file with comments | « chrome/browser/themes/theme_syncable_service_unittest.cc ('k') | chrome/browser/thumbnails/thumbnail_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698