clippy: Fix manual_map warnings (#31922)

components/script/dom/abstractworkerglobalscope.rs

@@ -109,10 +109,9 @@ pub fn run_worker_event_loop<T, WorkerMsg, Event>(
         + DomObject,
 {
     let scope = worker_scope.upcast::<WorkerGlobalScope>();
-    let devtools_port = match scope.from_devtools_sender() {
+    let devtools_port = scope
-        Some(_) => Some(scope.from_devtools_receiver()),
+        .from_devtools_sender()
-        None => None,
+        .map(|_| scope.from_devtools_receiver());
-    };
     let task_queue = worker_scope.task_queue();
     let event = select! {
         recv(worker_scope.control_receiver()) -> msg => worker_scope.from_control_msg(msg.unwrap()),

components/script/dom/element.rs

@@ -3204,11 +3204,9 @@ impl<'a> SelectorsElement for DomRoot<Element> {
     }
 
     fn containing_shadow_host(&self) -> Option<Self> {
-        if let Some(shadow_root) = self.upcast::<Node>().containing_shadow_root() {
+        self.upcast::<Node>()
-            Some(shadow_root.Host())
+            .containing_shadow_root()
-        } else {
+            .map(|shadow_root| shadow_root.Host())
-            None
-        }
     }
 
     fn is_pseudo_element(&self) -> bool {

components/script/dom/fakexrdevice.rs

@@ -84,16 +84,14 @@ pub fn view<Eye>(view: &FakeXRViewInit) -> Fallible<MockViewInit<Eye>> {
     };
     let viewport = Rect::new(origin, size);
 
-    let fov = if let Some(ref fov) = view.fieldOfView {
+    let fov = view.fieldOfView.as_ref().map(|fov| {
-        Some((
+        (
             fov.leftDegrees.to_radians(),
             fov.rightDegrees.to_radians(),
             fov.upDegrees.to_radians(),
             fov.downDegrees.to_radians(),
-        ))
+        )
-    } else {
+    });
-        None
-    };
 
     Ok(MockViewInit {
         projection,

components/script/dom/htmlinputelement.rs

@@ -1184,10 +1184,7 @@ impl HTMLInputElementMethods for HTMLInputElement {
 
     // https://html.spec.whatwg.org/multipage/#dom-input-files
     fn GetFiles(&self) -> Option<DomRoot<FileList>> {
-        match self.filelist.get() {
+        self.filelist.get().as_ref().map(|fl| fl.clone())
-            Some(ref fl) => Some(fl.clone()),
-            None => None,
-        }
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-input-defaultchecked

components/script/dom/htmlmediaelement.rs

@@ -1941,10 +1941,12 @@ impl HTMLMediaElement {
     }
 
     pub fn get_current_frame(&self) -> Option<VideoFrame> {
-        match self.video_renderer.lock().unwrap().current_frame_holder {
+        self.video_renderer
-            Some(ref holder) => Some(holder.get_frame()),
+            .lock()
-            None => None,
+            .unwrap()
-        }
+            .current_frame_holder
+            .as_ref()
+            .map(|holder| holder.get_frame())
     }
 
     /// By default the audio is rendered through the audio sink automatically
@@ -2088,15 +2090,14 @@ impl HTMLMediaElementMethods for HTMLMediaElement {
 
     // https://html.spec.whatwg.org/multipage/#dom-media-srcobject
     fn GetSrcObject(&self) -> Option<MediaStreamOrBlob> {
-        match *self.src_object.borrow() {
+        (*self.src_object.borrow())
-            Some(ref src_object) => Some(match src_object {
+            .as_ref()
+            .map(|src_object| match src_object {
                 SrcObject::Blob(blob) => MediaStreamOrBlob::Blob(DomRoot::from_ref(blob)),
                 SrcObject::MediaStream(stream) => {
                     MediaStreamOrBlob::MediaStream(DomRoot::from_ref(stream))
                 },
-            }),
+            })
-            None => None,
-        }
     }
 
     // https://html.spec.whatwg.org/multipage/#dom-media-srcobject
@@ -2659,10 +2660,10 @@ impl FetchResponseListener for HTMLMediaElementFetchListener {
                 let content_length =
                     if let Some(content_range) = headers.typed_get::<ContentRange>() {
                         content_range.bytes_len()
-                    } else if let Some(content_length) = headers.typed_get::<ContentLength>() {
-                        Some(content_length.0)
                     } else {
-                        None
+                        headers
+                            .typed_get::<ContentLength>()
+                            .map(|content_length| content_length.0)
                     };
 
                 // We only set the expected input size if it changes.

components/script/dom/mediadevices.rs

@@ -130,11 +130,8 @@ fn convert_culong(js: &ConstrainULong) -> Option<Constrain<u32>> {
                     max: range.parent.max,
                     ideal: range.ideal,
                 }))
-            } else if let Some(exact) = range.exact {
-                Some(Constrain::Value(exact))
             } else {
-                // the unspecified case is treated as all three being none
+                range.exact.map(Constrain::Value)
-                None
             }
         },
     }
@@ -150,11 +147,8 @@ fn convert_cdouble(js: &ConstrainDouble) -> Option<Constrain<f64>> {
                     max: range.parent.max.map(|x| *x),
                     ideal: range.ideal.map(|x| *x),
                 }))
-            } else if let Some(exact) = range.exact {
-                Some(Constrain::Value(*exact))
             } else {
-                // the unspecified case is treated as all three being none
+                range.exact.map(|exact| Constrain::Value(*exact))
-                None
             }
         },
     }

components/script/dom/xrinputsource.rs

@@ -122,13 +122,9 @@ impl XRInputSourceMethods for XRInputSource {
 
     // https://github.com/immersive-web/webxr-hands-input/blob/master/explainer.md
     fn GetHand(&self) -> Option<DomRoot<XRHand>> {
-        if let Some(ref hand) = self.info.hand_support {
+        self.info.hand_support.as_ref().map(|hand| {
-            Some(
+            self.hand
-                self.hand
+                .or_init(|| XRHand::new(&self.global(), self, hand.clone()))
-                    .or_init(|| XRHand::new(&self.global(), self, hand.clone())),
+        })
-            )
-        } else {
-            None
-        }
     }
 }

components/script/script_thread.rs

@@ -3352,10 +3352,10 @@ impl ScriptThread {
             _ => IsHTMLDocument::HTMLDocument,
         };
 
-        let referrer = match metadata.referrer {
+        let referrer = metadata
-            Some(ref referrer) => Some(referrer.clone().into_string()),
+            .referrer
-            None => None,
+            .as_ref()
-        };
+            .map(|referrer| referrer.clone().into_string());
 
         let referrer_policy = metadata
             .headers