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/profile.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/profile.cc')
| -rw-r--r-- | chrome/browser/profile.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/profile.cc b/chrome/browser/profile.cc index 9f62d39..112375a 100644 --- a/chrome/browser/profile.cc +++ b/chrome/browser/profile.cc @@ -1321,7 +1321,7 @@ void ProfileImpl::ReinitializeSpellCheckHost(bool force) { bool notify = false; if (spellcheck_host_.get()) { spellcheck_host_->UnsetObserver(); - spellcheck_host_.release(); + spellcheck_host_ = NULL; spellcheck_host_ready_ = false; notify = true; } |