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

Unified Diff: chrome/browser/ui/webui/favicon_source.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 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
« no previous file with comments | « chrome/browser/ui/webui/extensions/pack_extension_handler.cc ('k') | chrome/browser/ui/webui/nacl_ui.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/favicon_source.cc
diff --git a/chrome/browser/ui/webui/favicon_source.cc b/chrome/browser/ui/webui/favicon_source.cc
index 48359c998dd9b50607eda2e45173264079a9ea60..3970e0251935009898370468077f6df9eea668df 100644
--- a/chrome/browser/ui/webui/favicon_source.cc
+++ b/chrome/browser/ui/webui/favicon_source.cc
@@ -166,7 +166,7 @@ void FaviconSource::OnFaviconDataAvailable(
if (bitmap_result.is_valid()) {
// Forward the data along to the networking system.
- SendResponse(request.request_id, bitmap_result.bitmap_data);
+ SendResponse(request.request_id, bitmap_result.bitmap_data.get());
} else {
SendDefaultResponse(request);
}
@@ -189,7 +189,8 @@ void FaviconSource::SendDefaultResponse(const IconRequest& icon_request) {
resource_id = IDR_DEFAULT_FAVICON;
break;
}
- base::RefCountedMemory* default_favicon = default_favicons_[favicon_index];
+ base::RefCountedMemory* default_favicon =
+ default_favicons_[favicon_index].get();
if (!default_favicon) {
ui::ScaleFactor scale_factor = icon_request.scale_factor;
« no previous file with comments | « chrome/browser/ui/webui/extensions/pack_extension_handler.cc ('k') | chrome/browser/ui/webui/nacl_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698