[ads] Preserve search ad info bar preference when clearing ads data

Author: aseren

components/brave_ads/core/internal/prefs/obsolete_pref_util.cc

@@ -5,6 +5,7 @@
 
 #include "brave/components/brave_ads/core/public/prefs/obsolete_pref_util.h"
 
+#include "brave/components/brave_ads/core/public/prefs/pref_names.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 
@@ -22,6 +23,9 @@ constexpr char kHasMigratedRewardsState[] =
 constexpr char kShouldMigrateVerifiedRewardsUser[] =
     "brave.brave_ads.rewards.verified_user.should_migrate";
 
+constexpr char kObsoleteShouldShowSearchResultAdClickedInfoBar[] =
+    "brave.brave_ads.should_show_search_result_ad_clicked_infobar";
+
 }  // namespace
 
 void RegisterProfilePrefsForMigration(PrefRegistrySimple* const registry) {
@@ -32,6 +36,10 @@ void RegisterProfilePrefsForMigration(PrefRegistrySimple* const registry) {
 
   // Added 10/2024.
   registry->RegisterBooleanPref(kShouldMigrateVerifiedRewardsUser, false);
+
+  // Added 12/2024.
+  registry->RegisterBooleanPref(kObsoleteShouldShowSearchResultAdClickedInfoBar,
+                                false);
 }
 
 void MigrateObsoleteProfilePrefs(PrefService* const prefs) {
@@ -42,6 +50,14 @@ void MigrateObsoleteProfilePrefs(PrefService* const prefs) {
 
   // Added 10/2024.
   prefs->ClearPref(kShouldMigrateVerifiedRewardsUser);
+
+  // Added 05/2025.
+  if (prefs->HasPrefPath(kObsoleteShouldShowSearchResultAdClickedInfoBar)) {
+    prefs->SetBoolean(
+        prefs::kShouldShowSearchResultAdClickedInfoBar,
+        prefs->GetBoolean(kObsoleteShouldShowSearchResultAdClickedInfoBar));
+    prefs->ClearPref(kObsoleteShouldShowSearchResultAdClickedInfoBar);
+  }
 }
 
 }  // namespace brave_ads

components/brave_ads/core/public/prefs/pref_names.h

@@ -8,6 +8,8 @@
 
 namespace brave_ads::prefs {
 
+// IMPORTANT: Prefs that need clearing should be prefixed with `brave.brave_ads`
+
 // Ads prefs.
 inline constexpr char kDiagnosticId[] = "brave.brave_ads.diagnostics.id";
 
@@ -68,9 +70,6 @@ inline constexpr char kNotificationAdDidFallbackToCustom[] =
 inline constexpr char kShouldShowOnboardingNotification[] =
     "brave.brave_ads.should_show_my_first_ad_notification";
 
-inline constexpr char kShouldShowSearchResultAdClickedInfoBar[] =
-    "brave.brave_ads.should_show_search_result_ad_clicked_infobar";
-
 // Brave stats prefs.
 inline constexpr char kEnabledForLastProfile[] =
     "brave.brave_ads.enabled_last_profile";
@@ -80,6 +79,11 @@ inline constexpr char kEverEnabledForAnyProfile[] =
 // P2A prefs.
 inline constexpr char kP2APrefPathPrefix[] = "brave.weekly_storage.";
 
+// We don't prefix `brave.brave_ads.` to this pref path because we don't want
+// them cleared if the user clears ads data or resets Brave Rewards.
+inline constexpr char kShouldShowSearchResultAdClickedInfoBar[] =
+    "brave.search.search_result_ad.should_show_clicked_infobar";
+
 }  // namespace brave_ads::prefs
 
 #endif  // BRAVE_COMPONENTS_BRAVE_ADS_CORE_PUBLIC_PREFS_PREF_NAMES_H_