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

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

Issue 339017: Loosen RFC 1738 compliance check to allow underscores where we already allowe... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 1 month 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.cc ('k') | net/base/net_util.h » ('j') | 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) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 "base/message_loop.h" 5 #include "base/message_loop.h"
6 #include "base/string_util.h" 6 #include "base/string_util.h"
7 #include "chrome/browser/autocomplete/autocomplete.h" 7 #include "chrome/browser/autocomplete/autocomplete.h"
8 #include "chrome/common/notification_registrar.h" 8 #include "chrome/common/notification_registrar.h"
9 #include "chrome/common/notification_service.h" 9 #include "chrome/common/notification_service.h"
10 #include "testing/gtest/include/gtest/gtest.h" 10 #include "testing/gtest/include/gtest/gtest.h"
(...skipping 225 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 { L"view-source:http://www.foo.com/", AutocompleteInput::URL }, 236 { L"view-source:http://www.foo.com/", AutocompleteInput::URL },
237 { L"javascript:alert(\"Hey there!\");", AutocompleteInput::URL }, 237 { L"javascript:alert(\"Hey there!\");", AutocompleteInput::URL },
238 #if defined(OS_WIN) 238 #if defined(OS_WIN)
239 { L"C:\\Program Files", AutocompleteInput::URL }, 239 { L"C:\\Program Files", AutocompleteInput::URL },
240 { L"\\\\Server\\Folder\\File", AutocompleteInput::URL }, 240 { L"\\\\Server\\Folder\\File", AutocompleteInput::URL },
241 #endif // defined(OS_WIN) 241 #endif // defined(OS_WIN)
242 { L"http:foo", AutocompleteInput::URL }, 242 { L"http:foo", AutocompleteInput::URL },
243 { L"http://foo", AutocompleteInput::URL }, 243 { L"http://foo", AutocompleteInput::URL },
244 { L"http://foo.c", AutocompleteInput::URL }, 244 { L"http://foo.c", AutocompleteInput::URL },
245 { L"http://foo.com", AutocompleteInput::URL }, 245 { L"http://foo.com", AutocompleteInput::URL },
246 { L"http://foo_bar.com", AutocompleteInput::URL },
246 { L"http://-.com", AutocompleteInput::QUERY }, 247 { L"http://-.com", AutocompleteInput::QUERY },
248 { L"http://_foo_.com", AutocompleteInput::QUERY },
247 { L"http://foo.com:abc", AutocompleteInput::QUERY }, 249 { L"http://foo.com:abc", AutocompleteInput::QUERY },
248 { L"http://foo.com:123456", AutocompleteInput::QUERY }, 250 { L"http://foo.com:123456", AutocompleteInput::QUERY },
249 { L"http:user@foo.com", AutocompleteInput::URL }, 251 { L"http:user@foo.com", AutocompleteInput::URL },
250 { L"http://user@foo.com", AutocompleteInput::URL }, 252 { L"http://user@foo.com", AutocompleteInput::URL },
251 { L"http://user:pass@foo.com", AutocompleteInput::URL }, 253 { L"http://user:pass@foo.com", AutocompleteInput::URL },
252 { L"http://1.2", AutocompleteInput::URL }, 254 { L"http://1.2", AutocompleteInput::URL },
253 { L"http://1.2/45", AutocompleteInput::URL }, 255 { L"http://1.2/45", AutocompleteInput::URL },
254 { L"http:ps/2 games", AutocompleteInput::URL }, 256 { L"http:ps/2 games", AutocompleteInput::URL },
255 { L"http://ps/2 games", AutocompleteInput::URL }, 257 { L"http://ps/2 games", AutocompleteInput::URL },
256 { L"127.0.0.1", AutocompleteInput::URL }, 258 { L"127.0.0.1", AutocompleteInput::URL },
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
344 EXPECT_EQ(input_cases[i].scheme.len, scheme.len) << "Input: " << 346 EXPECT_EQ(input_cases[i].scheme.len, scheme.len) << "Input: " <<
345 input_cases[i].input; 347 input_cases[i].input;
346 EXPECT_EQ(input_cases[i].host.begin, host.begin) << "Input: " << 348 EXPECT_EQ(input_cases[i].host.begin, host.begin) << "Input: " <<
347 input_cases[i].input; 349 input_cases[i].input;
348 EXPECT_EQ(input_cases[i].host.len, host.len) << "Input: " << 350 EXPECT_EQ(input_cases[i].host.len, host.len) << "Input: " <<
349 input_cases[i].input; 351 input_cases[i].input;
350 } 352 }
351 } 353 }
352 354
353 } // namespace 355 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/autocomplete/autocomplete.cc ('k') | net/base/net_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698