From 140a1bc66cbf515dfb10347ee10b39d42dbe1ee0 Mon Sep 17 00:00:00 2001 From: "jar@google.com" Date: Tue, 2 Dec 2008 01:32:34 +0000 Subject: Add instrumentation to and tighten SDCH implementation r=huanr,kmixter,openvcdiff Review URL: http://codereview.chromium.org/12699 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@6193 0039d316-1c4b-4281-b951-d872f2087c98 --- net/base/sdch_manager.cc | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'net/base/sdch_manager.cc') diff --git a/net/base/sdch_manager.cc b/net/base/sdch_manager.cc index 6c5bf92..8b71a04 100644 --- a/net/base/sdch_manager.cc +++ b/net/base/sdch_manager.cc @@ -64,10 +64,10 @@ SdchManager::~SdchManager() { bool SdchManager::BlacklistDomain(const GURL& url) { if (!global_ ) return false; - std::string domain(url.host()); UMA_HISTOGRAM_MEDIUM_TIMES(L"Sdch.UptimeBeforeBlacklisting_M", Time::Now() - FieldTrialList::application_start_time()); - global_->blacklisted_domains_.insert(url.host()); + std::string domain(StringToLowerASCII(url.host())); + global_->blacklisted_domains_.insert(domain); return true; } @@ -88,7 +88,11 @@ const bool SdchManager::IsInSupportedDomain(const GURL& url) const { return true; std::string domain = StringToLowerASCII(url.host()); - return blacklisted_domains_.end() == blacklisted_domains_.find(domain); + bool was_blacklisted(blacklisted_domains_.end() == + blacklisted_domains_.find(domain)); + if (was_blacklisted) + SdchErrorRecovery(DOMAIN_BLACKLIST_INCLUDES_TARGET); + return was_blacklisted; } bool SdchManager::CanFetchDictionary(const GURL& referring_url, -- cgit v1.1