Don't strip punctuation inside quotes in history search queries.

chrome/browser/history/text_database_unittest.cc

 // Copyright (c) 2011 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 <string>
 
 #include "base/file_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/scoped_temp_dir.h"
 #include "base/string_util.h"
@@ skipping 211 matching lines @@
 
   // We don't want to be dependent on the exact snippet algorithm, but we know
   // since we searched for "COUNTTAG" which occurs at the beginning of each
   // document, that each snippet should start with that.
   EXPECT_TRUE(StartsWithASCII(UTF16ToUTF8(results[0].snippet.text()),
                               "COUNTTAG", false));
   EXPECT_TRUE(StartsWithASCII(UTF16ToUTF8(results[1].snippet.text()),
                               "COUNTTAG", false));
   EXPECT_TRUE(StartsWithASCII(UTF16ToUTF8(results[2].snippet.text()),
                               "COUNTTAG", false));
+
+  // Try a quoted query.
+  results.clear();
+  db->GetTextMatches("\"News for nerds\"", options, &results, &unique_urls,
+                     &first_time_searched);
+  EXPECT_EQ(1U, results.size());
+
+  // Try a quoted query that contains punctuation.
+  results.clear();
+  db->GetTextMatches("\"Slashdot: News\"", options, &results, &unique_urls,
+                     &first_time_searched);
+  EXPECT_EQ(1U, results.size());
+
+  // Ensure that punctuation inside quotes is not being ignored.
+  results.clear();
+  db->GetTextMatches("\"Slashdot News\"", options, &results, &unique_urls,
+                     &first_time_searched);
+  EXPECT_EQ(0U, results.size());
+
+  // Try a quoted query that should *only* match the URL.
+  results.clear();
+  db->GetTextMatches("\"www.google.com\"", options, &results, &unique_urls,
+                     &first_time_searched);
+  EXPECT_EQ(1U, results.size());
+  EXPECT_EQ(0U, results[0].title_match_positions.size());
 }
 
 TEST_F(TextDatabaseTest, TimeRange) {
   // Make a database with some pages.
   const int kIdee1 = 200801;
   scoped_ptr<TextDatabase> db(CreateDB(kIdee1, true, true));
   ASSERT_TRUE(!!db.get());
   AddAllTestData(db.get());
 
   // Beginning should be inclusive, and the ending exclusive.
@@ skipping 69 matching lines @@
 
   // There should be one result, the most recent one.
   EXPECT_EQ(1U, results.size());
   EXPECT_TRUE(ResultsHaveURL(results, kURL2));
 
   // The max time considered should be the date of the returned item.
   EXPECT_EQ(kTime2, first_time_searched.ToInternalValue());
 }
 
 }  // namespace history