Omnibox: Fix to Display Bookmarks With Leading Spaces

chrome/browser/autocomplete/bookmark_provider_unittest.cc

 // Copyright (c) 2012 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/autocomplete/bookmark_provider.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 44 matching lines @@
   {"featherhead resuscitates achlorhydric mockingbirds",
    "http://www.featherheads.com/c" },
   {"mockingbirds resuscitates featherheads achlorhydric",
    "http://www.featherheads.com/d" },
   // For testing URL boosting.
   {"burning worms #1", "http://www.burned.com/" },
   {"burning worms #2", "http://www.worms.com/" },
   {"worming burns #10", "http://www.burned.com/" },
   {"worming burns #20", "http://www.worms.com/" },
   {"jive music", "http://www.worms.com/" },
+  // For testing strange spacing in bookmark titles.
+  {" hello1  hello2  ", "http://whatever.com/" },
 };
 
 class BookmarkProviderTest : public testing::Test {
  public:
   BookmarkProviderTest();
 
  protected:
   virtual void SetUp() OVERRIDE;
 
   test::TestBookmarkClient client_;
@@ skipping 171 matching lines @@
     {"\"carry carbon\"",      1, {{{0, 12}, {0, 0}}}},
     {"\"carry carbon\" care", 1, {{{0, 12}, {13, 17}, {0, 0}}}},
     // Quoted terms require complete word matches.
     {"\"carry carbo\"",       0, {{{0, 0}}}},
     // This set uses duplicated and/or overlaps search terms in the title.
     {"frank",                 1, {{{0, 5}, {8, 13}, {16, 21}, {0, 0}}}},
     {"frankly",               1, {{{0, 7}, {8, 15}, {0, 0}}}},
     {"frankly frankly",       1, {{{0, 7}, {8, 15}, {0, 0}}}},
     {"foobar foo",            1, {{{0, 6}, {7, 13}, {0, 0}}}},
     {"foo foobar",            1, {{{0, 6}, {7, 13}, {0, 0}}}},
+    // This one makes sure that leading whitespace in the title is removed.
+    {"hello",                 1, {{{0, 5}, {7, 12}, {0, 0}}}},
   };
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(query_data); ++i) {
     AutocompleteInput input(base::ASCIIToUTF16(query_data[i].query),
                             base::string16::npos, base::string16(), GURL(),
                             metrics::OmniboxEventProto::INVALID_SPEC, false,
                             false, false, true,
                             ChromeAutocompleteSchemeClassifier(profile_.get()));
     provider_->Start(input, false);
     const ACMatches& matches(provider_->matches());
@@ skipping 194 matching lines @@
       ASSERT_EQ(2U, chunks.size()) << description;
       size_t offset;
       EXPECT_TRUE(base::StringToSizeT(chunks[0], &offset)) << description;
       EXPECT_EQ(offset, match.contents_class[i].offset) << description;
       int style;
       EXPECT_TRUE(base::StringToInt(chunks[1], &style)) << description;
       EXPECT_EQ(style, match.contents_class[i].style) << description;
     }
   }
 }