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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/log/net_log_event_type_list.h

Issue 2675343002: Server push cancellation: add NetLogs to track cache lookup transaction (Closed)
Patch Set: Always log the push lookup results Created 3 years, 10 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
« net/http/http_cache_lookup_manager.cc ('K') | « net/http/http_cache_lookup_manager_unittest.cc ('k') | net/log/net_log_source_type_list.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/log/net_log_event_type_list.h
diff --git a/net/log/net_log_event_type_list.h b/net/log/net_log_event_type_list.h
index bfdc0945062aa2c0b5477ed72ecaf358b175a18c..3b7eb62fd50c5caeb50b132984ad479fa5a15df8 100644
--- a/net/log/net_log_event_type_list.h
+++ b/net/log/net_log_event_type_list.h
@@ -1303,6 +1303,24 @@ EVENT_TYPE(BIDIRECTIONAL_STREAM_READY)
EVENT_TYPE(BIDIRECTIONAL_STREAM_FAILED)
// ------------------------------------------------------------------------
+// PUSH_LOOKUP_TRANSACTION
+// ------------------------------------------------------------------------
+
+// The start/end of a push lookup transaction for server push.
+// {
+// "source_dependency": <Source identifier for the server push lookp.
+// It can be a QUIC_SESSION or a HTTP2_SESSION>,
+// "pushed_url": <The url that has been pushed and looked up>,
+// }
+EVENT_TYPE(PUSH_LOOKUP_TRANSACTION)
+
+// Logs the result of the push lookup transaction when it completes.
+// {
+// "found_in_cache": <True if the pushed url is found in cache>,
+// }
+EVENT_TYPE(SERVER_PUSH_LOOKUP_COMPLETE)
+
+// ------------------------------------------------------------------------
// SpdySession
// ------------------------------------------------------------------------
« net/http/http_cache_lookup_manager.cc ('K') | « net/http/http_cache_lookup_manager_unittest.cc ('k') | net/log/net_log_source_type_list.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698