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

Unified Diff: chrome/browser/extensions/updater/safe_manifest_parser.cc

Issue 10071035: RefCounted types should not have public destructors, chrome/browser/extensions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Compile fix Created 8 years, 8 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/extensions/updater/safe_manifest_parser.cc
diff --git a/chrome/browser/extensions/updater/safe_manifest_parser.cc b/chrome/browser/extensions/updater/safe_manifest_parser.cc
index c633dafacdc0250c10c684f6a7d057664b4b7074..72871227b34f72b5c8d05f8230fb2cf4099b408f 100644
--- a/chrome/browser/extensions/updater/safe_manifest_parser.cc
+++ b/chrome/browser/extensions/updater/safe_manifest_parser.cc
@@ -28,11 +28,6 @@ SafeManifestParser::SafeManifestParser(const std::string& xml,
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
}
-SafeManifestParser::~SafeManifestParser() {
- // If we're using UtilityProcessHost, we may not be destroyed on
- // the UI or IO thread.
-}
-
void SafeManifestParser::Start() {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
if (!BrowserThread::PostTask(
@@ -42,6 +37,11 @@ void SafeManifestParser::Start() {
}
}
+SafeManifestParser::~SafeManifestParser() {
+ // If we're using UtilityProcessHost, we may not be destroyed on
+ // the UI or IO thread.
+}
+
void SafeManifestParser::ParseInSandbox() {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
« no previous file with comments | « chrome/browser/extensions/updater/safe_manifest_parser.h ('k') | chrome/browser/extensions/webstore_inline_installer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698