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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // NOTE: No header guards are used, since this file is intended to be expanded 5 // NOTE: No header guards are used, since this file is intended to be expanded
6 // directly into net_log.h. DO NOT include this file anywhere else. 6 // directly into net_log.h. DO NOT include this file anywhere else.
7 7
8 // In the event of a failure, a many end events will have a |net_error| 8 // In the event of a failure, a many end events will have a |net_error|
9 // parameter with the integer error code associated with the failure. Most 9 // parameter with the integer error code associated with the failure. Most
10 // of these parameters are not individually documented. 10 // of these parameters are not individually documented.
(...skipping 1285 matching lines...) Expand 10 before | Expand all | Expand 10 after
1296 EVENT_TYPE(BIDIRECTIONAL_STREAM_READY) 1296 EVENT_TYPE(BIDIRECTIONAL_STREAM_READY)
1297 1297
1298 // This event is used when stream has failed. 1298 // This event is used when stream has failed.
1299 // The following parameters are attached: 1299 // The following parameters are attached:
1300 // { 1300 // {
1301 // "net_error": <Net error code for the failure>, 1301 // "net_error": <Net error code for the failure>,
1302 // } 1302 // }
1303 EVENT_TYPE(BIDIRECTIONAL_STREAM_FAILED) 1303 EVENT_TYPE(BIDIRECTIONAL_STREAM_FAILED)
1304 1304
1305 // ------------------------------------------------------------------------ 1305 // ------------------------------------------------------------------------
1306 // SERVER_PUSH_LOOKUP_TRANSACTION
1307 // ------------------------------------------------------------------------
1308
1309 // The start/end of a push lookup transaction for server push.
1310 //
1311 // The START event has the parameters:
1312 // {
1313 // "source_dependency": <Source identifier for the server push lookp.
1314 // It can be a QUIC_SESSION or a HTTP2_SESSION>,
1315 // "pushed_url": <The url that has been pushed and looked up>,
1316 // }
1317 //
1318 // If the transaction doesn't find the resource in cache, then the END phase
eroman 2017/02/11 02:44:36 nit: de-indent this by two spaces?
Zhongyi Shi 2017/02/11 09:42:26 Done. I forgot to upload the patch set which fixes
1319 // has these parameters:
1320 // {
1321 // "net_error": <Net error code integer>,
1322 // }
1323 EVENT_TYPE(SERVER_PUSH_LOOKUP_TRANSACTION)
1324
1325 // ------------------------------------------------------------------------
1306 // SpdySession 1326 // SpdySession
1307 // ------------------------------------------------------------------------ 1327 // ------------------------------------------------------------------------
1308 1328
1309 // The start/end of a SpdySession. 1329 // The start/end of a SpdySession.
1310 // { 1330 // {
1311 // "host": <The host-port string>, 1331 // "host": <The host-port string>,
1312 // "proxy": <The Proxy PAC string>, 1332 // "proxy": <The Proxy PAC string>,
1313 // } 1333 // }
1314 EVENT_TYPE(HTTP2_SESSION) 1334 EVENT_TYPE(HTTP2_SESSION)
1315 1335
(...skipping 1784 matching lines...) Expand 10 before | Expand all | Expand 10 after
3100 // quality of the network has changed. 3120 // quality of the network has changed.
3101 // parameters: 3121 // parameters:
3102 // { 3122 // {
3103 // "http_rtt": <Current estimate of the HTTP RTT>, 3123 // "http_rtt": <Current estimate of the HTTP RTT>,
3104 // "transport_rtt": <Current estimate of the transport RTT>, 3124 // "transport_rtt": <Current estimate of the transport RTT>,
3105 // "downstream_throughput": <Current estimate of the downstream throughput>, 3125 // "downstream_throughput": <Current estimate of the downstream throughput>,
3106 // "effective_connection_type": <Current estimate of the effective connection 3126 // "effective_connection_type": <Current estimate of the effective connection
3107 // type>, 3127 // type>,
3108 // } 3128 // }
3109 EVENT_TYPE(NETWORK_QUALITY_CHANGED) 3129 EVENT_TYPE(NETWORK_QUALITY_CHANGED)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698