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

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

Issue 2228913003: Move part of browsing data counter text util method to components (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 4 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/browsing_data/browsing_data_counter_utils.cc
diff --git a/chrome/browser/browsing_data/browsing_data_counter_utils.cc b/chrome/browser/browsing_data/browsing_data_counter_utils.cc
index 4b7087f29544fbfd17766c2d633f845bc09281b2..02de813c03a7146d49da25f02e60fad95c70dfb3 100644
--- a/chrome/browser/browsing_data/browsing_data_counter_utils.cc
+++ b/chrome/browser/browsing_data/browsing_data_counter_utils.cc
@@ -12,11 +12,9 @@
#include "chrome/common/chrome_switches.h"
#include "chrome/common/pref_names.h"
#include "chrome/grit/generated_resources.h"
-#include "components/browsing_data/core/counters/autofill_counter.h"
-#include "components/browsing_data/core/counters/history_counter.h"
-#include "components/browsing_data/core/counters/passwords_counter.h"
#include "components/browsing_data/core/pref_names.h"
#include "components/prefs/pref_service.h"
+#include "grit/components_strings.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/text/bytes_formatting.h"
@@ -54,29 +52,16 @@ base::string16 FormatBytesMBOrHigher(
bytes, ui::DataUnits::DATA_UNITS_MEBIBYTE, true);
}
-base::string16 GetCounterTextFromResult(
+base::string16 GetChromeCounterTextFromResult(
const browsing_data::BrowsingDataCounter::Result* result) {
- base::string16 text;
std::string pref_name = result->source()->GetPrefName();
if (!result->Finished()) {
// The counter is still counting.
- text = l10n_util::GetStringUTF16(IDS_CLEAR_BROWSING_DATA_CALCULATING);
-
- } else if (pref_name == browsing_data::prefs::kDeletePasswords ||
- pref_name == browsing_data::prefs::kDeleteDownloadHistory) {
- // Counters with trivially formatted result: passwords and downloads.
- browsing_data::BrowsingDataCounter::ResultInt count =
- static_cast<const browsing_data::BrowsingDataCounter::FinishedResult*>(
- result)
- ->Value();
- text = l10n_util::GetPluralStringFUTF16(
- pref_name == browsing_data::prefs::kDeletePasswords
- ? IDS_DEL_PASSWORDS_COUNTER
- : IDS_DEL_DOWNLOADS_COUNTER,
- count);
+ return l10n_util::GetStringUTF16(IDS_CLEAR_BROWSING_DATA_CALCULATING);
+ }
- } else if (pref_name == browsing_data::prefs::kDeleteCache) {
+ if (pref_name == browsing_data::prefs::kDeleteCache) {
// Cache counter.
browsing_data::BrowsingDataCounter::ResultInt cache_size_bytes =
static_cast<const browsing_data::BrowsingDataCounter::FinishedResult*>(
@@ -93,109 +78,28 @@ base::string16 GetCounterTextFromResult(
static const int kBytesInAMegabyte = 1024 * 1024;
if (cache_size_bytes >= kBytesInAMegabyte) {
base::string16 formatted_size = FormatBytesMBOrHigher(cache_size_bytes);
- text = time_period == browsing_data::ALL_TIME
+ return time_period == browsing_data::ALL_TIME
? formatted_size
: l10n_util::GetStringFUTF16(
IDS_DEL_CACHE_COUNTER_UPPER_ESTIMATE, formatted_size);
- } else {
- text = l10n_util::GetStringUTF16(IDS_DEL_CACHE_COUNTER_ALMOST_EMPTY);
- }
-
- } else if (pref_name == browsing_data::prefs::kDeleteBrowsingHistory) {
- // History counter.
- const browsing_data::HistoryCounter::HistoryResult* history_result =
- static_cast<const browsing_data::HistoryCounter::HistoryResult*>(
- result);
- browsing_data::BrowsingDataCounter::ResultInt local_item_count =
- history_result->Value();
- bool has_synced_visits = history_result->has_synced_visits();
-
- text = has_synced_visits
- ? l10n_util::GetPluralStringFUTF16(
- IDS_DEL_BROWSING_HISTORY_COUNTER_SYNCED, local_item_count)
- : l10n_util::GetPluralStringFUTF16(
- IDS_DEL_BROWSING_HISTORY_COUNTER, local_item_count);
-
- } else if (pref_name == browsing_data::prefs::kDeleteFormData) {
- // Autofill counter.
- const browsing_data::AutofillCounter::AutofillResult* autofill_result =
- static_cast<const browsing_data::AutofillCounter::AutofillResult*>(
- result);
- browsing_data::AutofillCounter::ResultInt num_suggestions =
- autofill_result->Value();
- browsing_data::AutofillCounter::ResultInt num_credit_cards =
- autofill_result->num_credit_cards();
- browsing_data::AutofillCounter::ResultInt num_addresses =
- autofill_result->num_addresses();
-
- std::vector<base::string16> displayed_strings;
-
- if (num_credit_cards) {
- displayed_strings.push_back(l10n_util::GetPluralStringFUTF16(
- IDS_DEL_AUTOFILL_COUNTER_CREDIT_CARDS, num_credit_cards));
- }
- if (num_addresses) {
- displayed_strings.push_back(l10n_util::GetPluralStringFUTF16(
- IDS_DEL_AUTOFILL_COUNTER_ADDRESSES, num_addresses));
- }
- if (num_suggestions) {
- // We use a different wording for autocomplete suggestions based on the
- // length of the entire string.
- switch (displayed_strings.size()) {
- case 0:
- displayed_strings.push_back(l10n_util::GetPluralStringFUTF16(
- IDS_DEL_AUTOFILL_COUNTER_SUGGESTIONS, num_suggestions));
- break;
- case 1:
- displayed_strings.push_back(l10n_util::GetPluralStringFUTF16(
- IDS_DEL_AUTOFILL_COUNTER_SUGGESTIONS_LONG, num_suggestions));
- break;
- case 2:
- displayed_strings.push_back(l10n_util::GetPluralStringFUTF16(
- IDS_DEL_AUTOFILL_COUNTER_SUGGESTIONS_SHORT, num_suggestions));
- break;
- default:
- NOTREACHED();
- }
- }
-
- // Construct the resulting string from the sections in |displayed_strings|.
- switch (displayed_strings.size()) {
- case 0:
- text = l10n_util::GetStringUTF16(IDS_DEL_AUTOFILL_COUNTER_EMPTY);
- break;
- case 1:
- text = displayed_strings[0];
- break;
- case 2:
- text = l10n_util::GetStringFUTF16(IDS_DEL_AUTOFILL_COUNTER_TWO_TYPES,
- displayed_strings[0],
- displayed_strings[1]);
- break;
- case 3:
- text = l10n_util::GetStringFUTF16(IDS_DEL_AUTOFILL_COUNTER_THREE_TYPES,
- displayed_strings[0],
- displayed_strings[1],
- displayed_strings[2]);
- break;
- default:
- NOTREACHED();
}
+ return l10n_util::GetStringUTF16(IDS_DEL_CACHE_COUNTER_ALMOST_EMPTY);
+ }
- } else if (pref_name == browsing_data::prefs::kDeleteMediaLicenses) {
+ if (pref_name == browsing_data::prefs::kDeleteMediaLicenses) {
const MediaLicensesCounter::MediaLicenseResult* media_license_result =
static_cast<const MediaLicensesCounter::MediaLicenseResult*>(result);
if (media_license_result->Value() > 0) {
- text = l10n_util::GetStringFUTF16(
+ return l10n_util::GetStringFUTF16(
IDS_DEL_MEDIA_LICENSES_COUNTER_SITE_COMMENT,
base::UTF8ToUTF16(media_license_result->GetOneOrigin()));
- } else {
- text = l10n_util::GetStringUTF16(
- IDS_DEL_MEDIA_LICENSES_COUNTER_GENERAL_COMMENT);
}
+ return l10n_util::GetStringUTF16(
+ IDS_DEL_MEDIA_LICENSES_COUNTER_GENERAL_COMMENT);
+ }
#if defined(ENABLE_EXTENSIONS)
- } else if (pref_name == browsing_data::prefs::kDeleteHostedAppsData) {
+ if (pref_name == browsing_data::prefs::kDeleteHostedAppsData) {
// Hosted apps counter.
const HostedAppsCounter::HostedAppsResult* hosted_apps_result =
static_cast<const HostedAppsCounter::HostedAppsResult*>(result);
@@ -223,15 +127,15 @@ base::string16 GetCounterTextFromResult(
// The output string has both the number placeholder (#) and substitution
// placeholders ($1, $2, $3). First fetch the correct plural string first,
// then substitute the $ placeholders.
- text = base::ReplaceStringPlaceholders(
+ return base::ReplaceStringPlaceholders(
l10n_util::GetPluralStringFUTF16(
IDS_DEL_HOSTED_APPS_COUNTER, hosted_apps_count),
replacements,
nullptr);
-#endif
}
+#endif
- return text;
+ return browsing_data::GetCounterTextFromResult(result);
}
bool GetDeletionPreferenceFromDataType(

Powered by Google App Engine
This is Rietveld 408576698