Add support to previews/ for Server LoFi and LitePages

components/previews/core/previews_decider.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_PREVIEWS_CORE_PREVIEWS_DECIDER_H_
 #define COMPONENTS_PREVIEWS_CORE_PREVIEWS_DECIDER_H_
 
 #include "components/previews/core/previews_experiments.h"
 
+#include "net/nqe/effective_connection_type.h"
+
 namespace net {
 class URLRequest;
 }
 
 namespace previews {
 
 class PreviewsDecider {
  public:
-  // Whether |request| is allowed to show a preview of |type|.
+  // Whether |request| is allowed to show a preview of |type|. Previews that do
+  // not supply a network quality in
+  // GetEffectiveConnectionTypeThresholdForPreviewsType() should check the
+  // network quality before calling ShouldAllowPreview. |request| is the main

[tbansal1, 2017/05/05 21:31:31]

s/ShouldAllowPreview/ShouldAllowPreviewAtECT/

[RyanSturm, 2017/05/05 21:39:02]

Done.

+  // frame request of the page load. |type| is the type that would be allowed.
+  // If the current ECT better than |effective_connection_type_threshold|, the

[tbansal1, 2017/05/05 21:31:31]

s/better/strictly faster/
OR s/better/at least as fast as/

[RyanSturm, 2017/05/05 21:39:02]

Done.

+  // preview will be disallowed.
+  virtual bool ShouldAllowPreviewAtECT(
+      const net::URLRequest& request,
+      PreviewsType type,
+      net::EffectiveConnectionType effective_connection_type_threshold)
+      const = 0;
+
+  // Same as ShouldAllowPreviewAtECT, but uses the previews default
+  // EffectiveConnectionType.
   virtual bool ShouldAllowPreview(const net::URLRequest& request,
                                   PreviewsType type) const = 0;
 
  protected:
   PreviewsDecider() {}
   virtual ~PreviewsDecider() {}
 };
 
 }  // namespace previews
 
 #endif  // COMPONENTS_PREVIEWS_CORE_PREVIEWS_DECIDER_H_