diff options
| author | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-17 19:40:50 +0000 |
|---|---|---|
| committer | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-17 19:40:50 +0000 |
| commit | d55ad15d8c3e118d6f03dbf236d6b98c62d66fe9 (patch) | |
| tree | cecfa125cafc7e2d86bf5a5fb1d6f75d67753f4d /net/base/sdch_manager.cc | |
| parent | e8aa85cde74fd420944f97135361f81a958fe87c (diff) | |
| download | chromium_src-d55ad15d8c3e118d6f03dbf236d6b98c62d66fe9.zip chromium_src-d55ad15d8c3e118d6f03dbf236d6b98c62d66fe9.tar.gz chromium_src-d55ad15d8c3e118d6f03dbf236d6b98c62d66fe9.tar.bz2 | |
Improve modularity of recent SDCH dictionary load checkin
I felt guilty that I allowed url_request_job to have some logic
in it about whether it can load an SDCH dictinary or not. This
patch pushed the logic back into sdch_manager (where it belongs).
r=wtc
Review URL: http://codereview.chromium.org/20379
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9897 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base/sdch_manager.cc')
| -rw-r--r-- | net/base/sdch_manager.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/base/sdch_manager.cc b/net/base/sdch_manager.cc index 9097f88..1f1ff0a 100644 --- a/net/base/sdch_manager.cc +++ b/net/base/sdch_manager.cc @@ -172,8 +172,10 @@ bool SdchManager::CanFetchDictionary(const GURL& referring_url, return true; } -void SdchManager::FetchDictionary(const GURL& dictionary_url) { - if (fetcher_.get()) +void SdchManager::FetchDictionary(const GURL& request_url, + const GURL& dictionary_url) { + if (SdchManager::Global()->CanFetchDictionary(request_url, dictionary_url) && + fetcher_.get()) fetcher_->Schedule(dictionary_url); } |