Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6006)

Unified Diff: chrome/browser/previews/previews_service.cc

Issue 2760063002: Add support to previews/ for Server LoFi and LitePages (Closed)
Patch Set: tbansal comment Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/previews/previews_service.cc
diff --git a/chrome/browser/previews/previews_service.cc b/chrome/browser/previews/previews_service.cc
index 8ddcfd18ca285b87f1c1733f6bb92f6b5b9896d2..1b82bc89ca2b3a1f8dae4d29b3dce74ae543f6a8 100644
--- a/chrome/browser/previews/previews_service.cc
+++ b/chrome/browser/previews/previews_service.cc
@@ -10,6 +10,7 @@
#include "base/sequenced_task_runner.h"
#include "base/threading/sequenced_worker_pool.h"
#include "chrome/common/chrome_constants.h"
+#include "components/data_reduction_proxy/core/common/data_reduction_proxy_params.h"
#include "components/previews/core/previews_experiments.h"
#include "components/previews/core/previews_io_data.h"
#include "components/previews/core/previews_opt_out_store.h"
@@ -26,6 +27,13 @@ bool IsPreviewsTypeEnabled(previews::PreviewsType type) {
return previews::params::IsOfflinePreviewsEnabled();
case previews::PreviewsType::CLIENT_LOFI:
return previews::params::IsClientLoFiEnabled();
+ case previews::PreviewsType::SERVER_LOFI:
+ return data_reduction_proxy::params::IsLoFiOnViaFlags() ||
+ data_reduction_proxy::params::IsIncludedInLoFiEnabledFieldTrial();
+ case previews::PreviewsType::LITE_PAGE:
+ return (data_reduction_proxy::params::IsLoFiOnViaFlags() &&
+ data_reduction_proxy::params::AreLitePagesEnabledViaFlags()) ||
+ data_reduction_proxy::params::IsIncludedInLitePageFieldTrial();
case previews::PreviewsType::NONE:
case previews::PreviewsType::LAST:
break;
@@ -42,6 +50,10 @@ int GetPreviewsTypeVersion(previews::PreviewsType type) {
return previews::params::OfflinePreviewsVersion();
case previews::PreviewsType::CLIENT_LOFI:
return previews::params::ClientLoFiVersion();
+ case previews::PreviewsType::LITE_PAGE:
+ return data_reduction_proxy::params::LitePageVersion();
+ case previews::PreviewsType::SERVER_LOFI:
+ return data_reduction_proxy::params::ServerLoFiVersion();
case previews::PreviewsType::NONE:
case previews::PreviewsType::LAST:
break;
« no previous file with comments | « no previous file | chrome/browser/previews/previews_service_unittest.cc » ('j') | tools/metrics/histograms/histograms.xml » ('J')

Powered by Google App Engine
This is Rietveld 408576698