Index: net/http/http_cache_transaction.cc |
diff --git a/net/http/http_cache_transaction.cc b/net/http/http_cache_transaction.cc |
index 0b0700c123f21dcdad7540f8746383ed86c7072d..896db76e1586ab01e41e74120ba83da8b6017633 100644 |
--- a/net/http/http_cache_transaction.cc |
+++ b/net/http/http_cache_transaction.cc |
@@ -215,23 +215,6 @@ void RecordOfflineStatus(int load_flags, RequestOfflineStatus status) { |
} |
} |
-// TODO(rvargas): Remove once we get the data. |
-void RecordVaryHeaderHistogram(const net::HttpResponseInfo* response) { |
- enum VaryType { |
- VARY_NOT_PRESENT, |
- VARY_UA, |
- VARY_OTHER, |
- VARY_MAX |
- }; |
- VaryType vary = VARY_NOT_PRESENT; |
- if (response->vary_data.is_valid()) { |
- vary = VARY_OTHER; |
- if (response->headers->HasHeaderValue("vary", "user-agent")) |
- vary = VARY_UA; |
- } |
- UMA_HISTOGRAM_ENUMERATION("HttpCache.Vary", vary, VARY_MAX); |
-} |
- |
void RecordNoStoreHeaderHistogram(int load_flags, |
const net::HttpResponseInfo* response) { |
if (load_flags & net::LOAD_MAIN_FRAME) { |
@@ -1227,7 +1210,6 @@ int HttpCache::Transaction::DoSuccessfulSendRequest() { |
cache_->DoomMainEntryForUrl(request_->url); |
} |
- RecordVaryHeaderHistogram(new_response); |
RecordNoStoreHeaderHistogram(request_->load_flags, new_response); |
if (new_response_->headers->response_code() == 416 && |
@@ -1361,10 +1343,7 @@ int HttpCache::Transaction::DoCreateEntryComplete(int result) { |
return OK; |
} |
- if (result == OK) { |
- UMA_HISTOGRAM_BOOLEAN("HttpCache.OpenToCreateRace", false); |
- } else { |
- UMA_HISTOGRAM_BOOLEAN("HttpCache.OpenToCreateRace", true); |
+ if (result != OK) { |
// We have a race here: Maybe we failed to open the entry and decided to |
// create one, but by the time we called create, another transaction already |
// created the entry. If we want to eliminate this issue, we need an atomic |