Separate out observation sources as either HTTP layer or transport layer

net/nqe/network_quality_observation_source.h

Index: net/nqe/network_quality_observation_source.h
diff --git a/net/nqe/network_quality_observation_source.h b/net/nqe/network_quality_observation_source.h
index c1614027f277a84d25946ef31ca073e51bb0e528..0b784267f9829450314315ae01b0dba5b7c72cea 100644
--- a/net/nqe/network_quality_observation_source.h
+++ b/net/nqe/network_quality_observation_source.h
@@ -15,24 +15,57 @@ enum NetworkQualityObservationSource {
   // The observation was taken at the request layer, e.g., a round trip time
   // is recorded as the time between the request being sent and the first byte
   // being received.
-  NETWORK_QUALITY_OBSERVATION_SOURCE_URL_REQUEST = 0,
+  NETWORK_QUALITY_OBSERVATION_SOURCE_HTTP = 0,
 
   // The observation is taken from TCP statistics maintained by the kernel.
-  NETWORK_QUALITY_OBSERVATION_SOURCE_TCP,
+  // This source is depecated and should not be used.
+  //  DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_TCP,

[RyanSturm, 2016/10/31 21:13:46]

nit:s/DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_TCP,/DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_TCP
= 1,

and the rest of the deprecated entries

[tbansal1, 2016/10/31 23:40:04]

Done.

 
   // The observation is taken at the QUIC layer.
-  NETWORK_QUALITY_OBSERVATION_SOURCE_QUIC,
+  // This source is depecated and should not be used.
+  // DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_QUIC,
 
   // The observation is a previously cached estimate of the metric.
-  NETWORK_QUALITY_OBSERVATION_SOURCE_CACHED_ESTIMATE,
+  // This source is depecated and should not be used.
+  // DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_CACHED_ESTIMATE,
 
   // The observation is derived from network connection information provided
   // by the platform. For example, typical RTT and throughput values are used
   // for a given type of network connection.
-  NETWORK_QUALITY_OBSERVATION_SOURCE_DEFAULT_FROM_PLATFORM,
+  // This source is depecated and should not be used.
+  // DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_DEFAULT_FROM_PLATFORM,
 
   // The observation came from a Chromium-external source.
-  NETWORK_QUALITY_OBSERVATION_SOURCE_EXTERNAL_ESTIMATE,
+  // This source is depecated and should not be used.
+  // DEPRECATED_NETWORK_QUALITY_OBSERVATION_SOURCE_EXTERNAL_ESTIMATE,
+
+  // The observation was taken at the transport layer from the UDP sockets (when
+  // using QUIC), or from the TCP sockets.
+  NETWORK_QUALITY_OBSERVATION_SOURCE_TRANSPORT = 6,
+
+  // The observation is a previously cached estimate of the metric. The metric
+  // was computed at the HTTP layer.
+  NETWORK_QUALITY_OBSERVATION_SOURCE_HTTP_CACHED_ESTIMATE = 7,
+
+  // The observation is a previously cached estimate of the metric. The metric
+  // was computed at the transport layer.
+  NETWORK_QUALITY_OBSERVATION_SOURCE_TRANSPORT_CACHED_ESTIMATE = 8,
+
+  // The observation is derived from the network connection information provided
+  // by the platform. For example, typical RTT and throughput values are used
+  // for a given type of network connection.  The metric was provided for use
+  // at the HTTP layer.
+  NETWORK_QUALITY_OBSERVATION_SOURCE_DEFAULT_HTTP_FROM_PLATFORM = 9,
+
+  // The observation is derived from the network connection information provided
+  // by the platform. For example, typical RTT and throughput values are used
+  // for a given type of network connection.  The metric was provided for use
+  // at the transport layer.
+  NETWORK_QUALITY_OBSERVATION_SOURCE_DEFAULT_TRANSPORT_FROM_PLATFORM = 10,
+
+  // The observation came from a Chromium-external source. The metric was
+  // computed by the external source at the HTTP layer.
+  NETWORK_QUALITY_OBSERVATION_SOURCE_HTTP_EXTERNAL_ESTIMATE = 11,
 
   NETWORK_QUALITY_OBSERVATION_SOURCE_MAX,
 };