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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/http/http_cache_unittest.cc

Issue 2315613002: Extracted NetLog class's inner enum types into their own enum classes and (Closed)
Patch Set: Created 4 years, 3 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
« chrome/browser/renderer_host/safe_browsing_resource_throttle.cc ('K') | « net/http/http_cache_transaction.cc ('k') | net/http/http_network_transaction.cc » ('j') | net/log/net_log_event_type_list.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_cache_unittest.cc
diff --git a/net/http/http_cache_unittest.cc b/net/http/http_cache_unittest.cc
index d580c6b3a60ce0397a092a61a0c05edebd947ed9..16398badfe8a86a79a74baf2b75f3a0156d1cf28 100644
--- a/net/http/http_cache_unittest.cc
+++ b/net/http/http_cache_unittest.cc
@@ -43,6 +43,7 @@
#include "net/http/http_transaction_test_util.h"
#include "net/http/http_util.h"
#include "net/http/mock_http_cache.h"
+#include "net/log/net_log_event_type.h"
#include "net/log/test_net_log.h"
#include "net/log/test_net_log_entry.h"
#include "net/log/test_net_log_util.h"
@@ -596,12 +597,12 @@ class FakeWebSocketHandshakeStreamCreateHelper
// ignored.
bool ShouldIgnoreLogEntry(const TestNetLogEntry& entry) {
switch (entry.type) {
- case NetLog::TYPE_HTTP_CACHE_GET_BACKEND:
- case NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY:
- case NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY:
- case NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY:
- case NetLog::TYPE_HTTP_CACHE_DOOM_ENTRY:
- case NetLog::TYPE_HTTP_CACHE_READ_INFO:
+ case NetLogEventType::HTTP_CACHE_GET_BACKEND:
+ case NetLogEventType::HTTP_CACHE_OPEN_ENTRY:
+ case NetLogEventType::HTTP_CACHE_CREATE_ENTRY:
+ case NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY:
+ case NetLogEventType::HTTP_CACHE_DOOM_ENTRY:
+ case NetLogEventType::HTTP_CACHE_READ_INFO:
return false;
default:
return true;
@@ -617,7 +618,7 @@ void FilterLogEntries(TestNetLogEntry::List* entries) {
}
bool LogContainsEventType(const BoundTestNetLog& log,
- NetLog::EventType expected) {
+ NetLogEventType expected) {
TestNetLogEntry::List entries;
log.GetEntries(&entries);
for (size_t i = 0; i < entries.size(); i++) {
@@ -686,17 +687,23 @@ TEST(HttpCache, SimpleGETNoDiskCache) {
EXPECT_EQ(6u, entries.size());
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 0, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsBeginEvent(entries, 0,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 1, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsEndEvent(entries, 1,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 2, NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY));
+ LogContainsBeginEvent(entries, 2,
+ NetLogEventType::HTTP_CACHE_OPEN_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 3, NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY));
+ LogContainsEndEvent(entries, 3,
+ NetLogEventType::HTTP_CACHE_OPEN_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 4, NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY));
+ LogContainsBeginEvent(entries, 4,
+ NetLogEventType::HTTP_CACHE_CREATE_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 5, NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY));
+ LogContainsEndEvent(entries, 5,
+ NetLogEventType::HTTP_CACHE_CREATE_ENTRY));
EXPECT_EQ(1, cache.network_layer()->transaction_count());
EXPECT_EQ(0, cache.disk_cache()->open_count());
@@ -852,21 +859,29 @@ TEST(HttpCache, SimpleGET_LoadOnlyFromCache_Hit) {
EXPECT_EQ(8u, entries.size());
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 0, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsBeginEvent(entries, 0,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 1, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsEndEvent(entries, 1,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 2, NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY));
+ LogContainsBeginEvent(entries, 2,
+ NetLogEventType::HTTP_CACHE_OPEN_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 3, NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY));
+ LogContainsEndEvent(entries, 3,
+ NetLogEventType::HTTP_CACHE_OPEN_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 4, NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY));
+ LogContainsBeginEvent(entries, 4,
+ NetLogEventType::HTTP_CACHE_CREATE_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 5, NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY));
+ LogContainsEndEvent(entries, 5,
+ NetLogEventType::HTTP_CACHE_CREATE_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 6, NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY));
+ LogContainsBeginEvent(entries, 6,
+ NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 7, NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY));
+ LogContainsEndEvent(entries, 7,
+ NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY));
TestLoadTimingNetworkRequest(load_timing_info);
@@ -885,21 +900,29 @@ TEST(HttpCache, SimpleGET_LoadOnlyFromCache_Hit) {
EXPECT_EQ(8u, entries.size());
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 0, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsBeginEvent(entries, 0,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 1, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsEndEvent(entries, 1,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 2, NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY));
+ LogContainsBeginEvent(entries, 2,
+ NetLogEventType::HTTP_CACHE_OPEN_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 3, NetLog::TYPE_HTTP_CACHE_OPEN_ENTRY));
+ LogContainsEndEvent(entries, 3,
+ NetLogEventType::HTTP_CACHE_OPEN_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 4, NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY));
+ LogContainsBeginEvent(entries, 4,
+ NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 5, NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY));
+ LogContainsEndEvent(entries, 5,
+ NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 6, NetLog::TYPE_HTTP_CACHE_READ_INFO));
+ LogContainsBeginEvent(entries, 6,
+ NetLogEventType::HTTP_CACHE_READ_INFO));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 7, NetLog::TYPE_HTTP_CACHE_READ_INFO));
+ LogContainsEndEvent(entries, 7,
+ NetLogEventType::HTTP_CACHE_READ_INFO));
EXPECT_EQ(1, cache.network_layer()->transaction_count());
EXPECT_EQ(1, cache.disk_cache()->open_count());
@@ -1113,21 +1136,29 @@ TEST(HttpCache, SimpleGET_LoadBypassCache) {
EXPECT_EQ(8u, entries.size());
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 0, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsBeginEvent(entries, 0,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 1, NetLog::TYPE_HTTP_CACHE_GET_BACKEND));
+ LogContainsEndEvent(entries, 1,
+ NetLogEventType::HTTP_CACHE_GET_BACKEND));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 2, NetLog::TYPE_HTTP_CACHE_DOOM_ENTRY));
+ LogContainsBeginEvent(entries, 2,
+ NetLogEventType::HTTP_CACHE_DOOM_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 3, NetLog::TYPE_HTTP_CACHE_DOOM_ENTRY));
+ LogContainsEndEvent(entries, 3,
+ NetLogEventType::HTTP_CACHE_DOOM_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 4, NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY));
+ LogContainsBeginEvent(entries, 4,
+ NetLogEventType::HTTP_CACHE_CREATE_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 5, NetLog::TYPE_HTTP_CACHE_CREATE_ENTRY));
+ LogContainsEndEvent(entries, 5,
+ NetLogEventType::HTTP_CACHE_CREATE_ENTRY));
EXPECT_TRUE(
- LogContainsBeginEvent(entries, 6, NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY));
+ LogContainsBeginEvent(entries, 6,
+ NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY));
EXPECT_TRUE(
- LogContainsEndEvent(entries, 7, NetLog::TYPE_HTTP_CACHE_ADD_TO_ENTRY));
+ LogContainsEndEvent(entries, 7,
+ NetLogEventType::HTTP_CACHE_ADD_TO_ENTRY));
EXPECT_EQ(2, cache.network_layer()->transaction_count());
EXPECT_EQ(0, cache.disk_cache()->open_count());
@@ -3797,7 +3828,7 @@ TEST(HttpCache, SimpleGET_DoesntLogHeaders) {
log.bound());
EXPECT_FALSE(LogContainsEventType(
- log, NetLog::TYPE_HTTP_CACHE_CALLER_REQUEST_HEADERS));
+ log, NetLogEventType::HTTP_CACHE_CALLER_REQUEST_HEADERS));
}
TEST(HttpCache, RangeGET_LogsHeaders) {
@@ -3808,7 +3839,7 @@ TEST(HttpCache, RangeGET_LogsHeaders) {
log.bound());
EXPECT_TRUE(LogContainsEventType(
- log, NetLog::TYPE_HTTP_CACHE_CALLER_REQUEST_HEADERS));
+ log, NetLogEventType::HTTP_CACHE_CALLER_REQUEST_HEADERS));
}
TEST(HttpCache, ExternalValidation_LogsHeaders) {
@@ -3820,7 +3851,7 @@ TEST(HttpCache, ExternalValidation_LogsHeaders) {
RunTransactionTestWithLog(cache.http_cache(), transaction, log.bound());
EXPECT_TRUE(LogContainsEventType(
- log, NetLog::TYPE_HTTP_CACHE_CALLER_REQUEST_HEADERS));
+ log, NetLogEventType::HTTP_CACHE_CALLER_REQUEST_HEADERS));
}
TEST(HttpCache, SpecialHeaders_LogsHeaders) {
@@ -3832,7 +3863,7 @@ TEST(HttpCache, SpecialHeaders_LogsHeaders) {
RunTransactionTestWithLog(cache.http_cache(), transaction, log.bound());
EXPECT_TRUE(LogContainsEventType(
- log, NetLog::TYPE_HTTP_CACHE_CALLER_REQUEST_HEADERS));
+ log, NetLogEventType::HTTP_CACHE_CALLER_REQUEST_HEADERS));
}
// Tests that receiving 206 for a regular request is handled correctly.
@@ -3948,7 +3979,7 @@ TEST(HttpCache, RangeGET_NoValidation_LogsRestart) {
log.bound());
EXPECT_TRUE(LogContainsEventType(
- log, NetLog::TYPE_HTTP_CACHE_RESTART_PARTIAL_REQUEST));
+ log, NetLogEventType::HTTP_CACHE_RESTART_PARTIAL_REQUEST));
}
// Tests that a failure to conditionalize a regular request (no range) with a
@@ -5498,7 +5529,7 @@ TEST(HttpCache, RangeGET_FastFlakyServer) {
EXPECT_EQ(1, cache.disk_cache()->open_count());
EXPECT_EQ(1, cache.disk_cache()->create_count());
EXPECT_TRUE(LogContainsEventType(
- log, NetLog::TYPE_HTTP_CACHE_RE_SEND_PARTIAL_REQUEST));
+ log, NetLogEventType::HTTP_CACHE_RE_SEND_PARTIAL_REQUEST));
}
// Tests that when the server gives us less data than expected, we don't keep
« chrome/browser/renderer_host/safe_browsing_resource_throttle.cc ('K') | « net/http/http_cache_transaction.cc ('k') | net/http/http_network_transaction.cc » ('j') | net/log/net_log_event_type_list.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698