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

Unified Diff: extensions/common/extension.h

Issue 2839373003: struct UnloadedExtensionInfo -> enum UnloadedExtensionInfoReason (Closed)
Patch Set: Rebase Created 3 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
« no previous file with comments | « extensions/browser/warning_service.cc ('k') | extensions/common/extension.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/common/extension.h
diff --git a/extensions/common/extension.h b/extensions/common/extension.h
index 704b7dde823184203b50df09d4738ab889d00842..89e9a26b39c035b17527e03730ff585f5151d525 100644
--- a/extensions/common/extension.h
+++ b/extensions/common/extension.h
@@ -519,27 +519,18 @@ struct InstalledExtensionInfo {
const std::string& old_name);
};
-struct UnloadedExtensionInfo {
- // TODO(DHNishi): Move this enum to ExtensionRegistryObserver.
- enum Reason {
- REASON_UNDEFINED, // Undefined state used to initialize variables.
- REASON_DISABLE, // Extension is being disabled.
- REASON_UPDATE, // Extension is being updated to a newer version.
- REASON_UNINSTALL, // Extension is being uninstalled.
- REASON_TERMINATE, // Extension has terminated.
- REASON_BLACKLIST, // Extension has been blacklisted.
- REASON_PROFILE_SHUTDOWN, // Profile is being shut down.
- REASON_LOCK_ALL, // All extensions for the profile are blocked.
- REASON_MIGRATED_TO_COMPONENT, // Extension is being migrated to a component
- // action.
- };
-
- Reason reason;
-
- // The extension being unloaded - this should always be non-NULL.
- const Extension* extension;
-
- UnloadedExtensionInfo(const Extension* extension, Reason reason);
+// TODO(DHNishi): Move this enum to ExtensionRegistryObserver.
+enum class UnloadedExtensionReason {
+ UNDEFINED, // Undefined state used to initialize variables.
+ DISABLE, // Extension is being disabled.
+ UPDATE, // Extension is being updated to a newer version.
+ UNINSTALL, // Extension is being uninstalled.
+ TERMINATE, // Extension has terminated.
+ BLACKLIST, // Extension has been blacklisted.
+ PROFILE_SHUTDOWN, // Profile is being shut down.
+ LOCK_ALL, // All extensions for the profile are blocked.
+ MIGRATED_TO_COMPONENT, // Extension is being migrated to a component
+ // action.
};
// The details sent for EXTENSION_PERMISSIONS_UPDATED notifications.
« no previous file with comments | « extensions/browser/warning_service.cc ('k') | extensions/common/extension.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698