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

Unified Diff: extensions/browser/api/runtime/runtime_api.h

Issue 2893693002: Remove NOTIFICATION_EXTENSION_ENABLED. (Closed)
Patch Set: address comments Created 3 years, 6 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/BUILD.gn ('k') | extensions/browser/api/runtime/runtime_api.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/api/runtime/runtime_api.h
diff --git a/extensions/browser/api/runtime/runtime_api.h b/extensions/browser/api/runtime/runtime_api.h
index 3797908f8258df0a9da9b1bd2e0fbe2b5ebe0449..4246125439f02f6574a53fa68ae7a35c237259a5 100644
--- a/extensions/browser/api/runtime/runtime_api.h
+++ b/extensions/browser/api/runtime/runtime_api.h
@@ -13,6 +13,7 @@
#include "base/timer/timer.h"
#include "extensions/browser/api/runtime/runtime_api_delegate.h"
#include "extensions/browser/browser_context_keyed_api_factory.h"
+#include "extensions/browser/events/lazy_event_dispatch_util.h"
#include "extensions/browser/extension_function.h"
#include "extensions/browser/extension_registry_observer.h"
#include "extensions/browser/process_manager.h"
@@ -48,7 +49,8 @@ class ExtensionRegistry;
class RuntimeAPI : public BrowserContextKeyedAPI,
public ExtensionRegistryObserver,
public UpdateObserver,
- public ProcessManagerObserver {
+ public ProcessManagerObserver,
+ public LazyEventDispatchUtil::Observer {
public:
// The status of the restartAfterDelay request.
enum class RestartAfterDelayStatus {
@@ -96,14 +98,16 @@ class RuntimeAPI : public BrowserContextKeyedAPI,
// ExtensionRegistryObserver implementation.
void OnExtensionLoaded(content::BrowserContext* browser_context,
const Extension* extension) override;
- void OnExtensionWillBeInstalled(content::BrowserContext* browser_context,
- const Extension* extension,
- bool is_update,
- const std::string& old_name) override;
void OnExtensionUninstalled(content::BrowserContext* browser_context,
const Extension* extension,
UninstallReason reason) override;
+ // LazyEventDispatchUtil::Observer:
+ void OnExtensionInstalledAndLoaded(
+ content::BrowserContext* browser_context,
+ const Extension* extension,
+ const base::Version& previous_version) override;
+
// Cancels any previously scheduled restart request.
void MaybeCancelRunningDelayedRestartTimer();
@@ -130,14 +134,6 @@ class RuntimeAPI : public BrowserContextKeyedAPI,
// ProcessManagerObserver implementation:
void OnBackgroundHostStartup(const Extension* extension) override;
- // Pref related functions that deals with info about installed extensions that
- // has not been loaded yet.
- // Used to send chrome.runtime.onInstalled event upon loading the extensions.
- bool ReadPendingOnInstallInfoFromPref(const ExtensionId& extension_id,
- base::Version* previous_version);
- void RemovePendingOnInstallInfoFromPref(const ExtensionId& extension_id);
- void StorePendingOnInstallInfoToPref(const Extension* extension);
-
void AllowNonKioskAppsInRestartAfterDelayForTesting();
void set_min_duration_between_restarts_for_testing(base::TimeDelta delta) {
« no previous file with comments | « extensions/browser/BUILD.gn ('k') | extensions/browser/api/runtime/runtime_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698