diff options
| author | jar@google.com <jar@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-02 01:32:34 +0000 |
|---|---|---|
| committer | jar@google.com <jar@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-02 01:32:34 +0000 |
| commit | 140a1bc66cbf515dfb10347ee10b39d42dbe1ee0 (patch) | |
| tree | adfc4252be520d1db852d8f429cc3361d078554c /net/base/sdch_manager.cc | |
| parent | c949118827afb549daa94fe3171b5e415bb32ea9 (diff) | |
| download | chromium_src-140a1bc66cbf515dfb10347ee10b39d42dbe1ee0.zip chromium_src-140a1bc66cbf515dfb10347ee10b39d42dbe1ee0.tar.gz chromium_src-140a1bc66cbf515dfb10347ee10b39d42dbe1ee0.tar.bz2 | |
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
Diffstat (limited to 'net/base/sdch_manager.cc')
| -rw-r--r-- | net/base/sdch_manager.cc | 10 |
1 files changed, 7 insertions, 3 deletions
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, |