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

Unified Diff: chrome/browser/android/browsing_data/browsing_data_bridge.cc

Issue 2752263003: Count site data size for important sites (Closed)
Patch Set: rebase 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 | « chrome/browser/BUILD.gn ('k') | chrome/browser/engagement/important_sites_usage_counter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/android/browsing_data/browsing_data_bridge.cc
diff --git a/chrome/browser/android/browsing_data/browsing_data_bridge.cc b/chrome/browser/android/browsing_data/browsing_data_bridge.cc
index dabd5b4b222db0e76b326c0f660082427c06d19c..16d57d11e752b49ceaf4f3071518c7adc2e16a0b 100644
--- a/chrome/browser/android/browsing_data/browsing_data_bridge.cc
+++ b/chrome/browser/android/browsing_data/browsing_data_bridge.cc
@@ -43,8 +43,6 @@ using content::BrowsingDataRemover;
namespace {
-const size_t kMaxImportantSites = 5;
-
Profile* GetOriginalProfile() {
return ProfileManager::GetActiveUserProfile()->GetOriginalProfile();
}
@@ -194,8 +192,8 @@ static void FetchImportantSites(JNIEnv* env,
const JavaParamRef<jobject>& java_callback) {
Profile* profile = GetOriginalProfile();
std::vector<ImportantSitesUtil::ImportantDomainInfo> important_sites =
- ImportantSitesUtil::GetImportantRegisterableDomains(profile,
- kMaxImportantSites);
+ ImportantSitesUtil::GetImportantRegisterableDomains(
+ profile, ImportantSitesUtil::kMaxImportantSites);
bool dialog_disabled = ImportantSitesUtil::IsDialogDisabled(profile);
std::vector<std::string> important_domains;
@@ -222,7 +220,7 @@ static void FetchImportantSites(JNIEnv* env,
// This value should not change during a sessions, as it's used for UMA metrics.
static jint GetMaxImportantSites(JNIEnv* env,
const JavaParamRef<jclass>& clazz) {
- return kMaxImportantSites;
+ return ImportantSitesUtil::kMaxImportantSites;
}
static void MarkOriginAsImportantForTesting(
« no previous file with comments | « chrome/browser/BUILD.gn ('k') | chrome/browser/engagement/important_sites_usage_counter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698