diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-10 01:33:05 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-10 01:33:05 +0000 |
| commit | 71c012ae753d7f428e83cd4866a5516fc8c58b82 (patch) | |
| tree | 1015457c7339c21d8ff9c87807d34ba8c24d1a39 /chrome/browser/spellcheck_host.cc | |
| parent | 1fa528658d43f7e3fa7651a423430d11dc90d4bc (diff) | |
| download | chromium_src-71c012ae753d7f428e83cd4866a5516fc8c58b82.zip chromium_src-71c012ae753d7f428e83cd4866a5516fc8c58b82.tar.gz chromium_src-71c012ae753d7f428e83cd4866a5516fc8c58b82.tar.bz2 | |
Couple of prospective fix for memory test flakiness: addref/release the url request context getter appropriately; fix a leak.
Passing as a straight pointer without addreffing was copied from the previous spellchecker impl. Using .release() instead of = NULL was a plain old mistake.
BUG=none
TEST=memory test flakiness goes away hopefully?
Review URL: http://codereview.chromium.org/379015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31529 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/spellcheck_host.cc')
| -rw-r--r-- | chrome/browser/spellcheck_host.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/spellcheck_host.cc b/chrome/browser/spellcheck_host.cc index 91a1a9c..c9fea3ecf 100644 --- a/chrome/browser/spellcheck_host.cc +++ b/chrome/browser/spellcheck_host.cc @@ -220,7 +220,7 @@ void SpellCheckHost::DownloadDictionary() { GURL url = GURL(std::string(kDownloadServerUrl) + WideToUTF8( l10n_util::ToLower(bdict_file_.BaseName().ToWStringHack()))); fetcher_.reset(new URLFetcher(url, URLFetcher::GET, this)); - fetcher_->set_request_context(request_context_getter_); + fetcher_->set_request_context(request_context_getter_.get()); tried_to_download_ = true; fetcher_->Start(); } |