Index: chrome/browser/autocomplete/history_url_provider_unittest.cc |
diff --git a/chrome/browser/autocomplete/history_url_provider_unittest.cc b/chrome/browser/autocomplete/history_url_provider_unittest.cc |
index 893402c6f45fbf2698bc3713e55c62d1413d2af1..af55ae19e79f445a9d11b411b8725d4a5cd772a1 100644 |
--- a/chrome/browser/autocomplete/history_url_provider_unittest.cc |
+++ b/chrome/browser/autocomplete/history_url_provider_unittest.cc |
@@ -183,8 +183,6 @@ class HistoryURLProviderTest : public testing::Test, |
expected_urls, num_results, &type); |
} |
- void RunAdjustOffsetTest(const string16 text, size_t expected_offset); |
- |
content::TestBrowserThreadBundle thread_bundle_; |
ACMatches matches_; |
scoped_ptr<TestingProfile> profile_; |
@@ -286,19 +284,6 @@ void HistoryURLProviderTest::RunTest( |
EXPECT_EQ(expected_urls[i], matches_[i].destination_url.spec()); |
} |
-void HistoryURLProviderTest::RunAdjustOffsetTest(const string16 text, |
- size_t expected_offset) { |
- AutocompleteInput input(text, string16::npos, string16(), GURL(), false, |
- false, true, AutocompleteInput::ALL_MATCHES); |
- autocomplete_->Start(input, false); |
- if (!autocomplete_->done()) |
- base::MessageLoop::current()->Run(); |
- |
- matches_ = autocomplete_->matches(); |
- ASSERT_GE(matches_.size(), 1U) << "Input text: " << text; |
- EXPECT_EQ(expected_offset, matches_[0].inline_autocomplete_offset); |
-} |
- |
TEST_F(HistoryURLProviderTest, PromoteShorterURLs) { |
// Test that hosts get synthesized below popular pages. |
const std::string expected_nonsynth[] = { |
@@ -490,7 +475,8 @@ TEST_F(HistoryURLProviderTest, Fixup) { |
const std::string fixup_1[] = {"file:///C:/foo.txt"}; |
ASSERT_NO_FATAL_FAILURE(RunTest(input_1, string16(), false, fixup_1, |
arraysize(fixup_1))); |
- EXPECT_EQ(input_1.length(), matches_.front().inline_autocomplete_offset); |
+ EXPECT_EQ(ASCIIToUTF16("///C:/foo.txt"), |
+ matches_.front().inline_autocompletion); |
// Fixing up "http:/" should result in an inline autocomplete offset of just |
// after "http:/", not just after "http:". |
@@ -502,7 +488,8 @@ TEST_F(HistoryURLProviderTest, Fixup) { |
}; |
ASSERT_NO_FATAL_FAILURE(RunTest(input_2, string16(), false, fixup_2, |
arraysize(fixup_2))); |
- EXPECT_EQ(input_2.length(), matches_.front().inline_autocomplete_offset); |
+ EXPECT_EQ(ASCIIToUTF16("/bogussite.com/a"), |
+ matches_.front().inline_autocompletion); |
// Adding a TLD to a small number like "56" should result in "www.56.com" |
// rather than "0.0.0.56.com". |
@@ -522,12 +509,6 @@ TEST_F(HistoryURLProviderTest, Fixup) { |
arraysize(fixup_5)); |
} |
-TEST_F(HistoryURLProviderTest, AdjustOffset) { |
- RunAdjustOffsetTest(WideToUTF16(L"http://www.\uAD50\uC721"), 13); |
- RunAdjustOffsetTest(ASCIIToUTF16("http://spaces.com/path%20with%20spa"), 31); |
- RunAdjustOffsetTest(ASCIIToUTF16("http://ms/c++ s"), 15); |
-} |
- |
// Make sure the results for the input 'p' don't change between the first and |
// second passes. |
TEST_F(HistoryURLProviderTest, EmptyVisits) { |
@@ -580,7 +561,7 @@ TEST_F(HistoryURLProviderTest, DontAutocompleteOnTrailingWhitespace) { |
// None of the matches should attempt to autocomplete. |
matches_ = autocomplete_->matches(); |
for (size_t i = 0; i < matches_.size(); ++i) |
- EXPECT_EQ(string16::npos, matches_[i].inline_autocomplete_offset); |
+ EXPECT_TRUE(matches_[i].inline_autocompletion.empty()); |
} |
TEST_F(HistoryURLProviderTest, TreatEmailsAsSearches) { |