DataReductionProxy proto change to use exp=force_lite_page directive

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 dc203b99319c979c4bf6ea15c64da066277bc8e5..c2c1555eb14f5cbda5da79b1927de4270451a461 100644
--- a/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
+++ b/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
@@ -180,7 +180,7 @@ void ContentLoFiDecider::RemoveAcceptTransformHeader(
   headers->RemoveHeader(chrome_proxy_accept_transform_header());
 }
 
-void ContentLoFiDecider::MaybeSetIgnorePreviewsBlacklistDirective(
+void ContentLoFiDecider::MaybeSetForceLitePageDirective(
     net::HttpRequestHeaders* headers) const {
   if (!headers || !params::AreLitePagesEnabledViaFlags() ||

[megjablon, 2017/05/23 00:19:07]

Should we also check !params::IsLoFiSlowConnectionsOnlyViaFlags() here?

[dougarnett, 2017/05/23 17:02:16]

Hmm, I think we would need to check an OR with at least Always-On and
Cellular-Only as well as SlowConnectionViaFlags if so. Not clear to me what we
gain there (disable for mix of LoFi field trial and LitePages flag?).

       !IsLitePagePreviewRequested(*headers)) {
@@ -191,8 +191,7 @@ void ContentLoFiDecider::MaybeSetIgnorePreviewsBlacklistDirective(
   headers->RemoveHeader(chrome_proxy_header());
   if (!chrome_proxy_header_value.empty())
     chrome_proxy_header_value += ", ";
-  chrome_proxy_header_value +=
-      chrome_proxy_lite_page_ignore_blacklist_directive();
+  chrome_proxy_header_value += chrome_proxy_force_lite_page_directive();
   headers->SetHeader(chrome_proxy_header(), chrome_proxy_header_value);
 }