EnumerateModules: Address UI review comments....

chrome/browser/enumerate_modules_model_win.h

 // Copyright (c) 2010 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_ENUMERATE_MODULES_MODEL_WIN_H_
 #define CHROME_BROWSER_ENUMERATE_MODULES_MODEL_WIN_H_
 #pragma once
 
 #include <utility>
 #include <vector>
@@ skipping 11 matching lines @@
 
 // A helper class that implements the enumerate module functionality on the File
 // thread.
 class ModuleEnumerator : public base::RefCountedThreadSafe<ModuleEnumerator> {
  public:
   // What type of module we are dealing with. Loaded modules are modules we
   // detect as loaded in the process at the time of scanning. The others are
   // modules of interest and may or may not be loaded in the process at the
   // time of scan.
   enum ModuleType {
-    LOADED_MODULE,
-    SHELL_EXTENSION,
-    WINSOCK_MODULE_REGISTRATION,
+    LOADED_MODULE               = 1 << 0,
+    SHELL_EXTENSION             = 1 << 1,
+    WINSOCK_MODULE_REGISTRATION = 1 << 2,
   };
 
   // The blacklist status of the module. Suspected Bad modules have been
   // partially matched (ie. name matches and location, but not description)
   // whereas Confirmed Bad modules have been identified further (ie.
   // AuthentiCode signer matches).
   enum ModuleStatus {
     // This is returned by the matching function when comparing against the
     // blacklist and the module does not match the current entry in the
     // blacklist.
@@ skipping 44 matching lines @@
   };
 
   // A vector typedef of all modules enumerated.
   typedef std::vector<Module> ModulesVector;
 
   // A structure we populate with the blacklist entries.
   struct BlacklistEntry {
     const char* filename;
     const char* location;
     const char* desc_or_signer;
-    const char* version_from;
-    const char* version_to;
+    const char* version_from;  // Version where conflict started.
+    const char* version_to;    // First version that works.
     RecommendedAction help_tip;
   };
 
   // A static function that normalizes the module information in the |module|
   // struct. Module information needs to be normalized before comparing against
   // the blacklist. This is because the same module can be described in many
   // different ways, ie. file paths can be presented in long/short name form,
   // and are not case sensitive on Windows. Also, the version string returned
   // can include appended text, which we don't want to use during comparison
   // against the blacklist.
@@ skipping 164 matching lines @@
   int confirmed_bad_modules_detected_;
 
   // The number of suspected bad modules (not including confirmed bad ones)
   // found during last scan.
   int suspected_bad_modules_detected_;
 
   DISALLOW_COPY_AND_ASSIGN(EnumerateModulesModel);
 };
 
 #endif  // CHROME_BROWSER_ENUMERATE_MODULES_MODEL_WIN_H_