EnumerateModules: Address UI review comments....

chrome/browser/enumerate_modules_model_unittest_win.cc

 // 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.
 
 #include "base/string_number_conversions.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/enumerate_modules_model_win.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 142 matching lines @@
   }, {  // Matches: Name, location, signature, version (lower) => Confirmed.
     ModuleEnumerator::CONFIRMED_BAD,
     kStandardModule,
     { kMatchName, kMatchLocation, kMatchSignature,
       kVersionLow, kEmpty, ModuleEnumerator::SEE_LINK }
   }, {  // Matches: Name, location, signature, version (upper) => Confirmed.
     ModuleEnumerator::CONFIRMED_BAD,
     kStandardModule,
     { kMatchName, kMatchLocation, kMatchSignature,
       kEmpty, kVersionHigh, ModuleEnumerator::SEE_LINK }
+  }, {  // Matches: Name, Location, Version lower is inclusive => Confirmed.
+    ModuleEnumerator::CONFIRMED_BAD,
+    kStandardModule,
+    { kMatchName, kMatchLocation, kMatchSignature,
+      "1.0", "2.0", ModuleEnumerator::SEE_LINK }
+  }, {  // Matches: Name, Location, Version higher is exclusive => No match.
+    ModuleEnumerator::NOT_MATCHED,
+    kStandardModule,
+    { kMatchName, kMatchLocation, kEmpty,
+      "0.0", "1.0", ModuleEnumerator::SEE_LINK }
   }, {  // All empty fields doesn't produce a match.
     ModuleEnumerator::NOT_MATCHED,
-    {kType, kStatus, L"", L"", L"", L"", L""},
+    { kType, kStatus, L"", L"", L"", L"", L""},
     { "a.dll", "", "", "", "", ModuleEnumerator::SEE_LINK }
   },
 };
 
 TEST_F(EnumerateModulesTest, MatchFunction) {
   for (size_t i = 0; i < arraysize(kMatchineEntryList); ++i) {
     ModuleEnumerator::Module test = kMatchineEntryList[i].test_case;
     ModuleEnumerator::NormalizeModule(&test);
     ModuleEnumerator::BlacklistEntry blacklist =
         kMatchineEntryList[i].blacklist;
 
     SCOPED_TRACE("Test case no " + base::IntToString(i) +
                  ": '" + UTF16ToASCII(test.name) + "'");
     EXPECT_EQ(kMatchineEntryList[i].expected_result,
               ModuleEnumerator::Match(test, blacklist));
   }
 }