[ads] Do not fetch geo and catalog on iOS when Brave Rewards/News disabled

browser/brave_profile_prefs.cc

@@ -23,6 +23,8 @@
 #include "brave/components/brave_news/browser/brave_news_controller.h"
 #include "brave/components/brave_news/browser/brave_news_p3a.h"
 #include "brave/components/brave_news/browser/brave_news_pref_manager.h"
+#include "brave/components/brave_news/common/p3a_pref_names.h"
+#include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_perf_predictor/browser/p3a_bandwidth_savings_tracker.h"
 #include "brave/components/brave_perf_predictor/browser/perf_predictor_tab_helper.h"
 #include "brave/components/brave_rewards/common/pref_names.h"
@@ -190,7 +192,7 @@ void RegisterProfilePrefsForMigration(
 
   brave_rewards::RegisterProfilePrefsForMigration(registry);
 
-  brave_news::p3a::NewsMetrics::RegisterProfilePrefsForMigration(registry);
+  brave_news::p3a::prefs::RegisterProfileNewsMetricsPrefsForMigration(registry);
 
   // Added May 2023
 #if defined(TOOLKIT_VIEWS)
@@ -236,7 +238,7 @@ void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) {
 
   brave_shields::RegisterShieldsP3AProfilePrefs(registry);
 
-  brave_news::BraveNewsPrefManager::RegisterProfilePrefs(registry);
+  brave_news::prefs::RegisterProfilePrefs(registry);
 
   // TODO(shong): Migrate this to local state also and guard in ENABLE_WIDEVINE.
   // We don't need to display "don't ask widevine prompt option" in settings

browser/ui/day_zero_browser_ui_expt/BUILD.gn

@@ -12,7 +12,6 @@ source_set("unit_tests") {
 
   deps = [
     "//base",
-    "//brave/components/brave_news/browser",
     "//brave/components/brave_news/common",
     "//brave/components/brave_rewards/common",
     "//brave/components/brave_wallet/browser:pref_names",

browser/ui/day_zero_browser_ui_expt/day_zero_browser_ui_expt_manager.cc

@@ -10,7 +10,7 @@
 #include "base/time/time.h"
 #include "brave/browser/brave_browser_features.h"
 #include "brave/browser/brave_stats/first_run_util.h"
-#include "brave/components/brave_news/browser/locales_helper.h"
+#include "brave/components/brave_news/common/locales_helper.h"
 #include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_rewards/common/pref_names.h"
 #include "brave/components/brave_wallet/browser/pref_names.h"

browser/ui/day_zero_browser_ui_expt/day_zero_browser_ui_expt_unittest.cc

@@ -6,7 +6,7 @@
 #include "base/test/scoped_feature_list.h"
 #include "brave/browser/brave_browser_features.h"
 #include "brave/browser/ui/day_zero_browser_ui_expt/day_zero_browser_ui_expt_manager.h"
-#include "brave/components/brave_news/browser/locales_helper.h"
+#include "brave/components/brave_news/common/locales_helper.h"
 #include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_rewards/common/pref_names.h"
 #include "brave/components/brave_wallet/browser/pref_names.h"

chromium_src/chrome/browser/prefs/browser_prefs.cc

@@ -14,6 +14,8 @@
 #include "brave/components/ai_chat/core/common/buildflags/buildflags.h"
 #include "brave/components/brave_ads/core/public/prefs/obsolete_pref_util.h"
 #include "brave/components/brave_news/browser/brave_news_p3a.h"
+#include "brave/components/brave_news/common/p3a_pref_names.h"
+#include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_search_conversion/p3a.h"
 #include "brave/components/brave_shields/content/browser/brave_shields_p3a.h"
 #include "brave/components/brave_sync/brave_sync_prefs.h"
@@ -163,7 +165,7 @@ void MigrateObsoleteProfilePrefs(PrefService* profile_prefs,
   profile_prefs->ClearPref(sidebar::kSidebarAlignmentChangedTemporarily);
 #endif
 
-  brave_news::p3a::NewsMetrics::MigrateObsoleteProfilePrefs(profile_prefs);
+  brave_news::p3a::prefs::MigrateObsoleteProfileNewsMetricsPrefs(profile_prefs);
 
   // Added 2023-09
   ntp_background_images::ViewCounterService::MigrateObsoleteProfilePrefs(

chromium_src/ios/chrome/browser/shared/model/prefs/BUILD.gn

@@ -8,6 +8,7 @@ import("//brave/components/ipfs/buildflags/buildflags.gni")
 group("prefs") {
   deps = [
     "//brave/components/brave_ads/core",
+    "//brave/components/brave_news/common",
     "//brave/components/brave_rewards/common",
     "//brave/components/brave_shields/core/common",
     "//brave/components/brave_sync:prefs",

chromium_src/ios/chrome/browser/shared/model/prefs/DEPS

@@ -1,5 +1,6 @@
 include_rules = [
   "+brave/components/brave_ads/core/public",
+  "+brave/components/brave_news/common",
   "+brave/components/brave_rewards/common",
   "+brave/components/brave_sync",
   "+brave/components/brave_wallet/browser",

chromium_src/ios/chrome/browser/shared/model/prefs/browser_prefs.mm

@@ -7,6 +7,7 @@
 #include "brave/components/ai_chat/core/browser/model_service.h"
 #include "brave/components/ai_chat/core/common/pref_names.h"
 #include "brave/components/brave_ads/core/public/prefs/pref_registry.h"
+#include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_rewards/common/pref_registry.h"
 #include "brave/components/brave_shields/core/common/pref_names.h"
 #include "brave/components/brave_sync/brave_sync_prefs.h"
@@ -44,6 +45,7 @@ void BraveRegisterBrowserStatePrefs(
 #endif
   ai_chat::prefs::RegisterProfilePrefs(registry);
   ai_chat::ModelService::RegisterProfilePrefs(registry);
+  brave_news::prefs::RegisterProfilePrefs(registry);
 }
 
 void BraveRegisterLocalStatePrefs(PrefRegistrySimple* registry) {

components/brave_news/browser/BUILD.gn

@@ -40,8 +40,6 @@ static_library("browser") {
     "html_parsing.h",
     "initialization_promise.cc",
     "initialization_promise.h",
-    "locales_helper.cc",
-    "locales_helper.h",
     "network.cc",
     "network.h",
     "publishers_controller.cc",

components/brave_news/browser/brave_news_controller.cc

@@ -33,14 +33,14 @@
 #include "brave/components/brave_news/browser/channels_controller.h"
 #include "brave/components/brave_news/browser/direct_feed_controller.h"
 #include "brave/components/brave_news/browser/feed_v2_builder.h"
-#include "brave/components/brave_news/browser/locales_helper.h"
 #include "brave/components/brave_news/browser/network.h"
 #include "brave/components/brave_news/browser/publishers_controller.h"
 #include "brave/components/brave_news/browser/publishers_parsing.h"
 #include "brave/components/brave_news/browser/suggestions_controller.h"
 #include "brave/components/brave_news/browser/topics_fetcher.h"
 #include "brave/components/brave_news/common/brave_news.mojom.h"
 #include "brave/components/brave_news/common/features.h"
+#include "brave/components/brave_news/common/locales_helper.h"
 #include "brave/components/brave_news/common/subscriptions_snapshot.h"
 #include "brave/components/brave_private_cdn/private_cdn_helper.h"
 #include "brave/components/brave_private_cdn/private_cdn_request_helper.h"

components/brave_news/browser/brave_news_p3a.cc

@@ -13,13 +13,12 @@
 #include "base/metrics/histogram_functions.h"
 #include "base/metrics/histogram_macros.h"
 #include "brave/components/brave_news/browser/brave_news_pref_manager.h"
-#include "brave/components/brave_news/common/pref_names.h"
+#include "brave/components/brave_news/common/p3a_pref_names.h"
 #include "brave/components/brave_news/common/subscriptions_snapshot.h"
 #include "brave/components/brave_rewards/common/pref_names.h"
 #include "brave/components/p3a_utils/bucket.h"
 #include "brave/components/p3a_utils/feature_usage.h"
 #include "brave/components/time_period_storage/weekly_storage.h"
-#include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 
 namespace brave_news::p3a {
@@ -310,30 +309,6 @@ void NewsMetrics::RecordWeeklySessionCount(bool is_add) {
                                      total_session_count);
 }
 
-void NewsMetrics::RegisterProfilePrefs(PrefRegistrySimple* registry) {
-  registry->RegisterListPref(prefs::kBraveNewsWeeklySessionCount);
-  registry->RegisterListPref(prefs::kBraveNewsWeeklyDisplayAdViewedCount);
-  registry->RegisterListPref(prefs::kBraveNewsWeeklyAddedDirectFeedsCount);
-  registry->RegisterListPref(prefs::kBraveNewsTotalCardViews);
-  registry->RegisterListPref(prefs::kBraveNewsTotalCardVisits);
-  registry->RegisterListPref(prefs::kBraveNewsVisitDepthSum);
-  registry->RegisterListPref(prefs::kBraveNewsTotalSidebarFilterUsages);
-  p3a_utils::RegisterFeatureUsagePrefs(
-      registry, prefs::kBraveNewsFirstSessionTime,
-      prefs::kBraveNewsLastSessionTime, prefs::kBraveNewsUsedSecondDay, nullptr,
-      nullptr);
-  registry->RegisterBooleanPref(prefs::kBraveNewsWasEverEnabled, false);
-}
-
-void NewsMetrics::RegisterProfilePrefsForMigration(
-    PrefRegistrySimple* registry) {
-  // Reserved for future deprecated P3A-related prefs
-}
-
-void NewsMetrics::MigrateObsoleteProfilePrefs(PrefService* prefs) {
-  // Reserved for future deprecated P3A-related prefs
-}
-
 void NewsMetrics::OnConfigChanged() {
   DVLOG(1) << __FUNCTION__;
   if (was_enabled_ == pref_manager_->IsEnabled()) {

components/brave_news/browser/brave_news_p3a.h

@@ -12,7 +12,6 @@
 #include "base/timer/wall_clock_timer.h"
 #include "brave/components/brave_news/browser/brave_news_pref_manager.h"
 
-class PrefRegistrySimple;
 class PrefService;
 
 namespace brave_news::p3a {
@@ -59,10 +58,6 @@ class NewsMetrics : public BraveNewsPrefManager::PrefObserver {
   NewsMetrics(const NewsMetrics&) = delete;
   NewsMetrics& operator=(const NewsMetrics&) = delete;
 
-  static void RegisterProfilePrefs(PrefRegistrySimple* registry);
-  static void RegisterProfilePrefsForMigration(PrefRegistrySimple* registry);
-  static void MigrateObsoleteProfilePrefs(PrefService* prefs);
-
   void RecordAtInit();
   void RecordAtSessionStart();
 

components/brave_news/browser/brave_news_p3a_unittest.cc

@@ -10,9 +10,11 @@
 #include "base/test/metrics/histogram_tester.h"
 #include "brave/components/brave_news/browser/brave_news_controller.h"
 #include "brave/components/brave_news/browser/brave_news_pref_manager.h"
+#include "brave/components/brave_news/common/p3a_pref_names.h"
 #include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_rewards/common/pref_names.h"
 #include "brave/components/time_period_storage/weekly_storage.h"
+#include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/scoped_user_pref_update.h"
 #include "components/prefs/testing_pref_service.h"
 #include "content/public/test/browser_task_environment.h"
@@ -29,7 +31,7 @@ class BraveNewsP3ATest : public testing::Test {
   void SetUp() override {
     task_environment_.AdvanceClock(base::Days(2));
     PrefRegistrySimple* registry = pref_service_.registry();
-    BraveNewsPrefManager::RegisterProfilePrefs(registry);
+    ::brave_news::prefs::RegisterProfilePrefs(registry);
     registry->RegisterBooleanPref(brave_rewards::prefs::kEnabled, false);
 
     pref_manager_ = std::make_unique<BraveNewsPrefManager>(pref_service_);
@@ -343,14 +345,14 @@ TEST_F(BraveNewsP3ATest, TestIsEnabled) {
   PrefService* prefs = GetPrefs();
 
   // should not record "disabled" if never opted in
-  prefs->SetBoolean(prefs::kNewTabPageShowToday, false);
+  prefs->SetBoolean(::brave_news::prefs::kNewTabPageShowToday, false);
   histogram_tester_.ExpectTotalCount(kIsEnabledHistogramName, 0);
 
-  prefs->SetBoolean(prefs::kBraveNewsOptedIn, true);
-  prefs->SetBoolean(prefs::kNewTabPageShowToday, true);
+  prefs->SetBoolean(::brave_news::prefs::kBraveNewsOptedIn, true);
+  prefs->SetBoolean(::brave_news::prefs::kNewTabPageShowToday, true);
   histogram_tester_.ExpectUniqueSample(kIsEnabledHistogramName, 1, 1);
 
-  prefs->SetBoolean(prefs::kNewTabPageShowToday, false);
+  prefs->SetBoolean(::brave_news::prefs::kNewTabPageShowToday, false);
   histogram_tester_.ExpectBucketCount(kIsEnabledHistogramName, 0, 1);
 }
 

components/brave_news/browser/brave_news_pref_manager.cc

@@ -17,32 +17,16 @@
 #include "base/uuid.h"
 #include "brave/components/brave_news/browser/brave_news_p3a.h"
 #include "brave/components/brave_news/browser/channel_migrator.h"
-#include "brave/components/brave_news/browser/locales_helper.h"
 #include "brave/components/brave_news/common/brave_news.mojom.h"
 #include "brave/components/brave_news/common/pref_names.h"
 #include "brave/components/brave_news/common/subscriptions_snapshot.h"
 #include "components/prefs/pref_change_registrar.h"
 #include "components/prefs/pref_observer.h"
-#include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 #include "components/prefs/scoped_user_pref_update.h"
 
 namespace brave_news {
 
-// static
-void BraveNewsPrefManager::RegisterProfilePrefs(PrefRegistrySimple* registry) {
-  registry->RegisterBooleanPref(prefs::kShouldShowToolbarButton, true);
-  registry->RegisterBooleanPref(prefs::kNewTabPageShowToday,
-                                IsUserInDefaultEnabledLocale());
-  registry->RegisterBooleanPref(prefs::kBraveNewsOptedIn, false);
-  registry->RegisterDictionaryPref(prefs::kBraveNewsSources);
-  registry->RegisterDictionaryPref(prefs::kBraveNewsChannels);
-  registry->RegisterDictionaryPref(prefs::kBraveNewsDirectFeeds);
-  registry->RegisterBooleanPref(prefs::kBraveNewsOpenArticlesInNewTab, true);
-
-  p3a::NewsMetrics::RegisterProfilePrefs(registry);
-}
-
 BraveNewsPrefManager::BraveNewsPrefManager(PrefService& prefs) : prefs_(prefs) {
   pref_change_registrar_.Init(&*prefs_);
   pref_change_registrar_.Add(

components/brave_news/browser/brave_news_pref_manager.h

@@ -15,7 +15,6 @@
 #include "brave/components/brave_news/common/brave_news.mojom-forward.h"
 #include "components/prefs/pref_change_registrar.h"
 #include "components/prefs/pref_observer.h"
-#include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 #include "url/gurl.h"
 
@@ -36,8 +35,6 @@ class BraveNewsPrefManager {
     virtual void OnChannelsChanged() {}
   };
 
-  static void RegisterProfilePrefs(PrefRegistrySimple* registry);
-
   explicit BraveNewsPrefManager(PrefService& prefs);
   ~BraveNewsPrefManager();
 

components/brave_news/browser/feed_fetcher.cc

@@ -25,12 +25,12 @@
 #include "brave/components/brave_news/browser/combined_feed_parsing.h"
 #include "brave/components/brave_news/browser/direct_feed_fetcher.h"
 #include "brave/components/brave_news/browser/feed_controller.h"
-#include "brave/components/brave_news/browser/locales_helper.h"
 #include "brave/components/brave_news/browser/network.h"
 #include "brave/components/brave_news/browser/publishers_controller.h"
 #include "brave/components/brave_news/browser/urls.h"
 #include "brave/components/brave_news/common/brave_news.mojom-forward.h"
 #include "brave/components/brave_news/common/brave_news.mojom-shared.h"
+#include "brave/components/brave_news/common/locales_helper.h"
 #include "brave/components/brave_news/common/subscriptions_snapshot.h"
 #include "brave/components/brave_private_cdn/headers.h"
 #include "services/network/public/cpp/shared_url_loader_factory.h"

components/brave_news/browser/publishers_controller.cc

@@ -20,7 +20,6 @@
 #include "base/ranges/algorithm.h"
 #include "base/strings/strcat.h"
 #include "brave/components/api_request_helper/api_request_helper.h"
-#include "brave/components/brave_news/browser/locales_helper.h"
 #include "brave/components/brave_news/browser/network.h"
 #include "brave/components/brave_news/browser/publishers_parsing.h"
 #include "brave/components/brave_news/browser/urls.h"

components/brave_news/browser/publishers_controller.h

@@ -13,11 +13,11 @@
 #include "base/one_shot_event.h"
 #include "brave/components/api_request_helper/api_request_helper.h"
 #include "brave/components/brave_news/common/brave_news.mojom.h"
+#include "brave/components/brave_news/common/locales_helper.h"
 
 namespace brave_news {
 
 using GetPublishersCallback = mojom::BraveNewsController::GetPublishersCallback;
-using Publishers = base::flat_map<std::string, mojom::PublisherPtr>;
 
 class SubscriptionsSnapshot;
 

components/brave_news/browser/publishers_parsing.h

@@ -11,8 +11,8 @@
 
 #include "base/values.h"
 #include "brave/components/brave_news/browser/brave_news_pref_manager.h"
-#include "brave/components/brave_news/browser/publishers_controller.h"
 #include "brave/components/brave_news/common/brave_news.mojom-forward.h"
+#include "brave/components/brave_news/common/locales_helper.h"
 
 namespace brave_news {
 

components/brave_news/browser/publishers_parsing_unittest.cc

@@ -5,14 +5,12 @@
 
 #include "brave/components/brave_news/browser/publishers_parsing.h"
 
-#include <memory>
 #include <optional>
-#include <utility>
-#include <vector>
+#include <string>
 
 #include "base/containers/flat_map.h"
 #include "base/test/values_test_util.h"
-#include "brave/components/brave_news/common/brave_news.mojom-forward.h"
+#include "brave/components/brave_news/common/brave_news.mojom.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace brave_news {

components/brave_news/browser/test/BUILD.gn

@@ -20,7 +20,6 @@ source_set("brave_news_unit_tests") {
     "//brave/components/brave_news/browser/feed_sampling_unittest.cc",
     "//brave/components/brave_news/browser/html_parsing_unittest.cc",
     "//brave/components/brave_news/browser/initialization_promise_unittest.cc",
-    "//brave/components/brave_news/browser/locales_helper_unittest.cc",
     "//brave/components/brave_news/browser/publishers_controller_unittest.cc",
     "//brave/components/brave_news/browser/publishers_parsing_unittest.cc",
     "//brave/components/brave_news/browser/suggestions_controller_unittest.cc",

components/brave_news/common/BUILD.gn

@@ -9,7 +9,10 @@ import("//mojo/public/tools/bindings/mojom.gni")
 source_set("unit_tests") {
   testonly = true
 
-  sources = [ "subscriptions_snapshot_unittest.cc" ]
+  sources = [
+    "locales_helper_unittest.cc",
+    "subscriptions_snapshot_unittest.cc",
+  ]
 
   deps = [
     ":common",
@@ -22,15 +25,23 @@ static_library("common") {
   sources = [
     "features.cc",
     "features.h",
+    "locales_helper.cc",
+    "locales_helper.h",
+    "p3a_pref_names.cc",
+    "p3a_pref_names.h",
     "pref_names.cc",
     "pref_names.h",
     "subscriptions_snapshot.cc",
     "subscriptions_snapshot.h",
     "switches.h",
   ]
 
+  public_deps = [ ":mojom" ]
+
   deps = [
     "//base",
+    "//brave/components/l10n/common",
+    "//brave/components/p3a_utils",
     "//components/prefs",
     "//url",
   ]