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

Side by Side Diff: chrome/browser/autocomplete/autocomplete_input_unittest.cc

Issue 11896016: Omnibox: Trim Leading Spaces After Initial Forced-Query '?' (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: added tests Created 7 years, 11 months 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_input.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/autocomplete/autocomplete_input.h" 5 #include "chrome/browser/autocomplete/autocomplete_input.h"
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "base/string16.h" 8 #include "base/string16.h"
9 #include "base/utf_string_conversions.h" 9 #include "base/utf_string_conversions.h"
10 #include "build/build_config.h" 10 #include "build/build_config.h"
(...skipping 204 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 // regular case, no changes. 215 // regular case, no changes.
216 { ASCIIToUTF16("foo bar"), 3, ASCIIToUTF16("foo bar"), 3 }, 216 { ASCIIToUTF16("foo bar"), 3, ASCIIToUTF16("foo bar"), 3 },
217 217
218 // extra leading space. 218 // extra leading space.
219 { ASCIIToUTF16(" foo bar"), 3, ASCIIToUTF16("foo bar"), 1 }, 219 { ASCIIToUTF16(" foo bar"), 3, ASCIIToUTF16("foo bar"), 1 },
220 { ASCIIToUTF16(" foo bar"), 3, ASCIIToUTF16("foo bar"), 0 }, 220 { ASCIIToUTF16(" foo bar"), 3, ASCIIToUTF16("foo bar"), 0 },
221 { ASCIIToUTF16(" foo bar "), 2, ASCIIToUTF16("foo bar "), 0 }, 221 { ASCIIToUTF16(" foo bar "), 2, ASCIIToUTF16("foo bar "), 0 },
222 222
223 // forced query. 223 // forced query.
224 { ASCIIToUTF16("?foo bar"), 2, ASCIIToUTF16("foo bar"), 1 }, 224 { ASCIIToUTF16("?foo bar"), 2, ASCIIToUTF16("foo bar"), 1 },
225 { ASCIIToUTF16(" ?foo bar"), 4, ASCIIToUTF16("foo bar"), 1 },
226 { ASCIIToUTF16("? foo bar"), 4, ASCIIToUTF16("foo bar"), 1 },
227 { ASCIIToUTF16(" ? foo bar"), 6, ASCIIToUTF16("foo bar"), 1 },
225 }; 228 };
226 229
227 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(input_cases); ++i) { 230 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(input_cases); ++i) {
228 SCOPED_TRACE(input_cases[i].input); 231 SCOPED_TRACE(input_cases[i].input);
229 AutocompleteInput input(input_cases[i].input, 232 AutocompleteInput input(input_cases[i].input,
230 input_cases[i].cursor_position, 233 input_cases[i].cursor_position,
231 string16(), true, false, true, 234 string16(), true, false, true,
232 AutocompleteInput::ALL_MATCHES); 235 AutocompleteInput::ALL_MATCHES);
233 EXPECT_EQ(input_cases[i].normalized_input, input.text()); 236 EXPECT_EQ(input_cases[i].normalized_input, input.text());
234 EXPECT_EQ(input_cases[i].normalized_cursor_position, 237 EXPECT_EQ(input_cases[i].normalized_cursor_position,
235 input.cursor_position()); 238 input.cursor_position());
236 } 239 }
237 } 240 }
OLDNEW
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_input.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698