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

Unified Diff: chrome/installer/util/registry_entry.cc

Issue 623903002: Move shell_util's RegistryEntry class into a separate file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@shell_util-generic-associations
Patch Set: Do-over and rebase. Most of the changes have been taken care of in precursor CLs. Created 5 years 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/installer/util/registry_entry.h ('k') | chrome/installer/util/shell_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/util/registry_entry.cc
diff --git a/chrome/installer/util/registry_entry.cc b/chrome/installer/util/registry_entry.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6d51b064f8005385eedc3078334bdecf08e83916
--- /dev/null
+++ b/chrome/installer/util/registry_entry.cc
@@ -0,0 +1,107 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/installer/util/registry_entry.h"
+
+#include "base/logging.h"
+#include "base/strings/string_util.h"
+#include "base/win/registry.h"
+#include "chrome/installer/util/work_item.h"
+#include "chrome/installer/util/work_item_list.h"
+
+RegistryEntry::RegistryEntry(const base::string16& key_path,
+ const base::string16& value)
+ : key_path_(key_path),
+ name_(),
+ is_string_(true),
+ value_(value),
+ int_value_(0),
+ removal_flag_(RemovalFlag::NONE) {}
+
+RegistryEntry::RegistryEntry(const base::string16& key_path,
+ const base::string16& name,
+ const base::string16& value)
+ : key_path_(key_path),
+ name_(name),
+ is_string_(true),
+ value_(value),
+ int_value_(0),
+ removal_flag_(RemovalFlag::NONE) {}
+
+RegistryEntry::RegistryEntry(const base::string16& key_path,
+ const base::string16& name,
+ DWORD value)
+ : key_path_(key_path),
+ name_(name),
+ is_string_(false),
+ value_(),
+ int_value_(value),
+ removal_flag_(RemovalFlag::NONE) {}
+
+void RegistryEntry::AddToWorkItemList(HKEY root, WorkItemList* items) const {
+ if (removal_flag_ == RemovalFlag::VALUE) {
+ items->AddDeleteRegValueWorkItem(root, key_path_, WorkItem::kWow64Default,
+ name_);
+ } else if (removal_flag_ == RemovalFlag::KEY) {
+ items->AddDeleteRegKeyWorkItem(root, key_path_, WorkItem::kWow64Default);
+ } else {
+ DCHECK(removal_flag_ == RemovalFlag::NONE);
+ items->AddCreateRegKeyWorkItem(root, key_path_, WorkItem::kWow64Default);
+ if (is_string_) {
+ items->AddSetRegValueWorkItem(root, key_path_, WorkItem::kWow64Default,
+ name_, value_, true);
+ } else {
+ items->AddSetRegValueWorkItem(root, key_path_, WorkItem::kWow64Default,
+ name_, int_value_, true);
+ }
+ }
+}
+
+bool RegistryEntry::ExistsInRegistry(uint32 look_for_in) const {
+ DCHECK(look_for_in);
+
+ RegistryStatus status = DOES_NOT_EXIST;
+ if (look_for_in & LOOK_IN_HKCU)
+ status = StatusInRegistryUnderRoot(HKEY_CURRENT_USER);
+ if (status == DOES_NOT_EXIST && (look_for_in & LOOK_IN_HKLM))
+ status = StatusInRegistryUnderRoot(HKEY_LOCAL_MACHINE);
+ return status == SAME_VALUE;
+}
+
+bool RegistryEntry::KeyExistsInRegistry(uint32 look_for_in) const {
+ DCHECK(look_for_in);
+
+ RegistryStatus status = DOES_NOT_EXIST;
+ if (look_for_in & LOOK_IN_HKCU)
+ status = StatusInRegistryUnderRoot(HKEY_CURRENT_USER);
+ if (status == DOES_NOT_EXIST && (look_for_in & LOOK_IN_HKLM))
+ status = StatusInRegistryUnderRoot(HKEY_LOCAL_MACHINE);
+ return status != DOES_NOT_EXIST;
+}
+
+RegistryEntry::RegistryStatus RegistryEntry::StatusInRegistryUnderRoot(
+ HKEY root) const {
+ base::win::RegKey key(root, key_path_.c_str(), KEY_QUERY_VALUE);
Matt Giuca 2015/12/02 02:54:41 Note: This was the only line that changed during t
+ bool found = false;
+ bool correct_value = false;
+ if (is_string_) {
+ base::string16 read_value;
+ found = key.ReadValue(name_.c_str(), &read_value) == ERROR_SUCCESS;
+ if (found) {
+ correct_value =
+ read_value.size() == value_.size() &&
+ ::CompareString(
+ LOCALE_USER_DEFAULT, NORM_IGNORECASE, read_value.data(),
+ base::saturated_cast<int>(read_value.size()), value_.data(),
+ base::saturated_cast<int>(value_.size())) == CSTR_EQUAL;
+ }
+ } else {
+ DWORD read_value;
+ found = key.ReadValueDW(name_.c_str(), &read_value) == ERROR_SUCCESS;
+ if (found)
+ correct_value = read_value == int_value_;
+ }
+ return found ? (correct_value ? SAME_VALUE : DIFFERENT_VALUE)
+ : DOES_NOT_EXIST;
+}
« no previous file with comments | « chrome/installer/util/registry_entry.h ('k') | chrome/installer/util/shell_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698