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

Unified Diff: chrome/browser/conflicts/module_info_util_win.cc

Issue 2720513005: Add InspectModule() that returns a populated ModuleInspectionResult struct (Closed)
Patch Set: Another 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
Index: chrome/browser/conflicts/module_info_util_win.cc
diff --git a/chrome/browser/conflicts/module_info_util_win.cc b/chrome/browser/conflicts/module_info_util_win.cc
index 01b30b2e78ecea0562ea6d53c11013e92e4a7e44..d8bdbd753e385cb6e899cd7acdd6066122c41941 100644
--- a/chrome/browser/conflicts/module_info_util_win.cc
+++ b/chrome/browser/conflicts/module_info_util_win.cc
@@ -13,10 +13,15 @@
// This must be after wincrypt and wintrust.
#include <mscat.h>
+#include <limits>
#include <memory>
-#include <vector>
+#include <string>
+#include "base/environment.h"
+#include "base/i18n/case_conversion.h"
#include "base/scoped_generic.h"
+#include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
#include "base/win/scoped_handle.h"
namespace {
@@ -238,3 +243,49 @@ void GetCertificateInfo(const base::FilePath& filename,
certificate_info->path = filename;
certificate_info->subject = subject;
}
+
+StringMapping GetEnvironmentVariablesMapping(
+ const std::vector<base::string16>& environment_variables) {
+ std::unique_ptr<base::Environment> environment(base::Environment::Create());
+
+ StringMapping string_mapping;
+ for (const base::string16& variable : environment_variables) {
+ std::string value;
+ if (environment->GetVar(base::UTF16ToASCII(variable).c_str(), &value)) {
+ value = base::TrimString(value, "\\", base::TRIM_TRAILING).as_string();
+ string_mapping.push_back(
+ std::make_pair(base::i18n::ToLower(base::UTF8ToUTF16(value)),
+ L"%" + base::i18n::ToLower(variable) + L"%"));
+ }
+ }
+
+ return string_mapping;
+}
+
+void CollapseMatchingPrefixInPath(const StringMapping& prefix_mapping,
+ base::string16* path) {
+ const base::string16 path_copy = *path;
+ DCHECK_EQ(base::i18n::ToLower(path_copy), path_copy);
+
+ size_t min_length = std::numeric_limits<size_t>::max();
+ for (const auto& mapping : prefix_mapping) {
+ DCHECK_EQ(base::i18n::ToLower(mapping.first), mapping.first);
+ if (base::StartsWith(path_copy, mapping.first,
+ base::CompareCase::SENSITIVE)) {
+ // Make sure the matching prefix is a full path component.
+ if (path_copy[mapping.first.length()] != '\\' &&
+ path_copy[mapping.first.length()] != '\0') {
+ continue;
+ }
+
+ base::string16 collapsed_path = path_copy;
+ base::ReplaceFirstSubstringAfterOffset(&collapsed_path, 0, mapping.first,
+ mapping.second);
+ size_t length = collapsed_path.length() - mapping.second.length();
+ if (length < min_length) {
+ *path = collapsed_path;
+ min_length = length;
+ }
+ }
+ }
+}
« no previous file with comments | « chrome/browser/conflicts/module_info_util_win.h ('k') | chrome/browser/conflicts/module_info_util_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698