SdchManager: remove EnableSdchSupport

net/base/sdch_manager.cc

Index: net/base/sdch_manager.cc
diff --git a/net/base/sdch_manager.cc b/net/base/sdch_manager.cc
index e4104e0a96dadaced546d9e0880c3716df388837..447dee3179fa29f74085065499d694b8e81e628e 100644
--- a/net/base/sdch_manager.cc
+++ b/net/base/sdch_manager.cc
@@ -38,9 +38,6 @@ void StripTrailingDot(GURL* gurl) {
 
 namespace net {
 
-// static
-bool SdchManager::g_sdch_enabled_ = true;
-
 SdchManager::DictionarySet::DictionarySet() {}
 
 SdchManager::DictionarySet::~DictionarySet() {}
@@ -103,11 +100,6 @@ void SdchManager::SdchErrorRecovery(SdchProblemCode problem) {
                             SDCH_MAX_PROBLEM_CODE);
 }
 
-// static
-void SdchManager::EnableSdchSupport(bool enabled) {
-  g_sdch_enabled_ = enabled;
-}
-
 void SdchManager::BlacklistDomain(const GURL& url,
                                   SdchProblemCode blacklist_reason) {
   SetAllowLatencyExperiment(url, false);
@@ -167,9 +159,6 @@ int SdchManager::BlacklistDomainExponential(const std::string& domain) {
 
 SdchProblemCode SdchManager::IsInSupportedDomain(const GURL& url) {
   DCHECK(thread_checker_.CalledOnValidThread());
-  if (!g_sdch_enabled_ )
-    return SDCH_DISABLED;

[Randy Smith (Not in Mondays), 2015/08/17 15:39:43]

Should we obsolete this error code in the source enum?  (I.e. comment it out,
but reserve the space, since this almost certainly shows up in UMA, and edit
histograms.xml to match?)

[Elly Fong-Jones, 2015/08/17 17:26:50]

Done.

-
   if (blacklisted_domains_.empty())
     return SDCH_OK;
 
@@ -456,7 +445,7 @@ void SdchManager::UrlSafeBase64Encode(const std::string& input,
 scoped_ptr<base::Value> SdchManager::SdchInfoToValue() const {
   scoped_ptr<base::DictionaryValue> value(new base::DictionaryValue());
 
-  value->SetBoolean("sdch_enabled", sdch_enabled());
+  value->SetBoolean("sdch_enabled", true);
 
   scoped_ptr<base::ListValue> entry_list(new base::ListValue());
   for (const auto& entry: dictionaries_) {