SpellCheck refactoring: Moved user custom dictionary for spell check to SpellCheckProfile.

chrome/browser/profiles/profile_impl.cc

Index: chrome/browser/profiles/profile_impl.cc
diff --git a/chrome/browser/profiles/profile_impl.cc b/chrome/browser/profiles/profile_impl.cc
index 44c7774eaa464774db66c369108e4661d2fe09cf..a4fb56f84eafa3ecd94b497e18edaa80ad74945e 100644
--- a/chrome/browser/profiles/profile_impl.cc
+++ b/chrome/browser/profiles/profile_impl.cc
@@ -1759,7 +1759,7 @@ void ProfileImpl::ClearNetworkingHistorySince(base::Time time) {
 
 SpellCheckProfile* ProfileImpl::GetSpellCheckProfile() {
   if (!spellcheck_profile_.get())
-    spellcheck_profile_.reset(new SpellCheckProfile());
+    spellcheck_profile_ = new SpellCheckProfile();

[Miranda Callahan, 2011/10/12 14:33:05]

scoped_ptr still uses the .reset function.

[shinyak (Google), 2011/10/13 07:51:22]

I've changed SpellCheckProfile ref-counted object (in order to keep it alive
until receiving save task or something like that). So I made spellcheck_profile_
scoped_refptr instead of scoped_ptr. scoped_refptr does not have reset() method.

[Hironori Bono, 2011/10/13 09:29:59]

Is it possible to describe more details about why you need scoped_refptr<>? (As
written in Issue 79886, we are trying to remove scoped_refptr<> from
ProfileImpl.)

[Miranda Callahan, 2011/10/13 14:33:12]

My apologies, you're right, of course -- I don't know what happened to my brain
-- I looked at the .h file several times, too. :-/

Having said that -- I agree with hbono about trying to remove scoped_refptr from
Profile -- it would be great if it were possible to avoid this change!

   return spellcheck_profile_.get();
 }