fix: prevent some actions impossible on desktop (#863)

Author: renefloor

dogfooding/lib/app/app_content.dart

@@ -118,7 +118,7 @@ class _StreamDogFoodingAppContentState
   }
 
   void _tryConsumingIncomingCallFromTerminatedState() {
-    if (CurrentPlatform.isIos) return;
+    if (!CurrentPlatform.isAndroid) return;
 
     if (_router.routerDelegate.navigatorKey.currentContext == null) {
       // App is not running yet. Postpone consuming after app is in the foreground
@@ -143,32 +143,36 @@ class _StreamDogFoodingAppContentState
   void _observeCallKitEvents() {
     final streamVideo = locator.get<StreamVideo>();
 
-    _compositeSubscription.add(
-      streamVideo.observeCoreCallKitEvents(
-        onCallAccepted: (callToJoin) {
-          // Navigate to the call screen.
-          final extra = (
-            call: callToJoin,
-            connectOptions: null,
-          );
-
-          _router.push(CallRoute($extra: extra).location, extra: extra);
-        },
-      ),
-    );
+    // On mobile we depend on call kit notifications.
+    // On desktop and web they are (currently) not available, so we depend on a
+    // websocket which can receive a call when the app is open.
+    if (CurrentPlatform.isMobile) {
+      _compositeSubscription.add(
+        streamVideo.observeCoreCallKitEvents(
+          onCallAccepted: (callToJoin) {
+            // Navigate to the call screen.
+            final extra = (
+              call: callToJoin,
+              connectOptions: null,
+            );
 
-    // UNCOMMENT THIS TO SHOW IN-APP INCOMING SCREEN
-    // _compositeSubscription.add(streamVideo.state.incomingCall.listen((call) {
-    //   if (call == null) return;
+            _router.push(CallRoute($extra: extra).location, extra: extra);
+          },
+        ),
+      );
+    } else {
+      _compositeSubscription.add(streamVideo.state.incomingCall.listen((call) {
+        if (call == null) return;
 
-    //   // Navigate to the call screen.
-    //   final extra = (
-    //     call: call,
-    //     connectOptions: null,
-    //   );
+        // Navigate to the call screen.
+        final extra = (
+          call: call,
+          connectOptions: null,
+        );
 
-    //   _router.push(CallRoute($extra: extra).location, extra: extra);
-    // }));
+        _router.push(CallRoute($extra: extra).location, extra: extra);
+      }));
+    }
   }
 
   _observeFcmMessages() {

dogfooding/lib/screens/home_screen.dart

@@ -40,13 +40,15 @@ class _HomeScreenState extends State<HomeScreen> {
 
   @override
   void initState() {
-    [
-      Permission.notification,
-      Permission.camera,
-      Permission.microphone,
-    ].request();
+    if (CurrentPlatform.isMobile || CurrentPlatform.isWeb) {
+      [
+        Permission.notification,
+        Permission.camera,
+        Permission.microphone,
+      ].request();
 
-    StreamVideoPushNotificationManager.ensureFullScreenIntentPermission();
+      StreamVideoPushNotificationManager.ensureFullScreenIntentPermission();
+    }
 
     StreamBackgroundService.init(
       StreamVideo.instance,

packages/stream_video_push_notification/lib/src/stream_video_push_notification.dart

@@ -67,7 +67,7 @@ class StreamVideoPushNotificationManager implements PushNotificationManager {
     this.callerCustomizationCallback,
     this.registerApnDeviceToken = false,
   }) : _client = client {
-    if (CurrentPlatform.isWeb) return;
+    if (!CurrentPlatform.isMobile) return;
 
     SharedPreferences.getInstance().then((prefs) => _sharedPreferences = prefs);
 
@@ -363,7 +363,7 @@ class StreamVideoPushNotificationManager implements PushNotificationManager {
 
   @override
   Future<List<CallData>> activeCalls() async {
-    if (CurrentPlatform.isWeb) return [];
+    if (!CurrentPlatform.isMobile) return [];
 
     final activeCalls = await FlutterCallkitIncoming.activeCalls();
     if (activeCalls is! List) return [];
@@ -395,7 +395,7 @@ class StreamVideoPushNotificationManager implements PushNotificationManager {
         .toList();
 
     // This is a workaround for the issue in flutter_callkit_incoming
-    // where second CallKit call overrids data in showCallkitIncoming native method
+    // where second CallKit call overrides data in showCallkitIncoming native method
     // and it's not possible to end the call by callCid
     if (activeCalls.length == calls.length) {
       await endAllCalls();