Add InspectModule() that returns a populated ModuleInspectionResult struct

chrome/browser/conflicts/module_info_util_win.cc

 // Copyright 2017 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/browser/conflicts/module_info_util_win.h"
 
 #include <windows.h>
 
 #include <tlhelp32.h>
 #include <wincrypt.h>
 #include <wintrust.h>
 
 // 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 {
 
 // Helper for scoped tracking an HCERTSTORE.
 struct ScopedHCERTSTORETraits {
   static HCERTSTORE InvalidValue() { return nullptr; }
   static void Free(HCERTSTORE store) { ::CertCloseStore(store, 0); }
 };
 using ScopedHCERTSTORE =
@@ skipping 201 matching lines @@
     return;
 
   base::string16 subject = GetSubjectNameInFile(filename);
   if (subject.empty())
     return;
 
   certificate_info->type = CertificateType::CERTIFICATE_IN_FILE;
   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 CollapseMatchingPrefixInString(const StringMapping& prefix_mapping,
+                                    base::string16* string) {
+  const base::string16 string_copy = *string;
+  DCHECK_EQ(base::i18n::ToLower(string_copy), string_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(string_copy, mapping.first,
+                         base::CompareCase::SENSITIVE)) {
+      base::string16 collapsed_string = string_copy;
+      base::ReplaceFirstSubstringAfterOffset(&collapsed_string, 0,
+                                             mapping.first, mapping.second);
+      size_t length = collapsed_string.length() - mapping.second.length();

[chrisha, 2017/03/01 15:45:42]

Shouldn't we have logic to ensure the next character after the match is a slash?
Otherwise we could end up with "unnatural" substitutions:

c:\windows\system32_some_dir_i_made_up\foo.exe

would end up as:

%systemroot%_some_dir_i_made_up\foo.exe

Which maybe we don't want?

[Patrick Monette, 2017/03/02 20:27:07]

Done.

+      if (length < min_length) {
+        *string = collapsed_string;
+        min_length = length;
+      }
+    }
+  }
+}