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

Unified Diff: net/tools/quic/quic_http_response_cache.cc

Issue 2629723003: Revert of Add quic_logging (Closed)
Patch Set: Created 3 years, 11 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 | « net/tools/quic/quic_epoll_connection_helper.cc ('k') | net/tools/quic/quic_packet_reader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_http_response_cache.cc
diff --git a/net/tools/quic/quic_http_response_cache.cc b/net/tools/quic/quic_http_response_cache.cc
index a409078f73751849721e262accb1f158d3a8f665..7eead7bb9d03e36db5288f1ecd11b6a0af15050d 100644
--- a/net/tools/quic/quic_http_response_cache.cc
+++ b/net/tools/quic/quic_http_response_cache.cc
@@ -12,7 +12,6 @@
#include "base/stl_util.h"
#include "net/http/http_util.h"
#include "net/quic/platform/api/quic_bug_tracker.h"
-#include "net/quic/platform/api/quic_logging.h"
#include "net/quic/platform/api/quic_text_utils.h"
#include "net/spdy/spdy_http_utils.h"
@@ -53,8 +52,8 @@
while (start < file_contents_.length()) {
size_t pos = file_contents_.find("\n", start);
if (pos == string::npos) {
- QUIC_LOG(DFATAL) << "Headers invalid or empty, ignoring: "
- << file_name_.value();
+ LOG(DFATAL) << "Headers invalid or empty, ignoring: "
+ << file_name_.value();
return;
}
size_t len = pos - start;
@@ -72,8 +71,8 @@
if (line.substr(0, 4) == "HTTP") {
pos = line.find(" ");
if (pos == string::npos) {
- QUIC_LOG(DFATAL) << "Headers invalid or empty, ignoring: "
- << file_name_.value();
+ LOG(DFATAL) << "Headers invalid or empty, ignoring: "
+ << file_name_.value();
return;
}
spdy_headers_[":status"] = line.substr(pos + 1, 3);
@@ -82,8 +81,8 @@
// Headers are "key: value".
pos = line.find(": ");
if (pos == string::npos) {
- QUIC_LOG(DFATAL) << "Headers invalid or empty, ignoring: "
- << file_name_.value();
+ LOG(DFATAL) << "Headers invalid or empty, ignoring: "
+ << file_name_.value();
return;
}
spdy_headers_.AppendValueOrAddHeader(
@@ -236,9 +235,8 @@
QUIC_BUG << "cache_directory must not be empty.";
return;
}
- QUIC_LOG(INFO)
- << "Attempting to initialize QuicHttpResponseCache from directory: "
- << cache_directory;
+ VLOG(1) << "Attempting to initialize QuicHttpResponseCache from directory: "
+ << cache_directory;
FilePath directory(FilePath::FromUTF8Unsafe(cache_directory));
base::FileEnumerator file_list(directory, true, base::FileEnumerator::FILES);
std::list<std::unique_ptr<ResourceFile>> resource_files;
@@ -294,8 +292,8 @@
for (auto it = resource_range.first; it != resource_range.second; ++it) {
resources.push_back(it->second);
}
- QUIC_DVLOG(1) << "Found " << resources.size() << " push resources for "
- << request_url;
+ DVLOG(1) << "Found " << resources.size() << " push resources for "
+ << request_url;
return resources;
}
@@ -325,7 +323,7 @@
new_response->set_headers(std::move(response_headers));
new_response->set_body(response_body);
new_response->set_trailers(std::move(response_trailers));
- QUIC_DVLOG(1) << "Add response with key " << key;
+ DVLOG(1) << "Add response with key " << key;
responses_[key] = std::move(new_response);
}
@@ -344,10 +342,9 @@
continue;
}
- QUIC_DVLOG(1) << "Add request-resource association: request url "
- << request_url << " push url " << push_resource.request_url
- << " response headers "
- << push_resource.headers.DebugString();
+ DVLOG(1) << "Add request-resource association: request url " << request_url
+ << " push url " << push_resource.request_url
+ << " response headers " << push_resource.headers.DebugString();
{
QuicWriterMutexLock lock(&response_mutex_);
server_push_resources_.insert(std::make_pair(request_url, push_resource));
@@ -366,8 +363,8 @@
if (!found_existing_response) {
// Add a server push response to responses map, if it is not in the map.
StringPiece body = push_resource.body;
- QUIC_DVLOG(1) << "Add response for push resource: host " << host
- << " path " << path;
+ DVLOG(1) << "Add response for push resource: host " << host << " path "
+ << path;
AddResponse(host, path, push_resource.headers.Clone(), body);
}
}
« no previous file with comments | « net/tools/quic/quic_epoll_connection_helper.cc ('k') | net/tools/quic/quic_packet_reader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698