Update Effective Connection Type only on main frame requests

content/browser/loader/resource_loader.cc

Index: content/browser/loader/resource_loader.cc
diff --git a/content/browser/loader/resource_loader.cc b/content/browser/loader/resource_loader.cc
index c7d0b424634b51003e76945a08bc32ed6f4996cb..acd9a39f3ef5d1582a15306e3e06f86acb43a65a 100644
--- a/content/browser/loader/resource_loader.cc
+++ b/content/browser/loader/resource_loader.cc
@@ -90,7 +90,9 @@ void PopulateResourceResponse(ResourceRequestInfoImpl* info,
 
   response->head.effective_connection_type =
       net::EFFECTIVE_CONNECTION_TYPE_UNKNOWN;
-  if (info->IsMainFrame()) {
+
+  if (info->IsMainFrame() &&

[RyanSturm, 2016/08/08 18:13:37]

Is the info->IsMainFrame() part of this necessary? Seems Like
(info->GetResourceType() == RESOURCE_TYPE_MAIN_FRAME) implies
(info->IsMainFrame())

[tbansal1, 2016/08/08 18:35:25]

Done.

+      (info->GetResourceType() == RESOURCE_TYPE_MAIN_FRAME)) {

[RyanSturm, 2016/08/08 18:13:37]

#include "content/public/common/resource_type.h"

This should have already been included.

[tbansal1, 2016/08/08 18:35:25]

Done.

     net::NetworkQualityEstimator* estimator =
         request->context()->network_quality_estimator();
     if (estimator) {