Remove refptr usages from SpellCheckHost

chrome/browser/spellchecker/spellcheck_host.h

 // 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.
 
 #ifndef CHROME_BROWSER_SPELLCHECKER_SPELLCHECK_HOST_H_
 #define CHROME_BROWSER_SPELLCHECKER_SPELLCHECK_HOST_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
 #include "base/gtest_prod_util.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/memory/ref_counted.h"

[Hironori Bono, 2011/10/14 10:57:04]

nit: we do not need this line any longer once we remove scoped_refptr<> from
this class. :)

 #include "base/platform_file.h"
 #include "content/browser/browser_thread.h"
 
 namespace base {
 class WaitableEvent;
 }
 
 class Profile;
 class RenderProcessHost;
 class SpellCheckHostMetrics;
@@ skipping 17 matching lines @@
 // The following snippet describes how to use this class,
 //   std::vector<std::string> languages;
 //   SpellCheckHost::GetSpellCheckLanguages(profile_, &languages);
 //   spellcheck_host_ = SpellCheckHost::Create(
 //       observer, languages[0], req_getter);
 //
 // The class is intended to be owned by SpellCheckProfile so users should
 // retrieve the instance via Profile::GetSpellCheckHost().
 // Users should not hold the reference over the function scope because
 // the instance can be invalidated during the browser's lifecycle.
-class SpellCheckHost
-    : public base::RefCountedThreadSafe<SpellCheckHost,
-                                        BrowserThread::DeleteOnFileThread> {
+class SpellCheckHost {
  public:
   // Event types used for reporting the status of this class and its derived
   // classes to browser tests.
   enum EventType {
     BDICT_NOTINITIALIZED,
     BDICT_CORRUPTED,
   };
 
   virtual ~SpellCheckHost() {}
 
   // Creates the instance of SpellCheckHost implementation object.
-  static scoped_refptr<SpellCheckHost> Create(
+  static scoped_ptr<SpellCheckHost> Create(

[Hironori Bono, 2011/10/14 10:57:04]

We can change this return type to SpellCheckHost* when we use scoped_ptr<>.

       SpellCheckProfileProvider* profile,
       const std::string& language,
       net::URLRequestContextGetter* request_context_getter,
       SpellCheckHostMetrics* metrics);
 
   // Clears a profile which is set on creation.
   // Used to prevent calling back to a deleted object.
   virtual void UnsetProfile() = 0;
 
   // Pass the renderer some basic intialization information. Note that the
@@ skipping 37 matching lines @@
 
   // Attaches an event so browser tests can listen the status events.
   static void AttachStatusEvent(base::WaitableEvent* status_event);
 
   // Waits until a spellchecker updates its status. This function returns
   // immediately when we do not set an event to |status_event_|.
   static EventType WaitStatusEvent();
 };
 
 #endif  // CHROME_BROWSER_SPELLCHECKER_SPELLCHECK_HOST_H_