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

Unified Diff: chrome/common/resource_dispatcher.cc

Issue 326003: Make GetURLForDebugging return a const GURL (Closed)
Patch Set: no const Created 11 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 | « no previous file | chrome/common/resource_dispatcher_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/resource_dispatcher.cc
diff --git a/chrome/common/resource_dispatcher.cc b/chrome/common/resource_dispatcher.cc
index 47efd6b4cce9bd33b9857193c4b4c5a4982b676e..de66f9dca6d9c2ebc0276d455a88cc4b75fd66d5 100644
--- a/chrome/common/resource_dispatcher.cc
+++ b/chrome/common/resource_dispatcher.cc
@@ -316,7 +316,7 @@ void ResourceDispatcher::OnUploadProgress(
PendingRequestInfo& request_info = it->second;
RESOURCE_LOG("Dispatching upload progress for " <<
- request_info.peer->GetURLForDebugging());
+ request_info.peer->GetURLForDebugging().possibly_invalid_spec());
request_info.peer->OnUploadProgress(position, size);
// Acknowlegde reciept
@@ -351,7 +351,8 @@ void ResourceDispatcher::OnReceivedResponse(
}
}
- RESOURCE_LOG("Dispatching response for " << peer->GetURLForDebugging());
+ RESOURCE_LOG("Dispatching response for " <<
+ peer->GetURLForDebugging().possibly_invalid_spec());
peer->OnReceivedResponse(response_head, false);
}
@@ -379,7 +380,7 @@ void ResourceDispatcher::OnReceivedData(const IPC::Message& message,
if (data_len > 0 && shared_mem.Map(data_len)) {
RESOURCE_LOG("Dispatching " << data_len << " bytes for " <<
- request_info.peer->GetURLForDebugging());
+ request_info.peer->GetURLForDebugging().possibly_invalid_spec());
const char* data = static_cast<char*>(shared_mem.memory());
request_info.peer->OnReceivedData(data, data_len);
}
@@ -401,7 +402,7 @@ void ResourceDispatcher::OnReceivedRedirect(
PendingRequestInfo& request_info = it->second;
RESOURCE_LOG("Dispatching redirect for " <<
- request_info.peer->GetURLForDebugging());
+ request_info.peer->GetURLForDebugging().possibly_invalid_spec());
if (request_info.peer->OnReceivedRedirect(new_url, info)) {
message_sender()->Send(
@@ -426,7 +427,7 @@ void ResourceDispatcher::OnRequestComplete(int request_id,
webkit_glue::ResourceLoaderBridge::Peer* peer = request_info.peer;
RESOURCE_LOG("Dispatching complete for " <<
- request_info.peer->GetURLForDebugging());
+ request_info.peer->GetURLForDebugging().possibly_invalid_spec());
if (status.status() == URLRequestStatus::CANCELED &&
status.os_error() != net::ERR_ABORTED) {
« no previous file with comments | « no previous file | chrome/common/resource_dispatcher_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698