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

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

Issue 545068: Allow extensions to add, remove, or change their page action popup. (Closed)
Patch Set: Rebase, to run trybots with test data in place. Created 10 years, 11 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_page_actions_module.h
diff --git a/chrome/browser/extensions/extension_page_actions_module.h b/chrome/browser/extensions/extension_page_actions_module.h
index 8c0e2913a15c1c0792e22811faf945ef01cb78ad..3be9ce1f1dafe974eab32045d7ed1a68b1904d7f 100644
--- a/chrome/browser/extensions/extension_page_actions_module.h
+++ b/chrome/browser/extensions/extension_page_actions_module.h
@@ -10,6 +10,7 @@
class TabContents;
class ExtensionAction;
+// Base class for page action APIs.
class PageActionFunction : public SyncExtensionFunction {
protected:
virtual ~PageActionFunction() {}
@@ -22,54 +23,70 @@ class PageActionFunction : public SyncExtensionFunction {
TabContents* contents_;
};
+// Implement chrome.pageActions.enableForTab().
class EnablePageActionFunction : public PageActionFunction {
~EnablePageActionFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageActions.enableForTab")
};
+// Implement chrome.pageActions.disableForTab().
class DisablePageActionFunction : public PageActionFunction {
~DisablePageActionFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageActions.disableForTab")
};
+// Implement chrome.pageActions.show().
class PageActionShowFunction : public PageActionFunction {
~PageActionShowFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageAction.show")
};
+// Implement chrome.pageActions.hide().
class PageActionHideFunction : public PageActionFunction {
~PageActionHideFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageAction.hide")
};
+// Implement chrome.pageActions.setIcon().
class PageActionSetIconFunction : public PageActionFunction {
~PageActionSetIconFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageAction.setIcon")
};
+// Implement chrome.pageActions.setTitle().
class PageActionSetTitleFunction : public PageActionFunction {
~PageActionSetTitleFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageAction.setTitle")
};
+// Implement chrome.pageActions.setPopup().
+class PageActionSetPopupFunction : public PageActionFunction {
+ ~PageActionSetPopupFunction() {}
+ virtual bool RunImpl();
+ DECLARE_EXTENSION_FUNCTION_NAME("pageAction.setPopup")
+};
+
+// Implement chrome.pageActions.setBadgeBackgroundColor().
class PageActionSetBadgeBackgroundColorFunction : public PageActionFunction {
~PageActionSetBadgeBackgroundColorFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageAction.setBadgeBackgroundColor")
};
+// Implement chrome.pageActions.setBadgeTextColor().
class PageActionSetBadgeTextColorFunction : public PageActionFunction {
~PageActionSetBadgeTextColorFunction() {}
virtual bool RunImpl();
DECLARE_EXTENSION_FUNCTION_NAME("pageAction.setBadgeTextColor")
};
+// Implement chrome.pageActions.setBadgeText().
class PageActionSetBadgeTextFunction : public PageActionFunction {
~PageActionSetBadgeTextFunction() {}
virtual bool RunImpl();
« no previous file with comments | « chrome/browser/extensions/extension_function_dispatcher.cc ('k') | chrome/browser/extensions/extension_page_actions_module.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698