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

Unified Diff: chrome/browser/extensions/extension_management_api.h

Issue 7064033: Virtual destructors should have virtual keyword. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 7 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/extension_management_api.h
diff --git a/chrome/browser/extensions/extension_management_api.h b/chrome/browser/extensions/extension_management_api.h
index f7a75e599351c368f80c9ef9cd415d6b66773980..04b2c8cad3115639f750d48d861eec1de059fd30 100644
--- a/chrome/browser/extensions/extension_management_api.h
+++ b/chrome/browser/extensions/extension_management_api.h
@@ -19,31 +19,31 @@ class ExtensionManagementFunction : public SyncExtensionFunction {
};
class GetAllExtensionsFunction : public ExtensionManagementFunction {
- ~GetAllExtensionsFunction() {}
+ virtual ~GetAllExtensionsFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("management.getAll");
};
class GetExtensionByIdFunction : public ExtensionManagementFunction {
- ~GetExtensionByIdFunction() {}
+ virtual ~GetExtensionByIdFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("management.get");
};
class LaunchAppFunction : public ExtensionManagementFunction {
- ~LaunchAppFunction() {}
+ virtual ~LaunchAppFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("management.launchApp");
};
class SetEnabledFunction : public ExtensionManagementFunction {
- ~SetEnabledFunction() {}
+ virtual ~SetEnabledFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("management.setEnabled");
};
class UninstallFunction : public ExtensionManagementFunction {
- ~UninstallFunction() {}
+ virtual ~UninstallFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("management.uninstall");
};
« no previous file with comments | « chrome/browser/extensions/extension_infobar_module.h ('k') | chrome/browser/extensions/extension_page_actions_module.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698