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

Side by Side Diff: components/autofill/core/browser/autofill_regexes_unittest.cc

Issue 658993002: Convert ARRAYSIZE_UNSAFE -> arraysize in components/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/autofill/core/browser/autofill_regexes.h" 5 #include "components/autofill/core/browser/autofill_regexes.h"
6 6
7 #include "base/strings/string16.h" 7 #include "base/strings/string16.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "components/autofill/core/browser/autofill_regex_constants.h" 9 #include "components/autofill/core/browser/autofill_regex_constants.h"
10 #include "testing/gtest/include/gtest/gtest.h" 10 #include "testing/gtest/include/gtest/gtest.h"
(...skipping 16 matching lines...) Expand all
27 {"string", "tri"}, 27 {"string", "tri"},
28 // Substring at beginning 28 // Substring at beginning
29 {"string", "str"}, 29 {"string", "str"},
30 {"string", "^str"}, 30 {"string", "^str"},
31 // Substring at end 31 // Substring at end
32 {"string", "ring"}, 32 {"string", "ring"},
33 {"string", "ring$"}, 33 {"string", "ring$"},
34 // Case-insensitive 34 // Case-insensitive
35 {"StRiNg", "string"}, 35 {"StRiNg", "string"},
36 }; 36 };
37 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPositiveCases); ++i) { 37 for (size_t i = 0; i < arraysize(kPositiveCases); ++i) {
38 const TestCase& test_case = kPositiveCases[i]; 38 const TestCase& test_case = kPositiveCases[i];
39 SCOPED_TRACE(test_case.input); 39 SCOPED_TRACE(test_case.input);
40 SCOPED_TRACE(test_case.pattern); 40 SCOPED_TRACE(test_case.pattern);
41 EXPECT_TRUE(autofill::MatchesPattern(ASCIIToUTF16(test_case.input), 41 EXPECT_TRUE(autofill::MatchesPattern(ASCIIToUTF16(test_case.input),
42 ASCIIToUTF16(test_case.pattern))); 42 ASCIIToUTF16(test_case.pattern)));
43 } 43 }
44 44
45 const TestCase kNegativeCases[] = { 45 const TestCase kNegativeCases[] = {
46 // Empty string 46 // Empty string
47 {"", "Look, ma' -- a non-empty pattern!"}, 47 {"", "Look, ma' -- a non-empty pattern!"},
48 // Substring 48 // Substring
49 {"string", "trn"}, 49 {"string", "trn"},
50 // Substring at beginning 50 // Substring at beginning
51 {"string", " str"}, 51 {"string", " str"},
52 {"string", "^tri"}, 52 {"string", "^tri"},
53 // Substring at end 53 // Substring at end
54 {"string", "ring "}, 54 {"string", "ring "},
55 {"string", "rin$"}, 55 {"string", "rin$"},
56 }; 56 };
57 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kNegativeCases); ++i) { 57 for (size_t i = 0; i < arraysize(kNegativeCases); ++i) {
58 const TestCase& test_case = kNegativeCases[i]; 58 const TestCase& test_case = kNegativeCases[i];
59 SCOPED_TRACE(test_case.input); 59 SCOPED_TRACE(test_case.input);
60 SCOPED_TRACE(test_case.pattern); 60 SCOPED_TRACE(test_case.pattern);
61 EXPECT_FALSE(autofill::MatchesPattern(ASCIIToUTF16(test_case.input), 61 EXPECT_FALSE(autofill::MatchesPattern(ASCIIToUTF16(test_case.input),
62 ASCIIToUTF16(test_case.pattern))); 62 ASCIIToUTF16(test_case.pattern)));
63 } 63 }
64 } 64 }
65 65
66 } // namespace autofill 66 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698