From ce15763cc47adc14a5d7a13cc88301a5694c244b Mon Sep 17 00:00:00 2001 From: Jonathan Almeida Date: Fri, 29 May 2020 19:32:36 -0400 Subject: [PATCH] Use new AuotPushFeature.getSubscription API --- .../mozilla/fenix/push/WebPushEngineIntegration.kt | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/push/WebPushEngineIntegration.kt b/app/src/main/java/org/mozilla/fenix/push/WebPushEngineIntegration.kt index 7a7f2d512..c23bd13df 100644 --- a/app/src/main/java/org/mozilla/fenix/push/WebPushEngineIntegration.kt +++ b/app/src/main/java/org/mozilla/fenix/push/WebPushEngineIntegration.kt @@ -59,15 +59,9 @@ internal class WebPushEngineDelegate( override fun onGetSubscription(scope: String, onSubscription: (WebPushSubscription?) -> Unit) { // We don't have the appServerKey unless an app is creating a new subscription so we // allow the key to be null since it won't be overridden from a previous subscription. - pushFeature.subscribe( - scope = scope, - onSubscribeError = { - logger.error("Error on push onGetSubscription.") - onSubscription(null) - }, - onSubscribe = { subscription -> - onSubscription(subscription.toEnginePushSubscription()) - }) + pushFeature.getSubscription(scope) { + onSubscription(it?.toEnginePushSubscription()) + } } override fun onSubscribe(