Commit 68b56ff2 authored by Roger Yang's avatar Roger Yang Committed by mergify[bot]
Browse files

Issue #19846: Make sure default browser notification only happens once

parent 317cbe9c

app/src/main/java/org/mozilla/fenix/home/intent/DefaultBrowserIntentProcessor.kt

+0 −1
Original line number Diff line number Diff line
@@ -24,7 +24,6 @@ class DefaultBrowserIntentProcessor( 
    override fun process(intent: Intent, navController: NavController, out: Intent): Boolean {

        return if (isDefaultBrowserNotificationIntent(intent)) {

            activity.openSetDefaultBrowserOption()

            activity.settings().defaultBrowserNotificationDisplayed = true



            true

        } else {

app/src/main/java/org/mozilla/fenix/onboarding/DefaultBrowserNotificationWorker.kt

+4 −0
Original line number Diff line number Diff line
@@ -35,6 +35,10 @@ class DefaultBrowserNotificationWorker( 
        ensureChannelExists()

        NotificationManagerCompat.from(applicationContext)

            .notify(NOTIFICATION_TAG, NOTIFICATION_ID, buildNotification())



        // default browser notification should only happen once

        applicationContext.settings().defaultBrowserNotificationDisplayed = true



        return Result.success()

    }