Add InspectModule() that returns a populated ModuleInspectionResult struct

chrome/browser/conflicts/module_info_win.h

 // 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.
 
 #ifndef CHROME_BROWSER_CONFLICTS_MODULE_INFO_WIN_H_
 #define CHROME_BROWSER_CONFLICTS_MODULE_INFO_WIN_H_
 
+#include <memory>
+
 #include "base/files/file_path.h"
 #include "chrome/browser/conflicts/module_info_util_win.h"
 
 // ModuleInfoKey and ModuleInfoData are used in pair by the ModuleDatabase to
 // maintain information about a module, usually in a std::map.
 
 // Used by the ModuleDatabase as a unique identifier for a module.
 using ModuleId = int;
 
 // This is the constant portion of the module information, and is used to
@@ skipping 18 matching lines @@
   // TimeDateStamp from the module's IMAGE_FILE_HEADER.
   uint32_t module_time_date_stamp;
 
   // The ID of this module. This is a strictly incrementing value, and is used
   // by the ModuleDatabase to tie a module to the list of running processes in
   // which it is found. It is not part of the key for the module, but it is
   // immutable. This is simply the index of the module in the insertion order.
   ModuleId module_id;
 };
 
-// Holds more detailed information about a given module. Because most of this
+// Holds more detailed information about a given module. Because all of this
 // information is expensive to gather and require disk access, it should be

[chrisha, 2017/03/04 15:54:17]

requires*

[Patrick Monette, 2017/03/06 16:39:48]

Done.

-// collected on a task runner that allow blocking.
-struct ModuleInfoData {
-  ModuleInfoData();
-  ModuleInfoData(const ModuleInfoData& others);
-  ~ModuleInfoData();
-
-  // Set of all process types in which this module has been seen (may not be
-  // currently present in a process of that type). This is a conversion of
-  // ProcessType enumeration to a bitfield. See "ProcessTypeToBit" and
-  // "BitIndexToProcessType" for details.
-  uint32_t process_types;
-
-  // The following pieces of information are determined via a detailed
-  // inspection of the module.
+// collected via InspectModule() on a task runner that allow blocking.
+struct ModuleInspectionResult {
+  ModuleInspectionResult();
+  ~ModuleInspectionResult();
 
   // The module path, not including the basename. This is cleaned and normalized
   // so that common paths are converted to their environment variable mappings
   // (ie, %systemroot%). This makes i18n localized paths easily comparable.
   base::string16 location;
 
   // The basename of the module.
   base::string16 basename;
 
   // The name of the product the module belongs to.
   base::string16 product_name;
 
   // The module file description.
   base::string16 description;
 
   // The module version. This is usually in the form a.b.c.d (where a, b, c and
   // d are integers), but may also have fewer than 4 components.
   base::string16 version;
 
   // The certificate info for the module.
   CertificateInfo certificate_info;
 };
 
+// Contains the inspection result of a module and additional information that is
+// useful to the ModuleDatabase.
+struct ModuleInfoData {
+  ModuleInfoData();
+  ~ModuleInfoData();
+
+  // Set of all process types in which this module has been seen (may not be
+  // currently present in a process of that type). This is a conversion of
+  // ProcessType enumeration to a bitfield. See "ProcessTypeToBit" and
+  // "BitIndexToProcessType" for details.
+  uint32_t process_types;
+
+  // The inspection result obtained via InspectModule().
+  std::unique_ptr<ModuleInspectionResult> inspection_result;
+};
+
+// Given a module identified by |module_key|, returns a populated
+// ModuleInspectionResult that contains detailed information about the module on
+// disk. This is a blocking task that requires access to disk.
+std::unique_ptr<ModuleInspectionResult> InspectModule(
+    const StringMapping& env_variable_mapping,
+    const ModuleInfoKey& module_key);
+
+namespace internal {
+
+// Normalizes the information already contained in |inspection_result|. In
+// particular,

[chrisha, 2017/03/04 15:54:17]

particular:

[Patrick Monette, 2017/03/06 16:39:47]

Done.

+// - The path is split in 2 parts: The basename and the location.
+// - If it uses commas, the version string is modified to use periods.
+// - If there is one, the version string suffix is removed.
+// - If there is one, the trailing null character in the subject string of the
+//   certificate info is removed.
+//
+// Exposed for testing.
+void NormalizeInspectionResult(ModuleInspectionResult* inspection_result);
+
+}  // namespace internal
+
 #endif  // CHROME_BROWSER_CONFLICTS_MODULE_INFO_WIN_H_