Expose ECT to render frames, Blink and NetInfo

content/common/resource_messages.h

Index: content/common/resource_messages.h
diff --git a/content/common/resource_messages.h b/content/common/resource_messages.h
index 12e47d5f87a485a3407df3d0e4f103fa1f2742a9..41a6dc2a2abaaa99a5d187324b95905dd7789ec8 100644
--- a/content/common/resource_messages.h
+++ b/content/common/resource_messages.h
@@ -26,7 +26,6 @@
 #include "net/cert/signed_certificate_timestamp.h"
 #include "net/cert/signed_certificate_timestamp_and_status.h"
 #include "net/http/http_response_info.h"
-#include "net/nqe/effective_connection_type.h"
 #include "net/ssl/ssl_info.h"
 #include "net/url_request/redirect_info.h"
 #include "third_party/WebKit/public/platform/WebMixedContentContextType.h"
@@ -170,9 +169,6 @@ IPC_ENUM_TRAITS_MAX_VALUE(content::FetchRedirectMode,
 IPC_ENUM_TRAITS_MAX_VALUE(content::ServiceWorkerMode,
                           content::ServiceWorkerMode::LAST)
 
-IPC_ENUM_TRAITS_MAX_VALUE(net::EffectiveConnectionType,
-                          net::EFFECTIVE_CONNECTION_TYPE_LAST - 1)
-
 IPC_ENUM_TRAITS_MAX_VALUE(blink::WebMixedContentContextType,
                           blink::WebMixedContentContextType::kLast)
 
@@ -220,7 +216,6 @@ IPC_STRUCT_TRAITS_BEGIN(content::ResourceResponseInfo)
   IPC_STRUCT_TRAITS_MEMBER(cache_storage_cache_name)
   IPC_STRUCT_TRAITS_MEMBER(did_service_worker_navigation_preload)
   IPC_STRUCT_TRAITS_MEMBER(previews_state)
-  IPC_STRUCT_TRAITS_MEMBER(effective_connection_type)

[nasko, 2017/05/23 21:42:04]

Why is this removed from here and not the actual definition of
content::ResourceResponseInfo? This macro is supposed to describe the real C++
struct defined elsewhere and with this change they will mismatch.

[tbansal1, 2017/05/23 22:37:11]

Thanks. Fixed.
Also, filed crbug.com/725680 for cleanup.

   IPC_STRUCT_TRAITS_MEMBER(certificate)
   IPC_STRUCT_TRAITS_MEMBER(cert_status)
   IPC_STRUCT_TRAITS_MEMBER(ssl_connection_status)