commit 71c012ae753d7f428e83cd4866a5516fc8c58b82
author estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Tue Nov 10 01:33:05 2009
committer estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Tue Nov 10 01:33:05 2009
tree 1015457c7339c21d8ff9c87807d34ba8c24d1a39
parent 1fa528658d43f7e3fa7651a423430d11dc90d4bc

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

chrome/browser/profile.cc

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 @@
   bool notify = false;
   if (spellcheck_host_.get()) {
     spellcheck_host_->UnsetObserver();
-    spellcheck_host_.release();
+    spellcheck_host_ = NULL;
     spellcheck_host_ready_ = false;
     notify = true;
   }