Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6973)

Unified Diff: chrome/browser/spellchecker/spellcheck_host.h

Issue 7919003: Remove refptr usages from SpellCheckHost (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: remove TODO comments Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/spellchecker/spellcheck_host.h
diff --git a/chrome/browser/spellchecker/spellcheck_host.h b/chrome/browser/spellchecker/spellcheck_host.h
index 04272acd79834e6ef3ade97af0149ac91fec28a0..a439aa345d8931912d6a9e5e6a308449f46c8924 100644
--- a/chrome/browser/spellchecker/spellcheck_host.h
+++ b/chrome/browser/spellchecker/spellcheck_host.h
@@ -48,9 +48,7 @@ class URLRequestContextGetter;
// 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> {
Hironori Bono 2011/11/04 03:46:59 nit: 'BrowserThread' has been renamed to 'content:
+class SpellCheckHost {
public:
// Event types used for reporting the status of this class and its derived
// classes to browser tests.
@@ -62,7 +60,7 @@ class SpellCheckHost
virtual ~SpellCheckHost() {}
// Creates the instance of SpellCheckHost implementation object.
- static scoped_refptr<SpellCheckHost> Create(
+ static SpellCheckHost* Create(
SpellCheckProfileProvider* profile,
const std::string& language,
net::URLRequestContextGetter* request_context_getter,
« no previous file with comments | « no previous file | chrome/browser/spellchecker/spellcheck_host.cc » ('j') | chrome/browser/spellchecker/spellcheck_host_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698