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

Unified Diff: chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc

Issue 2910983002: Improve and fix counter texts (Closed)
Patch Set: fix ios failures, add random change to browsing_data_utils.h to trigger all tests Created 3 years, 7 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
« no previous file with comments | « no previous file | components/browsing_data/core/browsing_data_utils.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc b/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
index 16eeeced783df043ba7a01ba71e70cc1f01e8f1f..8406b656953835107463bf750c9808bccec94574 100644
--- a/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
@@ -46,12 +46,12 @@ TEST_F(BrowsingDataCounterUtilsTest, HostedAppsCounterResult) {
std::string apps_list;
std::string expected_output;
} kTestCases[] = {
- { "", "none" },
- { "App1", "1 app (App1)" },
- { "App1, App2", "2 apps (App1, App2)" },
- { "App1, App2, App3", "3 apps (App1, App2, and 1 more)" },
- { "App1, App2, App3, App4", "4 apps (App1, App2, and 2 more)" },
- { "App1, App2, App3, App4, App5", "5 apps (App1, App2, and 3 more)" },
+ {"", "None"},
+ {"App1", "1 app (App1)"},
+ {"App1, App2", "2 apps (App1, App2)"},
+ {"App1, App2, App3", "3 apps (App1, App2, and 1 more)"},
+ {"App1, App2, App3, App4", "4 apps (App1, App2, and 2 more)"},
+ {"App1, App2, App3, App4, App5", "5 apps (App1, App2, and 3 more)"},
};
for (const TestCase& test_case : kTestCases) {
« no previous file with comments | « no previous file | components/browsing_data/core/browsing_data_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698