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

Unified Diff: chrome/browser/extensions/component_extensions_whitelist/whitelist.cc

Issue 2129413002: Use output_all_resource_defines=false in some chrome/ grd files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase, address comment, more whitelist.cc cleanup Created 4 years, 5 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 | « chrome/browser/browser_resources.grd ('k') | chrome/browser/extensions/component_loader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/component_extensions_whitelist/whitelist.cc
diff --git a/chrome/browser/extensions/component_extensions_whitelist/whitelist.cc b/chrome/browser/extensions/component_extensions_whitelist/whitelist.cc
index 39783ef4dd899aae1ac470d38d0f015f2a9d8215..ca4d2cfb3063c427598b250ca67275bd0dc40ec5 100644
--- a/chrome/browser/extensions/component_extensions_whitelist/whitelist.cc
+++ b/chrome/browser/extensions/component_extensions_whitelist/whitelist.cc
@@ -10,8 +10,8 @@
#include "base/macros.h"
#include "build/build_config.h"
#include "chrome/common/extensions/extension_constants.h"
+#include "chrome/grit/browser_resources.h"
#include "extensions/common/constants.h"
-#include "grit/browser_resources.h"
#if defined(ENABLE_APP_LIST) && defined(OS_CHROMEOS)
#include "chrome/browser/ui/app_list/google_now_extension.h"
@@ -27,7 +27,7 @@
namespace extensions {
bool IsComponentExtensionWhitelisted(const std::string& extension_id) {
- const char* allowed[] = {
+ const char* const kAllowed[] = {
extension_misc::kHotwordSharedModuleId,
extension_misc::kInAppPaymentsSupportAppId,
#if defined(ENABLE_MEDIA_ROUTER)
@@ -41,16 +41,17 @@ bool IsComponentExtensionWhitelisted(const std::string& extension_id) {
#endif
};
- for (size_t i = 0; i < arraysize(allowed); ++i) {
- if (extension_id == allowed[i])
+ for (size_t i = 0; i < arraysize(kAllowed); ++i) {
+ if (extension_id == kAllowed[i])
return true;
}
#if defined(ENABLE_APP_LIST) && defined(OS_CHROMEOS)
std::string google_now_extension_id;
if (GetGoogleNowExtensionId(&google_now_extension_id) &&
- google_now_extension_id == extension_id)
+ google_now_extension_id == extension_id) {
return true;
+ }
#endif
#if defined(OS_CHROMEOS)
@@ -66,51 +67,58 @@ bool IsComponentExtensionWhitelisted(const std::string& extension_id) {
}
bool IsComponentExtensionWhitelisted(int manifest_resource_id) {
- int allowed[] = {
- IDR_BOOKMARKS_MANIFEST,
- IDR_CHROME_APP_MANIFEST,
- IDR_CLOUDPRINT_MANIFEST,
- IDR_CONNECTIVITY_DIAGNOSTICS_MANIFEST,
- IDR_CRYPTOTOKEN_MANIFEST,
- IDR_FEEDBACK_MANIFEST,
- IDR_GAIA_AUTH_MANIFEST,
- IDR_GOOGLE_NOW_MANIFEST,
- IDR_HANGOUT_SERVICES_MANIFEST,
- IDR_HOTWORD_AUDIO_VERIFICATION_MANIFEST,
- IDR_HOTWORD_MANIFEST,
- IDR_IDENTITY_API_SCOPE_APPROVAL_MANIFEST,
- IDR_NETWORK_SPEECH_SYNTHESIS_MANIFEST,
- IDR_WALLPAPERMANAGER_MANIFEST,
- IDR_WEBSTORE_MANIFEST,
-#if defined(IMAGE_LOADER_EXTENSION)
- IDR_IMAGE_LOADER_MANIFEST,
+ switch (manifest_resource_id) {
+ // Please keep the list in alphabetical order.
+ case IDR_BOOKMARKS_MANIFEST:
+#if defined(ENABLE_APP_LIST)
+ case IDR_CHROME_APP_MANIFEST:
#endif
-#if defined(OS_CHROMEOS)
- IDR_ARC_SUPPORT_MANIFEST,
- IDR_AUDIO_PLAYER_MANIFEST,
- IDR_CHROME_APPS_WEBSTORE_WIDGET_MANIFEST,
- IDR_CONNECTIVITY_DIAGNOSTICS_LAUNCHER_MANIFEST,
- IDR_CONNECTIVITY_DIAGNOSTICS_MANIFEST,
- IDR_CROSH_BUILTIN_MANIFEST,
- IDR_DEMO_APP_MANIFEST,
- IDR_EASY_UNLOCK_MANIFEST,
- IDR_EASY_UNLOCK_MANIFEST_SIGNIN,
- IDR_ECHO_MANIFEST,
- IDR_FILEMANAGER_MANIFEST,
- IDR_FIRST_RUN_DIALOG_MANIFEST,
- IDR_GALLERY_MANIFEST,
- IDR_GENIUS_APP_MANIFEST,
- IDR_HELP_MANIFEST,
- IDR_KEYBOARD_MANIFEST,
- IDR_MOBILE_MANIFEST,
- IDR_QUICKOFFICE_MANIFEST,
- IDR_VIDEO_PLAYER_MANIFEST,
- IDR_WALLPAPERMANAGER_MANIFEST,
+ case IDR_CLOUDPRINT_MANIFEST:
+ case IDR_CRYPTOTOKEN_MANIFEST:
+ case IDR_FEEDBACK_MANIFEST:
+ case IDR_GAIA_AUTH_MANIFEST:
+#if defined(ENABLE_GOOGLE_NOW)
+ case IDR_GOOGLE_NOW_MANIFEST:
#endif
- };
+#if defined(GOOGLE_CHROME_BUILD) || defined(ENABLE_HANGOUT_SERVICES_EXTENSION)
+ case IDR_HANGOUT_SERVICES_MANIFEST:
+#endif
+#if defined(ENABLE_HOTWORDING)
+ case IDR_HOTWORD_AUDIO_VERIFICATION_MANIFEST:
+ case IDR_HOTWORD_MANIFEST:
+#endif
+ case IDR_IDENTITY_API_SCOPE_APPROVAL_MANIFEST:
+#if defined(IMAGE_LOADER_EXTENSION)
+ case IDR_IMAGE_LOADER_MANIFEST:
+#endif
+ case IDR_NETWORK_SPEECH_SYNTHESIS_MANIFEST:
+ case IDR_WEBSTORE_MANIFEST:
- for (size_t i = 0; i < arraysize(allowed); ++i) {
- if (manifest_resource_id == allowed[i])
+#if defined(OS_CHROMEOS)
+ // Separate ChromeOS list, as it is quite large.
+ case IDR_ARC_SUPPORT_MANIFEST:
+ case IDR_AUDIO_PLAYER_MANIFEST:
+ case IDR_CHROME_APPS_WEBSTORE_WIDGET_MANIFEST:
+ case IDR_CONNECTIVITY_DIAGNOSTICS_LAUNCHER_MANIFEST:
+ case IDR_CONNECTIVITY_DIAGNOSTICS_MANIFEST:
+ case IDR_CROSH_BUILTIN_MANIFEST:
+ case IDR_DEMO_APP_MANIFEST:
+ case IDR_EASY_UNLOCK_MANIFEST:
+ case IDR_EASY_UNLOCK_MANIFEST_SIGNIN:
+ case IDR_ECHO_MANIFEST:
+ case IDR_FILEMANAGER_MANIFEST:
+ case IDR_FIRST_RUN_DIALOG_MANIFEST:
+ case IDR_GALLERY_MANIFEST:
+ case IDR_KEYBOARD_MANIFEST:
+ case IDR_MOBILE_MANIFEST:
+ case IDR_VIDEO_PLAYER_MANIFEST:
+ case IDR_WALLPAPERMANAGER_MANIFEST:
+#if defined(GOOGLE_CHROME_BUILD)
+ case IDR_GENIUS_APP_MANIFEST:
+ case IDR_HELP_MANIFEST:
+ case IDR_QUICKOFFICE_MANIFEST:
+#endif // defined(GOOGLE_CHROME_BUILD)
+#endif // defined(OS_CHROMEOS)
return true;
}
« no previous file with comments | « chrome/browser/browser_resources.grd ('k') | chrome/browser/extensions/component_loader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698