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

Unified Diff: chrome/browser/extensions/api/context_menus/context_menus_api.h

Issue 2328643003: [Extensions] Convert some ChromeSyncExtensionFunctions (Closed)
Patch Set: Remove GetProfile() usage Created 4 years, 3 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/api/context_menus/context_menus_api.h
diff --git a/chrome/browser/extensions/api/context_menus/context_menus_api.h b/chrome/browser/extensions/api/context_menus/context_menus_api.h
index 8c05e683a3cdb53316e6b2a1c51cae363a4859c1..78e00b7415cf51f3d13b6d783c04acc5e05c8a93 100644
--- a/chrome/browser/extensions/api/context_menus/context_menus_api.h
+++ b/chrome/browser/extensions/api/context_menus/context_menus_api.h
@@ -5,11 +5,11 @@
#ifndef CHROME_BROWSER_EXTENSIONS_API_CONTEXT_MENUS_CONTEXT_MENUS_API_H_
#define CHROME_BROWSER_EXTENSIONS_API_CONTEXT_MENUS_CONTEXT_MENUS_API_H_
-#include "chrome/browser/extensions/chrome_extension_function.h"
+#include "extensions/browser/extension_function.h"
namespace extensions {
-class ContextMenusCreateFunction : public ChromeSyncExtensionFunction {
+class ContextMenusCreateFunction : public UIThreadExtensionFunction {
public:
DECLARE_EXTENSION_FUNCTION("contextMenus.create", CONTEXTMENUS_CREATE)
@@ -17,10 +17,10 @@ class ContextMenusCreateFunction : public ChromeSyncExtensionFunction {
~ContextMenusCreateFunction() override {}
// ExtensionFunction:
- bool RunSync() override;
+ ResponseAction Run() override;
};
-class ContextMenusUpdateFunction : public ChromeSyncExtensionFunction {
+class ContextMenusUpdateFunction : public UIThreadExtensionFunction {
public:
DECLARE_EXTENSION_FUNCTION("contextMenus.update", CONTEXTMENUS_UPDATE)
@@ -28,10 +28,10 @@ class ContextMenusUpdateFunction : public ChromeSyncExtensionFunction {
~ContextMenusUpdateFunction() override {}
// ExtensionFunction:
- bool RunSync() override;
+ ResponseAction Run() override;
};
-class ContextMenusRemoveFunction : public ChromeSyncExtensionFunction {
+class ContextMenusRemoveFunction : public UIThreadExtensionFunction {
public:
DECLARE_EXTENSION_FUNCTION("contextMenus.remove", CONTEXTMENUS_REMOVE)
@@ -39,10 +39,10 @@ class ContextMenusRemoveFunction : public ChromeSyncExtensionFunction {
~ContextMenusRemoveFunction() override {}
// ExtensionFunction:
- bool RunSync() override;
+ ResponseAction Run() override;
};
-class ContextMenusRemoveAllFunction : public ChromeSyncExtensionFunction {
+class ContextMenusRemoveAllFunction : public UIThreadExtensionFunction {
public:
DECLARE_EXTENSION_FUNCTION("contextMenus.removeAll", CONTEXTMENUS_REMOVEALL)
@@ -50,7 +50,7 @@ class ContextMenusRemoveAllFunction : public ChromeSyncExtensionFunction {
~ContextMenusRemoveAllFunction() override {}
// ExtensionFunction:
- bool RunSync() override;
+ ResponseAction Run() override;
};
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698