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

Unified Diff: chrome/browser/history/top_sites_cache_unittest.cc

Issue 655413002: Convert ARRAYSIZE_UNSAFE -> arraysize in chrome/browser/. (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/history/top_sites_cache_unittest.cc
diff --git a/chrome/browser/history/top_sites_cache_unittest.cc b/chrome/browser/history/top_sites_cache_unittest.cc
index 899f924bf5386871734515e521cd3340335a069f..cc3eb792c419368ad5564bf25d8b3a2fc74d2773 100644
--- a/chrome/browser/history/top_sites_cache_unittest.cc
+++ b/chrome/browser/history/top_sites_cache_unittest.cc
@@ -101,7 +101,7 @@ TEST_F(TopSitesCacheTest, GetCanonicalURL) {
// Prefix should not work: as-is.
{"http://www.youtube.com/a", "http://www.youtube.com/a"},
};
- for (size_t i = 0; i < ARRAYSIZE_UNSAFE(test_cases); ++i) {
+ for (size_t i = 0; i < arraysize(test_cases); ++i) {
std::string expected(test_cases[i].expected);
std::string query(test_cases[i].query);
EXPECT_EQ(expected, cache_.GetCanonicalURL(GURL(query)).spec())
@@ -200,7 +200,7 @@ TEST_F(TopSitesCacheTest, GetGeneralizedCanonicalURL) {
// Specialized match only: fails.
{"", "http://www.chromium.org/a"},
};
- for (size_t i = 0; i < ARRAYSIZE_UNSAFE(test_cases); ++i) {
+ for (size_t i = 0; i < arraysize(test_cases); ++i) {
std::string expected(test_cases[i].expected);
std::string query(test_cases[i].query);
GURL result(cache_.GetGeneralizedCanonicalURL(GURL(query)));
@@ -233,7 +233,7 @@ TEST_F(TopSitesCacheTest, GetPrefixCanonicalURLDiffByQuery) {
{"http://www.dest.com/2", "http://www.source.com/a/t?q=4"},
};
- for (size_t i = 0; i < ARRAYSIZE_UNSAFE(test_cases); ++i) {
+ for (size_t i = 0; i < arraysize(test_cases); ++i) {
std::string expected(test_cases[i].expected);
std::string query(test_cases[i].query);
GURL result(cache_.GetGeneralizedCanonicalURL(GURL(query)));
« no previous file with comments | « chrome/browser/history/in_memory_url_index_unittest.cc ('k') | chrome/browser/history/url_utils_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698