Archives/iceraven-browser
2
0
Fork 0
mirror of https://github.com/fork-maintainers/iceraven-browser synced 2024-11-19 09:25:34 +00:00
Code Issues Releases Wiki Activity

[fenix] For https://github.com/mozilla-mobile/fenix/issues/5334: disable notification for private custom tabs

Browse Source
This commit is contained in:
Severin Rudie 2019-12-27 15:51:03 -08:00 committed by Mihai Adrian
parent 429ad5d0d6
commit 5d33e79c59
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/src/main/java/org/mozilla/fenix/session/NotificationSessionObserver.kt
View File

@ -32,7 +32,9 @@ class NotificationSessionObserver(
}
override fun onSessionAdded(session: Session) {
if (session.private) {
// Custom tabs are meant to feel like part of the app that opened them, not Fenix, so we
// don't need to show a 'close tab' notification for them
if (session.private && !session.isCustomTabSession()) {
SessionNotificationService.start(context)
}
}

9
app/src/main/java/org/mozilla/fenix/session/SessionNotificationService.kt
View File

@ -26,7 +26,14 @@ import org.mozilla.fenix.ext.metrics
import org.mozilla.fenix.ext.sessionsOfType
/**
* As long as a session is active this service will keep the notification (and our process) alive.
* Manages notifications for private tabs.
*
* Private tab notifications solve two problems for us:
* 1 - They allow users to interact with us from outside of the app (example: by closing all
* private tabs).
* 2 - The notification will keep our process alive, allowing us to keep private tabs in memory.
*
* As long as a session is active this service will keep its notification alive.
*/
class SessionNotificationService : Service() {