Remove refptr usages from SpellCheckHost

chrome/browser/spellchecker/spellcheck_host.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #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"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/spellchecker/spellcheck_host_impl.h"
 #include "chrome/browser/spellchecker/spellchecker_platform_engine.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/spellcheck_common.h"
 
 namespace {
 
 // An event used by browser tests to receive status events from this class and
 // its derived classes.
 base::WaitableEvent* g_status_event = NULL;
 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
 int SpellCheckHost::GetSpellCheckLanguages(
     Profile* profile,
     std::vector<std::string>* languages) {
   StringPrefMember accept_languages_pref;
   StringPrefMember dictionary_language_pref;
   accept_languages_pref.Init(prefs::kAcceptLanguages, profile->GetPrefs(),
                              NULL);
@@ skipping 50 matching lines @@
 }
 
 // static
 SpellCheckHost::EventType SpellCheckHost::WaitStatusEvent() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   if (g_status_event)
     g_status_event->Wait();
   return g_status_type;
 }