Commit 17bb64ec authored by Eric Lawrence's avatar Eric Lawrence Committed by Commit Bot

Fix misleading comment in NativeProcessLauncher::FindManifest

Update the code's comment to match the behavior; HKCU is checked first,
and then HKLM is consulted.

Bug: 1034919
Change-Id: I9e649766b6ce2328322b287267c97e9344449075
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2393975Reviewed-by: default avatarSergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Eric Lawrence [MSFT] <ericlaw@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#808979}
parent d76f9740
...@@ -64,7 +64,8 @@ base::FilePath NativeProcessLauncher::FindManifest( ...@@ -64,7 +64,8 @@ base::FilePath NativeProcessLauncher::FindManifest(
std::string* error_message) { std::string* error_message) {
base::string16 host_name_wide = base::UTF8ToUTF16(host_name); base::string16 host_name_wide = base::UTF8ToUTF16(host_name);
// Try to find the key in HKEY_LOCAL_MACHINE and then in HKEY_CURRENT_USER. // If permitted, look in HKEY_CURRENT_USER first. If the manifest isn't found
// there, then try HKEY_LOCAL_MACHINE. https://crbug.com/1034919#c6
base::string16 path_str; base::string16 path_str;
bool found = false; bool found = false;
if (allow_user_level_hosts) if (allow_user_level_hosts)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment