Fix firefox screenshare

CHANGELOG.md

@@ -6,6 +6,20 @@ All user visible changes to this project will be documented in this file. This p
 
 
 
+## [0.4.1] · 2023-??-??
+[0.4.1]: /../../tree/medea-jason-0.4.1
+
+[Diff](https://github.com/instrumentisto/medea-jason/compare/medea-jason-0.4.0...medea-jason-0.4.1)
+
+### Fixed
+
+- Screen sharing in Firefox ([#135]).
+
+[#135]: /../../pull/135
+
+
+
+
 ## [0.4.0] · 2023-07-11
 [0.4.0]: /../../tree/medea-jason-0.4.0
 

src/media/constraints.rs

@@ -997,10 +997,7 @@ impl AudioTrackConstraints {
     ) -> bool {
         let track = track.as_ref();
         satisfies_track(track, MediaKind::Audio).await
-            && ConstrainString::satisfies(
-                &self.device_id,
-                &Some(track.device_id()),
-            )
+            && ConstrainString::satisfies(&self.device_id, &track.device_id())
         // TODO returns Result<bool, Error>
     }
 
@@ -1213,10 +1210,7 @@ impl DeviceVideoTrackConstraints {
     ) -> bool {
         let track = track.as_ref();
         satisfies_track(track, MediaKind::Video).await
-            && ConstrainString::satisfies(
-                &self.device_id,
-                &Some(track.device_id()),
-            )
+            && ConstrainString::satisfies(&self.device_id, &track.device_id())
             && ConstrainString::satisfies(
                 &self.facing_mode,
                 &track.facing_mode(),
@@ -1301,10 +1295,7 @@ impl DisplayVideoTrackConstraints {
     ) -> bool {
         let track = track.as_ref();
         satisfies_track(track, MediaKind::Video).await
-            && ConstrainString::satisfies(
-                &self.device_id,
-                &Some(track.device_id()),
-            )
+            && ConstrainString::satisfies(&self.device_id, &track.device_id())
             && ConstrainU32::satisfies(self.height, track.height())
             && ConstrainU32::satisfies(self.width, track.width())
             && track.guess_is_from_display()

src/platform/dart/media_track.rs

@@ -163,10 +163,10 @@ impl MediaStreamTrack {
     /// [1]: https://w3.org/TR/mediacapture-streams#dfn-deviceid
     #[inline]
     #[must_use]
-    pub fn device_id(&self) -> String {
+    pub fn device_id(&self) -> Option<String> {
         let device_id =
             unsafe { media_stream_track::device_id(self.inner.get()) };
-        unsafe { dart_string_into_rust(device_id) }
+        Some(unsafe { dart_string_into_rust(device_id) })
     }
 
     /// Returns [kind][1] of this [`MediaStreamTrack`].

src/platform/wasm/media_track.rs

@@ -116,12 +116,11 @@ impl MediaStreamTrack {
     /// [1]: https://tinyurl.com/w3-streams#dom-mediatracksettings-deviceid
     /// [2]: https://w3.org/TR/mediacapture-streams#mediastreamtrack
     #[must_use]
-    pub fn device_id(&self) -> String {
+    pub fn device_id(&self) -> Option<String> {
         #[allow(clippy::unwrap_used)]
         get_property_by_name(&self.sys_track.get_settings(), "deviceId", |v| {
             v.as_string()
         })
-        .unwrap()
     }
 
     /// Return a [`facingMode`][1] of the underlying [MediaStreamTrack][2].