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

Unified Diff: chrome/browser/safe_browsing/protocol_manager.cc

Issue 8416020: Handle additional feedback from http://codereview.chromium.org/8395038/. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 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/safe_browsing/protocol_manager.cc
===================================================================
--- chrome/browser/safe_browsing/protocol_manager.cc (revision 107626)
+++ chrome/browser/safe_browsing/protocol_manager.cc (working copy)
@@ -263,10 +263,10 @@
} else {
HandleGetHashError(Time::Now());
if (source->GetStatus().status() == net::URLRequestStatus::FAILED) {
- VLOG(1) << "SafeBrowsing GetHash request for: " << source->GetUrl()
+ VLOG(1) << "SafeBrowsing GetHash request for: " << source->GetURL()
<< " failed with error: " << source->GetStatus().error();
} else {
- VLOG(1) << "SafeBrowsing GetHash request for: " << source->GetUrl()
+ VLOG(1) << "SafeBrowsing GetHash request for: " << source->GetURL()
<< " failed with error: " << source->GetResponseCode();
}
}
@@ -297,9 +297,9 @@
std::string data;
source->GetResponseAsString(&data);
parsed_ok = HandleServiceResponse(
- source->GetUrl(), data.data(), static_cast<int>(data.length()));
+ source->GetURL(), data.data(), static_cast<int>(data.length()));
if (!parsed_ok) {
- VLOG(1) << "SafeBrowsing request for: " << source->GetUrl()
+ VLOG(1) << "SafeBrowsing request for: " << source->GetURL()
<< " failed parse.";
must_back_off = true;
chunk_request_urls_.clear();
@@ -338,10 +338,10 @@
chunk_request_urls_.clear();
UpdateFinished(false);
if (source->GetStatus().status() == net::URLRequestStatus::FAILED) {
- VLOG(1) << "SafeBrowsing request for: " << source->GetUrl()
+ VLOG(1) << "SafeBrowsing request for: " << source->GetURL()
<< " failed with error: " << source->GetStatus().error();
} else {
- VLOG(1) << "SafeBrowsing request for: " << source->GetUrl()
+ VLOG(1) << "SafeBrowsing request for: " << source->GetURL()
<< " failed with error: " << source->GetResponseCode();
}
}

Powered by Google App Engine
This is Rietveld 408576698