Skip to content

Commit 4fa1fcf

Browse files
authored
Merge pull request #11084 from brave/pr11063_bsc-reenable-p3a-android_1.32.x
Re-enable P3A and remove the backfill (uplift to 1.32.x)
2 parents 37792e4 + 10d84b9 commit 4fa1fcf

File tree

3 files changed

+0
-24
lines changed

3 files changed

+0
-24
lines changed

android/java/org/chromium/chrome/browser/app/BraveActivity.java

-6
Original file line numberDiff line numberDiff line change
@@ -475,12 +475,6 @@ public void finishNativeInitialization() {
475475
BraveSyncInformers.show();
476476
BraveAndroidSyncDisabledInformer.showInformers();
477477

478-
if (!PackageUtils.isFirstInstall(this)
479-
&& !OnboardingPrefManager.getInstance().isP3AEnabledForExistingUsers()) {
480-
BravePrefServiceBridge.getInstance().setP3AEnabled(true);
481-
OnboardingPrefManager.getInstance().setP3AEnabledForExistingUsers(true);
482-
}
483-
484478
if (!OnboardingPrefManager.getInstance().isOneTimeNotificationStarted()
485479
&& PackageUtils.isFirstInstall(this)) {
486480
RetentionNotificationUtil.scheduleNotification(this, RetentionNotificationUtil.HOUR_3);

android/java/org/chromium/chrome/browser/onboarding/OnboardingPrefManager.java

-12
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,6 @@
3434
public class OnboardingPrefManager {
3535
private static final String PREF_ONBOARDING = "onboarding";
3636
private static final String PREF_P3A_ONBOARDING = "p3a_onboarding";
37-
private static final String PREF_P3A_ENABLED_FOR_EXISTING_USERS =
38-
"p3a_enabled_for_existing_users";
3937
private static final String PREF_CROSS_PROMO_MODAL = "cross_promo_modal";
4038
private static final String PREF_ONBOARDING_V2 = "onboarding_v2";
4139
private static final String PREF_NEXT_ONBOARDING_DATE = "next_onboarding_date";
@@ -109,16 +107,6 @@ public void setOnboardingShown(boolean isShown) {
109107
sharedPreferencesEditor.apply();
110108
}
111109

112-
public boolean isP3AEnabledForExistingUsers() {
113-
return mSharedPreferences.getBoolean(PREF_P3A_ENABLED_FOR_EXISTING_USERS, false);
114-
}
115-
116-
public void setP3AEnabledForExistingUsers(boolean isShown) {
117-
SharedPreferences.Editor sharedPreferencesEditor = mSharedPreferences.edit();
118-
sharedPreferencesEditor.putBoolean(PREF_P3A_ENABLED_FOR_EXISTING_USERS, isShown);
119-
sharedPreferencesEditor.apply();
120-
}
121-
122110
/**
123111
* Returns the user preference for whether the onboarding is shown.
124112
*/

components/p3a/brave_p3a_service.cc

-6
Original file line numberDiff line numberDiff line change
@@ -402,9 +402,6 @@ void BraveP3AService::StartScheduledUpload() {
402402

403403
// Only upload if service is enabled.
404404
bool p3a_enabled = local_state_->GetBoolean(brave::kP3AEnabled);
405-
#if defined(OS_ANDROID)
406-
p3a_enabled = false;
407-
#endif // OS_ANDROID
408405
if (p3a_enabled) {
409406
const std::string log = log_store_->staged_log();
410407
const std::string log_type = log_store_->staged_log_type();
@@ -417,9 +414,6 @@ void BraveP3AService::StartScheduledUpload() {
417414
void BraveP3AService::OnHistogramChanged(const char* histogram_name,
418415
uint64_t name_hash,
419416
base::HistogramBase::Sample sample) {
420-
#if defined(OS_ANDROID)
421-
return;
422-
#endif // OS_ANDROID
423417
std::unique_ptr<base::HistogramSamples> samples =
424418
base::StatisticsRecorder::FindHistogram(histogram_name)->SnapshotDelta();
425419

0 commit comments

Comments
 (0)