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

Unified Diff: chrome/browser/renderer_host/resource_dispatcher_host.cc

Issue 4192012: Convert implicit scoped_refptr constructor calls to explicit ones, part 1 (Closed) Base URL: http://git.chromium.org/git/chromium.git
Patch Set: fix presubmit Created 10 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
Index: chrome/browser/renderer_host/resource_dispatcher_host.cc
diff --git a/chrome/browser/renderer_host/resource_dispatcher_host.cc b/chrome/browser/renderer_host/resource_dispatcher_host.cc
index b9d75b72803b35a2c7972ee8351ecbc0930943a6..28252b8ce1eaba70684a07869d7bd9063b0ac108 100644
--- a/chrome/browser/renderer_host/resource_dispatcher_host.cc
+++ b/chrome/browser/renderer_host/resource_dispatcher_host.cc
@@ -707,7 +707,7 @@ void ResourceDispatcherHost::BeginDownload(
request_id_--;
- scoped_refptr<ResourceHandler> handler =
+ scoped_refptr<ResourceHandler> handler(
new DownloadResourceHandler(this,
child_id,
route_id,
@@ -716,7 +716,7 @@ void ResourceDispatcherHost::BeginDownload(
download_file_manager_.get(),
request,
prompt_for_save_location,
- save_info);
+ save_info));
if (safe_browsing_->enabled()) {
handler = CreateSafeBrowsingResourceHandler(handler, child_id, route_id,
@@ -757,11 +757,11 @@ void ResourceDispatcherHost::BeginSaveFile(const GURL& url,
// requests. Does nothing if they are already loaded.
PluginService::GetInstance()->LoadChromePlugins(this);
- scoped_refptr<ResourceHandler> handler =
+ scoped_refptr<ResourceHandler> handler(
new SaveFileResourceHandler(child_id,
route_id,
url,
- save_file_manager_.get());
+ save_file_manager_.get()));
request_id_--;
bool known_proto = URLRequest::IsHandledURL(url);
@@ -1023,7 +1023,7 @@ void ResourceDispatcherHost::OnReceivedRedirect(URLRequest* request,
return;
}
- scoped_refptr<ResourceResponse> response = new ResourceResponse;
+ scoped_refptr<ResourceResponse> response(new ResourceResponse);
PopulateResourceResponse(request,
info->replace_extension_localization_templates(), response);
if (!info->resource_handler()->OnRequestRedirected(info->request_id(),
@@ -1123,7 +1123,7 @@ void ResourceDispatcherHost::OnResponseStarted(URLRequest* request) {
bool ResourceDispatcherHost::CompleteResponseStarted(URLRequest* request) {
ResourceDispatcherHostRequestInfo* info = InfoForRequest(request);
- scoped_refptr<ResourceResponse> response = new ResourceResponse;
+ scoped_refptr<ResourceResponse> response(new ResourceResponse);
PopulateResourceResponse(request,
info->replace_extension_localization_templates(), response);
« no previous file with comments | « chrome/browser/renderer_host/browser_render_process_host.cc ('k') | chrome/browser/renderer_host/resource_message_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698