Add an about:flag to support alternative data saver features

components/data_reduction_proxy/content/browser/content_lofi_decider.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/data_reduction_proxy/content/browser/content_lofi_decider.h"
 
 #include <string>
 
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
@@ skipping 162 matching lines @@
   std::string transform_type;
   base::TrimWhitespaceASCII(tokens[0], base::TRIM_ALL, &transform_type);
   return transform_type == lite_page_directive();
 }
 
 void ContentLoFiDecider::RemoveAcceptTransformHeader(
     net::HttpRequestHeaders* headers) const {
   headers->RemoveHeader(chrome_proxy_accept_transform_header());
 }
 
-void ContentLoFiDecider::MaybeSetIgnorePreviewsBlacklistDirective(

[tbansal1, 2017/05/19 16:27:23]

Thanks for the cleanup. The new logic is much better.

[RyanSturm, 2017/05/23 22:04:57]

Acknowledged.

-    net::HttpRequestHeaders* headers) const {
-  if (!headers || !params::AreLitePagesEnabledViaFlags() ||
-      !IsLitePagePreviewRequested(*headers)) {
-    return;
-  }
-  std::string chrome_proxy_header_value;
-  headers->GetHeader(chrome_proxy_header(), &chrome_proxy_header_value);
-  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();
-  headers->SetHeader(chrome_proxy_header(), chrome_proxy_header_value);
-}
-
 bool ContentLoFiDecider::ShouldRecordLoFiUMA(
     const net::URLRequest& request) const {
   const content::ResourceRequestInfo* request_info =
       content::ResourceRequestInfo::ForRequest(&request);
 
   // User is not using Lo-Fi.
   if (!request_info ||
       !(request_info->GetPreviewsState() & content::SERVER_LOFI_ON ||
         request_info->GetPreviewsState() & content::SERVER_LITE_PAGE_ON)) {
     return false;
@@ skipping 13 matching lines @@
 }
 
 bool ContentLoFiDecider::IsClientLoFiAutoReloadRequest(
     const net::URLRequest& request) const {
   const content::ResourceRequestInfo* request_info =
       content::ResourceRequestInfo::ForRequest(&request);
   return request_info &&
          (request_info->GetPreviewsState() & content::CLIENT_LOFI_AUTO_RELOAD);
 }
 }  // namespace data_reduction_proxy