Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(989)

Unified Diff: chrome/browser/autocomplete/autocomplete_result_unittest.cc

Issue 11414303: Make Google Search autocomplete provider cursor aware. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autocomplete/autocomplete_result_unittest.cc
diff --git a/chrome/browser/autocomplete/autocomplete_result_unittest.cc b/chrome/browser/autocomplete/autocomplete_result_unittest.cc
index 788f568c121361a863b72ab6774b80ab86b49d9d..65d4cc4d987aa2d54b8a37323a95a6f33266eb0b 100644
--- a/chrome/browser/autocomplete/autocomplete_result_unittest.cc
+++ b/chrome/browser/autocomplete/autocomplete_result_unittest.cc
@@ -115,8 +115,8 @@ void AutocompleteResultTest::RunCopyOldMatchesTest(
const TestData* last, size_t last_size,
const TestData* current, size_t current_size,
const TestData* expected, size_t expected_size) {
- AutocompleteInput input(ASCIIToUTF16("a"), string16(), false, false, false,
- AutocompleteInput::ALL_MATCHES);
+ AutocompleteInput input(ASCIIToUTF16("a"), string16::npos, string16(), false,
+ false, false, AutocompleteInput::ALL_MATCHES);
ACMatches last_matches;
PopulateAutocompleteMatches(last, last_size, &last_matches);
@@ -147,8 +147,8 @@ TEST_F(AutocompleteResultTest, Swap) {
// Swap with a single match.
ACMatches matches;
AutocompleteMatch match;
- AutocompleteInput input(ASCIIToUTF16("a"), string16(), false, false, false,
- AutocompleteInput::ALL_MATCHES);
+ AutocompleteInput input(ASCIIToUTF16("a"), string16::npos, string16(), false,
+ false, false, AutocompleteInput::ALL_MATCHES);
matches.push_back(match);
r1.AppendMatches(matches);
r1.SortAndCull(input, test_util_.profile());
@@ -228,8 +228,8 @@ TEST_F(AutocompleteResultTest, SortAndCullEmptyDestinationURLs) {
AutocompleteResult result;
result.AppendMatches(matches);
- AutocompleteInput input(string16(), string16(), false, false, false,
- AutocompleteInput::ALL_MATCHES);
+ AutocompleteInput input(string16(), string16::npos, string16(), false, false,
+ false, AutocompleteInput::ALL_MATCHES);
result.SortAndCull(input, test_util_.profile());
// Of the two results with the same non-empty destination URL, the
@@ -272,8 +272,8 @@ TEST_F(AutocompleteResultTest, SortAndCullDuplicateSearchURLs) {
AutocompleteResult result;
result.AppendMatches(matches);
- AutocompleteInput input(string16(), string16(), false, false, false,
- AutocompleteInput::ALL_MATCHES);
+ AutocompleteInput input(string16(), string16::npos, string16(), false, false,
+ false, AutocompleteInput::ALL_MATCHES);
result.SortAndCull(input, test_util_.profile());
// We expect the 3rd and 4th results to be removed.

Powered by Google App Engine
This is Rietveld 408576698