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

Unified Diff: chrome_elf/chrome_elf_util_unittest.cc

Issue 2760853002: Remove use of PRODUCT_STRING_PATH in chrome_elf. (Closed)
Patch Set: now Created 3 years, 9 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_elf/chrome_elf_security.cc ('k') | chrome_elf/run_all_unittests.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_elf/chrome_elf_util_unittest.cc
diff --git a/chrome_elf/chrome_elf_util_unittest.cc b/chrome_elf/chrome_elf_util_unittest.cc
index db25af06171a46fe01f686166573969e4652f65b..5fc1fc5f8361893758d1c766637c01f02be6220e 100644
--- a/chrome_elf/chrome_elf_util_unittest.cc
+++ b/chrome_elf/chrome_elf_util_unittest.cc
@@ -8,6 +8,7 @@
#include "base/test/test_reg_util_win.h"
#include "base/win/registry.h"
+#include "chrome/install_static/install_util.h"
#include "chrome_elf/chrome_elf_constants.h"
#include "chrome_elf/chrome_elf_security.h"
#include "chrome_elf/nt_registry/nt_registry.h"
@@ -17,14 +18,16 @@ namespace {
bool SetSecurityFinchFlag(bool creation) {
bool success = true;
+ const base::string16 finch_path(install_static::GetRegistryPath().append(
+ elf_sec::kRegSecurityFinchKeyName));
base::win::RegKey security_key(HKEY_CURRENT_USER, L"", KEY_ALL_ACCESS);
if (creation) {
if (ERROR_SUCCESS !=
- security_key.CreateKey(elf_sec::kRegSecurityFinchPath, KEY_QUERY_VALUE))
+ security_key.CreateKey(finch_path.c_str(), KEY_QUERY_VALUE))
success = false;
} else {
- if (ERROR_SUCCESS != security_key.DeleteKey(elf_sec::kRegSecurityFinchPath))
+ if (ERROR_SUCCESS != security_key.DeleteKey(finch_path.c_str()))
success = false;
}
« no previous file with comments | « chrome_elf/chrome_elf_security.cc ('k') | chrome_elf/run_all_unittests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698