Do not trigger Lo-Fi on main frame requests

components/data_reduction_proxy/content/browser/content_lofi_decider.cc

Index: components/data_reduction_proxy/content/browser/content_lofi_decider.cc
diff --git a/components/data_reduction_proxy/content/browser/content_lofi_decider.cc b/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
index 12b323dc659fbd0e6744089fee207021599081dd..970457b37d2299b9ef486fc1444e5a8b74fbe833 100644
--- a/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
+++ b/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
@@ -67,8 +67,7 @@ bool ContentLoFiDecider::MaybeAddLoFiDirectiveToHeaders(
 
   // If in the preview field trial or the preview flag is enabled, only add the
   // "q=preview" directive on main frame requests. Do not add Lo-Fi directives
-  // to other requests when previews are enabled. If previews are not enabled,
-  // add "q=low".
+  // to other requests when previews are enabled.
   if (lofi_preview_via_flag_or_field_trial) {
     if (request.load_flags() & net::LOAD_MAIN_FRAME) {
       if (params::AreLoFiPreviewsEnabledViaFlags()) {
@@ -77,12 +76,11 @@ bool ContentLoFiDecider::MaybeAddLoFiDirectiveToHeaders(
       }
       header_value += chrome_proxy_lo_fi_preview_directive();
     }
-  } else {
+  } else if (!(request.load_flags() & net::LOAD_MAIN_FRAME)) {
+    // If previews are not enabled, add "q=low" for main frame requests.

[megjablon, 2016/06/22 20:56:29]

If previews are not enabled, add "q=low" for requests that are not main frame
requests.

[tbansal1, 2016/06/22 21:41:02]

Done.

     header_value += chrome_proxy_lo_fi_directive();
   }
 
-  // |header_value| may be empty because the Lo-Fi directive is not added
-  // to subrequests when preview mode is enabled.
   if (!header_value.empty())
     headers->SetHeader(chrome_proxy_header(), header_value);