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

Unified Diff: chrome/common/extensions/extension_switch_utils.h

Issue 10565017: Parse the script_badge manifest section. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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
Index: chrome/common/extensions/extension_switch_utils.h
diff --git a/chrome/common/extensions/extension_switch_utils.h b/chrome/common/extensions/extension_switch_utils.h
index e2c53ca167eea5adcfe5373e900a85c79a198003..4965dc0111aafdd437eaf16aad0ce16e43502b6f 100644
--- a/chrome/common/extensions/extension_switch_utils.h
+++ b/chrome/common/extensions/extension_switch_utils.h
@@ -14,6 +14,20 @@ bool IsEasyOffStoreInstallEnabled();
bool IsActionBoxEnabled();
+// TODO(jyasskin): Convert this to a generic switch system that wraps the whole
+// interaction with CommandLine::HasSwitch and doesn't require globals.
not at google - send to devlin 2012/06/18 22:59:41 yeah, seems like this functionality would be nice
+enum EnabledState {
+ ENABLED,
+ DISABLED
+};
+
+// Using this makes all other access to IsActionBoxEnabled NOT thread-safe.
+class ScopedSetActionBoxForTest {
+ public:
+ explicit ScopedSetActionBoxForTest(EnabledState);
+ ~ScopedSetActionBoxForTest();
+};
+
} // switch_utils
} // extensions

Powered by Google App Engine
This is Rietveld 408576698