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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc

Issue 2922663002: Data Reduction Proxy: Remove duplicate functions (Closed)
Patch Set: megjablon comments Created 3 years, 6 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: components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc
index b07e7dd8c8bbdafba501863d38e0ea58852e0d19..0c671cabd760f2f708949ef7ea9a1a164b398031 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc
@@ -54,7 +54,6 @@ const char kDataReductionPassThroughHeader[] =
DataReductionProxySettings::DataReductionProxySettings()
: unreachable_(false),
deferred_initialization_(false),
- promo_allowed_(false),
data_reduction_proxy_enabled_pref_name_(),
prefs_(NULL),
config_(nullptr),
@@ -79,11 +78,6 @@ void DataReductionProxySettings::InitPrefMembers() {
base::Unretained(this)));
}
-void DataReductionProxySettings::UpdateConfigValues() {
- DCHECK(config_);
- promo_allowed_ = config_->promo_allowed();
-}
-
void DataReductionProxySettings::InitDataReductionProxySettings(
const std::string& data_reduction_proxy_enabled_pref_name,
PrefService* prefs,
@@ -102,7 +96,6 @@ void DataReductionProxySettings::InitDataReductionProxySettings(
data_reduction_proxy_service_ = std::move(data_reduction_proxy_service);
data_reduction_proxy_service_->AddObserver(this);
InitPrefMembers();
- UpdateConfigValues();
RecordDataReductionInit();
data_reduction_proxy_service_->InitializeLoFiPrefs();

Powered by Google App Engine
This is Rietveld 408576698