Add NetLog events for HttpServerPropertiesManager

net/log/net_log_event_type_list.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // NOTE: No header guards are used, since this file is intended to be expanded
 // directly into net_log.h. DO NOT include this file anywhere else.
 
 // In the event of a failure, a many end events will have a |net_error|
 // parameter with the integer error code associated with the failure.  Most
 // of these parameters are not individually documented.
@@ skipping 3162 matching lines @@
 // quality of the network has changed.
 // parameters:
 //  {
 //    "http_rtt": <Current estimate of the HTTP RTT>,
 //    "transport_rtt": <Current estimate of the transport RTT>,
 //    "downstream_throughput": <Current estimate of the downstream throughput>,
 //    "effective_connection_type": <Current estimate of the effective connection
 //                                  type>,
 //  }
 EVENT_TYPE(NETWORK_QUALITY_CHANGED)
+
+// -----------------------------------------------------------------------------
+// Http Server Properties Manager related events
+// -----------------------------------------------------------------------------
+
+// This event is emitted when HttpServerPropertiesManager initialization starts
+// and finishes.
+EVENT_TYPE(HTTP_SERVER_PROPERTIES_INITIALIZATION)

[Zhongyi Shi, 2017/06/19 19:50:24]

nit: since these are all manager specific, could you change those event_type
names to be HTPP_SERVER_PROPERTIES_MANAGER_*

[xunjieli, 2017/06/19 20:09:32]

Do you feel strongly about adding a MANAGER_ suffix?
The names are getting rather long. Some of the events are not named exactly
after the classes. E.g. We have HTTP_STREAM_JOB_CONTROLLER rather than the full
class name HTTP_STREAM_FACTORY_[IMPL_JOB]_CONTROLLER. We might in the future add
NetLog events in HttpServerProperties and group them together with these. My
preference will be to keep the names short while not introducing ambiguity.

[Zhongyi Shi, 2017/06/20 20:00:53]

Acknowledged. Okay, sounds like you have considered this. Yeah, I was thinking
that we  might will add NetLog events for HttpServerPropertiesImpl, and we might
want to differentiate or aggregate the NetLogs from impl and manager. 

+
+// This event is emitted when HttpServerPropertiesManager is updating in-memory
+// version of HttpServerProperties from the serialized version from perfs/disk.
+EVENT_TYPE(HTTP_SERVER_PROPERTIES_UPDATE_CACHE)
+
+// This event is emitted when HttpServerPropertiesManager is persisting
+// in-memory version of HttpServerProperties to prefs/disk.
+EVENT_TYPE(HTTP_SERVER_PROPERTIES_UPDATE_PREFS)