Remove refptr usages from SpellCheckHost

chrome/browser/spellchecker/spellcheck_host.cc

Index: chrome/browser/spellchecker/spellcheck_host.cc
diff --git a/chrome/browser/spellchecker/spellcheck_host.cc b/chrome/browser/spellchecker/spellcheck_host.cc
index bfdad0fad4422b538eb1b3b35e15c681ebed4510..32a1d7bda3cce399a8dc803a47b9e6eeacc7af87 100644
--- a/chrome/browser/spellchecker/spellcheck_host.cc
+++ b/chrome/browser/spellchecker/spellcheck_host.cc
@@ -4,6 +4,7 @@
 
 #include "chrome/browser/spellchecker/spellcheck_host.h"
 
+#include "base/memory/scoped_ptr.h"

[dcheng, 2011/10/20 23:24:34]

I'm surprised you need to add this as we don't ever used scoped_ptr directly in
this file. Are we missing an include in another header file?

 #include "base/string_split.h"
 #include "base/synchronization/waitable_event.h"
 #include "chrome/browser/prefs/pref_member.h"
@@ -23,21 +24,19 @@ SpellCheckHost::EventType g_status_type = SpellCheckHost::BDICT_NOTINITIALIZED;
 }  // namespace
 
 // static
-scoped_refptr<SpellCheckHost> SpellCheckHost::Create(
+SpellCheckHost* SpellCheckHost::Create(
     SpellCheckProfileProvider* profile,
     const std::string& language,
     net::URLRequestContextGetter* request_context_getter,
     SpellCheckHostMetrics* metrics) {
-  scoped_refptr<SpellCheckHostImpl> host =
-      new SpellCheckHostImpl(profile,
-                             language,
-                             request_context_getter,
+  SpellCheckHostImpl* host =
+      new SpellCheckHostImpl(profile, language, request_context_getter,
                              metrics);
   if (!host)
     return NULL;
 
   host->Initialize();
-  return host;
+  return (SpellCheckHost*)host;

[dcheng, 2011/10/20 23:24:34]

You shouldn't need this cast.

 }
 
 // static