Count site data size for important sites

chrome/browser/android/browsing_data/browsing_data_bridge.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/browsing_data/browsing_data_bridge.h"
 
 #include <jni.h>
 #include <stddef.h>
 
 #include <memory>
@@ skipping 23 matching lines @@
 #include "jni/BrowsingDataBridge_jni.h"
 
 using base::android::AttachCurrentThread;
 using base::android::JavaParamRef;
 using base::android::JavaRef;
 using base::android::ScopedJavaLocalRef;
 using base::android::ScopedJavaGlobalRef;
 
 namespace {
 
-const size_t kMaxImportantSites = 5;
-
 Profile* GetOriginalProfile() {
   return ProfileManager::GetActiveUserProfile()->GetOriginalProfile();
 }
 
 // Merges |task_count| BrowsingDataRemover completion callbacks and redirects
 // them back into Java.
 class ClearBrowsingDataObserver : public BrowsingDataRemover::Observer {
  public:
   // |obj| is expected to be the object passed into ClearBrowsingData(); e.g. a
   // ChromePreference.
@@ skipping 156 matching lines @@
       chrome::GetChannel(),
       base::Bind(&EnableDialogAboutOtherFormsOfBrowsingHistory,
                  ScopedJavaGlobalRef<jobject>(env, listener)));
 }
 
 static void FetchImportantSites(JNIEnv* env,
                                 const JavaParamRef<jclass>& clazz,
                                 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;
   std::vector<int32_t> important_domain_reasons;
   std::vector<std::string> important_domain_examples;
   for (const ImportantSitesUtil::ImportantDomainInfo& info : important_sites) {
     important_domains.push_back(info.registerable_domain);
     important_domain_reasons.push_back(info.reason_bitfield);
     important_domain_examples.push_back(info.example_origin.spec());
   }
 
   ScopedJavaLocalRef<jobjectArray> java_domains =
       base::android::ToJavaArrayOfStrings(env, important_domains);
   ScopedJavaLocalRef<jintArray> java_reasons =
       base::android::ToJavaIntArray(env, important_domain_reasons);
   ScopedJavaLocalRef<jobjectArray> java_origins =
       base::android::ToJavaArrayOfStrings(env, important_domain_examples);
 
   Java_ImportantSitesCallback_onImportantRegisterableDomainsReady(
       env, java_callback.obj(), java_domains.obj(), java_origins.obj(),
       java_reasons.obj(), dialog_disabled);
 }
 
 // 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(
     JNIEnv* env,
     const JavaParamRef<jclass>& clazz,
     const JavaParamRef<jstring>& jorigin) {
   GURL origin(base::android::ConvertJavaStringToUTF8(jorigin));
   CHECK(origin.is_valid());
   ImportantSitesUtil::MarkOriginAsImportantForTesting(GetOriginalProfile(),
                                                       origin);
 }