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

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: address eroman's comments 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
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..5b0ed678c6d53816e9cefd767f20574791bbda5f 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)
// ------------------------------------------------------------------------
+// SERVER_PUSH_LOOKUP_TRANSACTION
+// ------------------------------------------------------------------------
+
+// The start/end of a push lookup transaction for server push.
+// {
eroman 2017/02/10 23:57:47 Please clarify which event these parameters are at
Zhongyi Shi 2017/02/11 02:21:11 Done.
+// "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(SERVER_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)
eroman 2017/02/10 23:57:47 See comments in the .cc file about merging this wi
Zhongyi Shi 2017/02/11 02:21:11 Done.
+
+// ------------------------------------------------------------------------
// SpdySession
// ------------------------------------------------------------------------

Powered by Google App Engine
This is Rietveld 408576698