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

Unified Diff: components/component_updater/component_updater_paths.cc

Issue 1937683002: Implement support in DefaultComponentInstaller for picking up bundled (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready for review. Created 4 years, 8 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: components/component_updater/component_updater_paths.cc
diff --git a/components/component_updater/component_updater_paths.cc b/components/component_updater/component_updater_paths.cc
index 510725b35795fe5500d730a2421b010b424c30f4..ae8f2d7935d627ca6b0c849200de25865e3e45fe 100644
--- a/components/component_updater/component_updater_paths.cc
+++ b/components/component_updater/component_updater_paths.cc
@@ -12,47 +12,41 @@ namespace component_updater {
namespace {
// This key gives the root directory of all the component installations.
-static int g_components_root_key = -1;
+static int g_components_system_root_key = -1;
+static int g_components_user_root_key = -1;
} // namespace
bool PathProvider(int key, base::FilePath* result) {
- DCHECK_GT(g_components_root_key, 0);
+ DCHECK_GT(g_components_user_root_key, 0);
+ DCHECK_GT(g_components_system_root_key, 0);
// Early exit here to prevent a potential infinite loop when we retrieve
- // the path for g_components_root_key.
+ // the path for g_components_*_root_key.
if (key < PATH_START || key > PATH_END)
return false;
+ switch (key) {
+ case DIR_COMPONENT_SYSTEM:
+ return PathService::Get(g_components_system_root_key, result);
+ case DIR_COMPONENT_USER:
+ return PathService::Get(g_components_user_root_key, result);
+ }
+
base::FilePath cur;
- if (!PathService::Get(g_components_root_key, &cur))
+ if (!PathService::Get(g_components_user_root_key, &cur))
return false;
switch (key) {
- case DIR_COMPONENT_CLD2:
- cur = cur.Append(FILE_PATH_LITERAL("CLD"));
- break;
case DIR_RECOVERY_BASE:
cur = cur.Append(FILE_PATH_LITERAL("recovery"));
break;
case DIR_SWIFT_SHADER:
cur = cur.Append(FILE_PATH_LITERAL("SwiftShader"));
break;
- case DIR_SW_REPORTER:
- cur = cur.Append(FILE_PATH_LITERAL("SwReporter"));
- break;
- case DIR_COMPONENT_EV_WHITELIST:
- cur = cur.Append(FILE_PATH_LITERAL("EVWhitelist"));
- break;
case DIR_SUPERVISED_USER_WHITELISTS:
cur = cur.Append(FILE_PATH_LITERAL("SupervisedUserWhitelists"));
break;
- case DIR_CERT_TRANS_TREE_STATES:
- cur = cur.Append(FILE_PATH_LITERAL("CertificateTransparency"));
- break;
- case DIR_ORIGIN_TRIAL_KEYS:
- cur = cur.Append(FILE_PATH_LITERAL("OriginTrials"));
- break;
default:
return false;
}
@@ -63,12 +57,19 @@ bool PathProvider(int key, base::FilePath* result) {
// This cannot be done as a static initializer sadly since Visual Studio will
// eliminate this object file if there is no direct entry point into it.
-void RegisterPathProvider(int components_root_key) {
- DCHECK_EQ(g_components_root_key, -1);
- DCHECK_GT(components_root_key, 0);
- DCHECK(components_root_key < PATH_START || components_root_key > PATH_END);
+void RegisterPathProvider(int components_system_root_key,
+ int components_user_root_key) {
+ DCHECK_EQ(g_components_system_root_key, -1);
+ DCHECK_EQ(g_components_user_root_key, -1);
+ DCHECK_GT(components_system_root_key, 0);
+ DCHECK_GT(components_user_root_key, 0);
+ DCHECK(components_system_root_key < PATH_START ||
+ components_system_root_key > PATH_END);
+ DCHECK(components_user_root_key < PATH_START ||
+ components_user_root_key > PATH_END);
- g_components_root_key = components_root_key;
+ g_components_system_root_key = components_system_root_key;
+ g_components_user_root_key = components_user_root_key;
PathService::RegisterProvider(PathProvider, PATH_START, PATH_END);
}

Powered by Google App Engine
This is Rietveld 408576698