aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/dom
diff options
context:
space:
mode:
Diffstat (limited to 'components/script/dom')
-rw-r--r--components/script/dom/dedicatedworkerglobalscope.rs2
-rw-r--r--components/script/dom/element.rs2
-rw-r--r--components/script/dom/filereader.rs4
-rw-r--r--components/script/dom/fontface.rs20
-rw-r--r--components/script/dom/formdata.rs14
-rw-r--r--components/script/dom/globalscope.rs38
-rw-r--r--components/script/dom/htmlmediaelement.rs8
-rw-r--r--components/script/dom/htmlselectelement.rs4
-rw-r--r--components/script/dom/mediadevices.rs20
-rw-r--r--components/script/dom/node.rs24
-rw-r--r--components/script/dom/readablestream.rs64
-rw-r--r--components/script/dom/servoparser/html.rs2
-rw-r--r--components/script/dom/servoparser/mod.rs2
-rw-r--r--components/script/dom/webglframebuffer.rs12
-rw-r--r--components/script/dom/webgpu/gpudevice.rs2
-rw-r--r--components/script/dom/window.rs6
16 files changed, 113 insertions, 111 deletions
diff --git a/components/script/dom/dedicatedworkerglobalscope.rs b/components/script/dom/dedicatedworkerglobalscope.rs
index ef8e59fc70e..605bc3dcde9 100644
--- a/components/script/dom/dedicatedworkerglobalscope.rs
+++ b/components/script/dom/dedicatedworkerglobalscope.rs
@@ -114,7 +114,7 @@ impl QueuedTaskConversion for DedicatedWorkerScriptMsg {
_ => return None,
};
let script_msg = match common_worker_msg {
- WorkerScriptMsg::Common(ref script_msg) => script_msg,
+ WorkerScriptMsg::Common(script_msg) => script_msg,
_ => return None,
};
match script_msg {
diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs
index 35ccd0d89e2..16bdd5a213f 100644
--- a/components/script/dom/element.rs
+++ b/components/script/dom/element.rs
@@ -1618,7 +1618,7 @@ impl Element {
if *name == local_name!("id") || *name == local_name!("name") {
match maybe_attr {
None => true,
- Some(ref attr) => matches!(*attr.value(), AttrValue::Atom(_)),
+ Some(attr) => matches!(*attr.value(), AttrValue::Atom(_)),
}
} else {
true
diff --git a/components/script/dom/filereader.rs b/components/script/dom/filereader.rs
index b4e37e2cc2d..2f6e1796ab7 100644
--- a/components/script/dom/filereader.rs
+++ b/components/script/dom/filereader.rs
@@ -149,8 +149,8 @@ impl FileReaderSharedFunctionality {
let resultmime = blob_type.parse::<Mime>().ok();
resultmime.and_then(|mime| {
mime.params()
- .find(|(ref k, _)| &mime::CHARSET == k)
- .and_then(|(_, ref v)| Encoding::for_label(v.as_ref().as_bytes()))
+ .find(|(k, _)| &mime::CHARSET == k)
+ .and_then(|(_, v)| Encoding::for_label(v.as_ref().as_bytes()))
})
});
diff --git a/components/script/dom/fontface.rs b/components/script/dom/fontface.rs
index c21f3a86a08..89156cca3a8 100644
--- a/components/script/dom/fontface.rs
+++ b/components/script/dom/fontface.rs
@@ -95,16 +95,16 @@ fn parse_font_face_descriptors(
);
let FontFaceDescriptors {
- ref ascentOverride,
- ref descentOverride,
- ref display,
- ref featureSettings,
- ref lineGapOverride,
- ref stretch,
- ref style,
- ref unicodeRange,
- ref variationSettings,
- ref weight,
+ ascentOverride,
+ descentOverride,
+ display,
+ featureSettings,
+ lineGapOverride,
+ stretch,
+ style,
+ unicodeRange,
+ variationSettings,
+ weight,
} = input_descriptors;
let _ = variationSettings; // TODO: Stylo doesn't parse font-variation-settings yet.
diff --git a/components/script/dom/formdata.rs b/components/script/dom/formdata.rs
index 2a3a52df8eb..6c8c7e057a9 100644
--- a/components/script/dom/formdata.rs
+++ b/components/script/dom/formdata.rs
@@ -181,10 +181,8 @@ impl FormDataMethods<crate::DomTypeHolder> for FormData {
.iter()
.find(|(datum_name, _)| datum_name.0 == name.0)
.map(|(_, datum)| match &datum.value {
- FormDatumValue::String(ref s) => {
- FileOrUSVString::USVString(USVString(s.to_string()))
- },
- FormDatumValue::File(ref b) => FileOrUSVString::File(DomRoot::from_ref(b)),
+ FormDatumValue::String(s) => FileOrUSVString::USVString(USVString(s.to_string())),
+ FormDatumValue::File(b) => FileOrUSVString::File(DomRoot::from_ref(b)),
})
}
@@ -199,10 +197,10 @@ impl FormDataMethods<crate::DomTypeHolder> for FormData {
}
Some(match &datum.value {
- FormDatumValue::String(ref s) => {
+ FormDatumValue::String(s) => {
FileOrUSVString::USVString(USVString(s.to_string()))
},
- FormDatumValue::File(ref b) => FileOrUSVString::File(DomRoot::from_ref(b)),
+ FormDatumValue::File(b) => FileOrUSVString::File(DomRoot::from_ref(b)),
})
})
.collect()
@@ -308,8 +306,8 @@ impl Iterable for FormData {
let data = self.data.borrow();
let datum = &data.get(n as usize).unwrap().1;
match &datum.value {
- FormDatumValue::String(ref s) => FileOrUSVString::USVString(USVString(s.to_string())),
- FormDatumValue::File(ref b) => FileOrUSVString::File(DomRoot::from_ref(b)),
+ FormDatumValue::String(s) => FileOrUSVString::USVString(USVString(s.to_string())),
+ FormDatumValue::File(b) => FileOrUSVString::File(DomRoot::from_ref(b)),
}
}
diff --git a/components/script/dom/globalscope.rs b/components/script/dom/globalscope.rs
index ceeca775f53..bea3e1c3357 100644
--- a/components/script/dom/globalscope.rs
+++ b/components/script/dom/globalscope.rs
@@ -1338,7 +1338,7 @@ impl GlobalScope {
/// TODO: Also remove them if they do not have an event-listener.
/// see <https://github.com/servo/servo/issues/25772>
pub(crate) fn perform_a_broadcast_channel_garbage_collection_checkpoint(&self) {
- let is_empty = if let BroadcastChannelState::Managed(router_id, ref mut channels) =
+ let is_empty = if let BroadcastChannelState::Managed(router_id, channels) =
&mut *self.broadcast_channel_state.borrow_mut()
{
channels.retain(|name, ref mut channels| {
@@ -1552,7 +1552,7 @@ impl GlobalScope {
BlobTracker::Blob(weak) => weak.root().is_none(),
};
if garbage_collected && !blob_info.has_url {
- if let BlobData::File(ref f) = blob_info.blob_impl.blob_data() {
+ if let BlobData::File(f) = blob_info.blob_impl.blob_data() {
self.decrement_file_ref(f.get_id());
}
false
@@ -1569,7 +1569,7 @@ impl GlobalScope {
.borrow_mut()
.drain()
.for_each(|(_id, blob_info)| {
- if let BlobData::File(ref f) = blob_info.blob_impl.blob_data() {
+ if let BlobData::File(f) = blob_info.blob_impl.blob_data() {
self.decrement_file_ref(f.get_id());
}
});
@@ -1594,7 +1594,7 @@ impl GlobalScope {
.get(blob_id)
.expect("get_blob_bytes for an unknown blob.");
match blob_info.blob_impl.blob_data() {
- BlobData::Sliced(ref parent, ref rel_pos) => Some((*parent, rel_pos.clone())),
+ BlobData::Sliced(parent, rel_pos) => Some((*parent, rel_pos.clone())),
_ => None,
}
};
@@ -1615,7 +1615,7 @@ impl GlobalScope {
.get(blob_id)
.expect("get_blob_bytes_non_sliced called for a unknown blob.");
match blob_info.blob_impl.blob_data() {
- BlobData::File(ref f) => {
+ BlobData::File(f) => {
let (buffer, is_new_buffer) = match f.get_cache() {
Some(bytes) => (bytes, false),
None => {
@@ -1631,7 +1631,7 @@ impl GlobalScope {
Ok(buffer)
},
- BlobData::Memory(ref s) => Ok(s.clone()),
+ BlobData::Memory(s) => Ok(s.clone()),
BlobData::Sliced(_, _) => panic!("This blob doesn't have a parent."),
}
}
@@ -1649,7 +1649,7 @@ impl GlobalScope {
.get(blob_id)
.expect("get_blob_bytes_or_file_id for an unknown blob.");
match blob_info.blob_impl.blob_data() {
- BlobData::Sliced(ref parent, ref rel_pos) => Some((*parent, rel_pos.clone())),
+ BlobData::Sliced(parent, rel_pos) => Some((*parent, rel_pos.clone())),
_ => None,
}
};
@@ -1679,11 +1679,11 @@ impl GlobalScope {
.get(blob_id)
.expect("get_blob_bytes_non_sliced_or_file_id called for a unknown blob.");
match blob_info.blob_impl.blob_data() {
- BlobData::File(ref f) => match f.get_cache() {
+ BlobData::File(f) => match f.get_cache() {
Some(bytes) => BlobResult::Bytes(bytes.clone()),
None => BlobResult::File(f.get_id(), f.get_size() as usize),
},
- BlobData::Memory(ref s) => BlobResult::Bytes(s.clone()),
+ BlobData::Memory(s) => BlobResult::Bytes(s.clone()),
BlobData::Sliced(_, _) => panic!("This blob doesn't have a parent."),
}
}
@@ -1705,7 +1705,7 @@ impl GlobalScope {
.get(blob_id)
.expect("get_blob_size called for a unknown blob.");
match blob_info.blob_impl.blob_data() {
- BlobData::Sliced(ref parent, ref rel_pos) => Some((*parent, rel_pos.clone())),
+ BlobData::Sliced(parent, rel_pos) => Some((*parent, rel_pos.clone())),
_ => None,
}
};
@@ -1715,8 +1715,8 @@ impl GlobalScope {
.get(&parent_id)
.expect("Parent of blob whose size is unknown.");
let parent_size = match parent_info.blob_impl.blob_data() {
- BlobData::File(ref f) => f.get_size(),
- BlobData::Memory(ref v) => v.len() as u64,
+ BlobData::File(f) => f.get_size(),
+ BlobData::Memory(v) => v.len() as u64,
BlobData::Sliced(_, _) => panic!("Blob ancestry should be only one level."),
};
rel_pos.to_abs_range(parent_size as usize).len() as u64
@@ -1726,8 +1726,8 @@ impl GlobalScope {
.get(blob_id)
.expect("Blob whose size is unknown.");
match blob_info.blob_impl.blob_data() {
- BlobData::File(ref f) => f.get_size(),
- BlobData::Memory(ref v) => v.len() as u64,
+ BlobData::File(f) => f.get_size(),
+ BlobData::Memory(v) => v.len() as u64,
BlobData::Sliced(_, _) => {
panic!("It was previously checked that this blob does not have a parent.")
},
@@ -1747,7 +1747,7 @@ impl GlobalScope {
blob_info.has_url = true;
match blob_info.blob_impl.blob_data() {
- BlobData::Sliced(ref parent, ref rel_pos) => Some((*parent, rel_pos.clone())),
+ BlobData::Sliced(parent, rel_pos) => Some((*parent, rel_pos.clone())),
_ => None,
}
};
@@ -1758,8 +1758,8 @@ impl GlobalScope {
.expect("Parent of blob whose url is requested is unknown.");
let parent_file_id = self.promote(parent_info, /* set_valid is */ false);
let parent_size = match parent_info.blob_impl.blob_data() {
- BlobData::File(ref f) => f.get_size(),
- BlobData::Memory(ref v) => v.len() as u64,
+ BlobData::File(f) => f.get_size(),
+ BlobData::Memory(v) => v.len() as u64,
BlobData::Sliced(_, _) => panic!("Blob ancestry should be only one level."),
};
let parent_size = rel_pos.to_abs_range(parent_size as usize).len() as u64;
@@ -1827,7 +1827,7 @@ impl GlobalScope {
BlobData::Sliced(_, _) => {
panic!("Sliced blobs should use create_sliced_url_id instead of promote.");
},
- BlobData::File(ref f) => {
+ BlobData::File(f) => {
if set_valid {
let origin = get_blob_origin(&global_url);
let (tx, rx) = profile_ipc::channel(self.time_profiler_chan().clone()).unwrap();
@@ -1845,7 +1845,7 @@ impl GlobalScope {
return f.get_id();
}
},
- BlobData::Memory(ref mut bytes_in) => mem::swap(bytes_in, &mut bytes),
+ BlobData::Memory(bytes_in) => mem::swap(bytes_in, &mut bytes),
};
let origin = get_blob_origin(&global_url);
diff --git a/components/script/dom/htmlmediaelement.rs b/components/script/dom/htmlmediaelement.rs
index f9520ab1fdc..c68d225c6a3 100644
--- a/components/script/dom/htmlmediaelement.rs
+++ b/components/script/dom/htmlmediaelement.rs
@@ -213,7 +213,7 @@ impl VideoFrameRenderer for MediaFrameRenderer {
);
match &mut self.current_frame {
- Some(ref mut current_frame)
+ Some(current_frame)
if current_frame.width == frame.get_width() &&
current_frame.height == frame.get_height() =>
{
@@ -233,7 +233,7 @@ impl VideoFrameRenderer for MediaFrameRenderer {
updates.push(ImageUpdate::DeleteImage(old_image_key));
}
},
- Some(ref mut current_frame) => {
+ Some(current_frame) => {
self.old_frame = Some(current_frame.image_key);
let Some(new_image_key) = self.compositor_api.generate_image_key() else {
@@ -971,7 +971,7 @@ impl HTMLMediaElement {
Some(ServoUrl::parse(&blob_url).expect("infallible"));
self.fetch_request(None, None);
},
- SrcObject::MediaStream(ref stream) => {
+ SrcObject::MediaStream(stream) => {
let tracks = &*stream.get_tracks();
for (pos, track) in tracks.iter().enumerate() {
if self
@@ -1788,7 +1788,7 @@ impl HTMLMediaElement {
// fetching where we left.
if let Some(ref current_fetch_context) = *self.current_fetch_context.borrow() {
match current_fetch_context.cancel_reason() {
- Some(ref reason) if *reason == CancelReason::Backoff => {
+ Some(reason) if *reason == CancelReason::Backoff => {
// XXX(ferjm) Ideally we should just create a fetch request from
// where we left. But keeping track of the exact next byte that the
// media backend expects is not the easiest task, so I'm simply
diff --git a/components/script/dom/htmlselectelement.rs b/components/script/dom/htmlselectelement.rs
index 8f8a9d15601..445b1058018 100644
--- a/components/script/dom/htmlselectelement.rs
+++ b/components/script/dom/htmlselectelement.rs
@@ -114,7 +114,9 @@ impl HTMLSelectElement {
}
// https://html.spec.whatwg.org/multipage/#concept-select-option-list
- pub(crate) fn list_of_options(&self) -> impl Iterator<Item = DomRoot<HTMLOptionElement>> {
+ pub(crate) fn list_of_options(
+ &self,
+ ) -> impl Iterator<Item = DomRoot<HTMLOptionElement>> + use<'_> {
self.upcast::<Node>().children().flat_map(|node| {
if node.is::<HTMLOptionElement>() {
let node = DomRoot::downcast::<HTMLOptionElement>(node).unwrap();
diff --git a/components/script/dom/mediadevices.rs b/components/script/dom/mediadevices.rs
index 2ae2ebaca10..f2f5a45d09c 100644
--- a/components/script/dom/mediadevices.rs
+++ b/components/script/dom/mediadevices.rs
@@ -118,22 +118,20 @@ fn convert_constraints(js: &BooleanOrMediaTrackConstraints) -> Option<MediaTrack
match js {
BooleanOrMediaTrackConstraints::Boolean(false) => None,
BooleanOrMediaTrackConstraints::Boolean(true) => Some(Default::default()),
- BooleanOrMediaTrackConstraints::MediaTrackConstraints(ref c) => {
- Some(MediaTrackConstraintSet {
- height: c.parent.height.as_ref().and_then(convert_culong),
- width: c.parent.width.as_ref().and_then(convert_culong),
- aspect: c.parent.aspectRatio.as_ref().and_then(convert_cdouble),
- frame_rate: c.parent.frameRate.as_ref().and_then(convert_cdouble),
- sample_rate: c.parent.sampleRate.as_ref().and_then(convert_culong),
- })
- },
+ BooleanOrMediaTrackConstraints::MediaTrackConstraints(c) => Some(MediaTrackConstraintSet {
+ height: c.parent.height.as_ref().and_then(convert_culong),
+ width: c.parent.width.as_ref().and_then(convert_culong),
+ aspect: c.parent.aspectRatio.as_ref().and_then(convert_cdouble),
+ frame_rate: c.parent.frameRate.as_ref().and_then(convert_cdouble),
+ sample_rate: c.parent.sampleRate.as_ref().and_then(convert_culong),
+ }),
}
}
fn convert_culong(js: &ConstrainULong) -> Option<Constrain<u32>> {
match js {
ConstrainULong::ClampedUnsignedLong(val) => Some(Constrain::Value(*val)),
- ConstrainULong::ConstrainULongRange(ref range) => {
+ ConstrainULong::ConstrainULongRange(range) => {
if range.parent.min.is_some() || range.parent.max.is_some() {
Some(Constrain::Range(ConstrainRange {
min: range.parent.min,
@@ -150,7 +148,7 @@ fn convert_culong(js: &ConstrainULong) -> Option<Constrain<u32>> {
fn convert_cdouble(js: &ConstrainDouble) -> Option<Constrain<f64>> {
match js {
ConstrainDouble::Double(val) => Some(Constrain::Value(**val)),
- ConstrainDouble::ConstrainDoubleRange(ref range) => {
+ ConstrainDouble::ConstrainDoubleRange(range) => {
if range.parent.min.is_some() || range.parent.max.is_some() {
Some(Constrain::Range(ConstrainRange {
min: range.parent.min.map(|x| *x),
diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs
index 2837e25c08a..d248028cc2b 100644
--- a/components/script/dom/node.rs
+++ b/components/script/dom/node.rs
@@ -752,14 +752,18 @@ impl Node {
TreeIterator::new(self, shadow_including)
}
- pub(crate) fn inclusively_following_siblings(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn inclusively_following_siblings(
+ &self,
+ ) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: Some(DomRoot::from_ref(self)),
next_node: |n| n.GetNextSibling(),
}
}
- pub(crate) fn inclusively_preceding_siblings(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn inclusively_preceding_siblings(
+ &self,
+ ) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: Some(DomRoot::from_ref(self)),
next_node: |n| n.GetPreviousSibling(),
@@ -799,14 +803,14 @@ impl Node {
.any(|ancestor| &*ancestor == self)
}
- pub(crate) fn following_siblings(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn following_siblings(&self) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: self.GetNextSibling(),
next_node: |n| n.GetNextSibling(),
}
}
- pub(crate) fn preceding_siblings(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn preceding_siblings(&self) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: self.GetPreviousSibling(),
next_node: |n| n.GetPreviousSibling(),
@@ -827,7 +831,7 @@ impl Node {
}
}
- pub(crate) fn descending_last_children(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn descending_last_children(&self) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: self.GetLastChild(),
next_node: |n| n.GetLastChild(),
@@ -1090,7 +1094,7 @@ impl Node {
Ok(NodeList::new_simple_list(&window, iter, CanGc::note()))
}
- pub(crate) fn ancestors(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn ancestors(&self) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: self.GetParentNode(),
next_node: |n| n.GetParentNode(),
@@ -1101,7 +1105,7 @@ impl Node {
pub(crate) fn inclusive_ancestors(
&self,
shadow_including: ShadowIncluding,
- ) -> impl Iterator<Item = DomRoot<Node>> {
+ ) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: Some(DomRoot::from_ref(self)),
next_node: move |n| {
@@ -1143,21 +1147,21 @@ impl Node {
self.is_connected() && self.owner_doc().browsing_context().is_some()
}
- pub(crate) fn children(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn children(&self) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: self.GetFirstChild(),
next_node: |n| n.GetNextSibling(),
}
}
- pub(crate) fn rev_children(&self) -> impl Iterator<Item = DomRoot<Node>> {
+ pub(crate) fn rev_children(&self) -> impl Iterator<Item = DomRoot<Node>> + use<> {
SimpleNodeIterator {
current: self.GetLastChild(),
next_node: |n| n.GetPreviousSibling(),
}
}
- pub(crate) fn child_elements(&self) -> impl Iterator<Item = DomRoot<Element>> {
+ pub(crate) fn child_elements(&self) -> impl Iterator<Item = DomRoot<Element>> + use<> {
self.children()
.filter_map(DomRoot::downcast as fn(_) -> _)
.peekable()
diff --git a/components/script/dom/readablestream.rs b/components/script/dom/readablestream.rs
index 4baccfdba90..52e12a9a248 100644
--- a/components/script/dom/readablestream.rs
+++ b/components/script/dom/readablestream.rs
@@ -281,13 +281,13 @@ impl ReadableStream {
/// Call into the release steps of the controller,
pub(crate) fn perform_release_steps(&self) -> Fallible<()> {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => {
+ Some(ControllerType::Default(controller)) => {
let controller = controller
.get()
.ok_or_else(|| Error::Type("Stream should have controller.".to_string()))?;
controller.perform_release_steps()
},
- Some(ControllerType::Byte(ref controller)) => {
+ Some(ControllerType::Byte(controller)) => {
let controller = controller
.get()
.ok_or_else(|| Error::Type("Stream should have controller.".to_string()))?;
@@ -307,11 +307,11 @@ impl ReadableStream {
can_gc: CanGc,
) {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => controller
+ Some(ControllerType::Default(controller)) => controller
.get()
.expect("Stream should have controller.")
.perform_pull_steps(read_request, can_gc),
- Some(ControllerType::Byte(ref controller)) => controller
+ Some(ControllerType::Byte(controller)) => controller
.get()
.expect("Stream should have controller.")
.perform_pull_steps(cx, read_request, can_gc),
@@ -333,7 +333,7 @@ impl ReadableStream {
can_gc: CanGc,
) {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Byte(ref controller)) => controller
+ Some(ControllerType::Byte(controller)) => controller
.get()
.expect("Stream should have controller.")
.perform_pull_into(cx, read_into_request, view, options, can_gc),
@@ -348,7 +348,7 @@ impl ReadableStream {
/// <https://streams.spec.whatwg.org/#readable-stream-add-read-request>
pub(crate) fn add_read_request(&self, read_request: &ReadRequest) {
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
let Some(reader) = reader.get() else {
panic!("Attempt to add a read request without having first acquired a reader.");
};
@@ -369,7 +369,7 @@ impl ReadableStream {
pub(crate) fn add_read_into_request(&self, read_request: &ReadIntoRequest) {
match self.reader.borrow().as_ref() {
// Assert: stream.[[reader]] implements ReadableStreamBYOBReader.
- Some(ReaderType::BYOB(ref reader)) => {
+ Some(ReaderType::BYOB(reader)) => {
let Some(reader) = reader.get() else {
unreachable!(
"Attempt to add a read into request without having first acquired a reader."
@@ -392,7 +392,7 @@ impl ReadableStream {
/// Note: in other use cases this call happens via the controller.
pub(crate) fn enqueue_native(&self, bytes: Vec<u8>, can_gc: CanGc) {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => controller
+ Some(ControllerType::Default(controller)) => controller
.get()
.expect("Stream should have controller.")
.enqueue_native(bytes, can_gc),
@@ -418,7 +418,7 @@ impl ReadableStream {
// Let reader be stream.[[reader]].
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
let Some(reader) = reader.get() else {
// If reader is undefined, return.
return;
@@ -427,7 +427,7 @@ impl ReadableStream {
// Perform ! ReadableStreamDefaultReaderErrorReadRequests(reader, e).
reader.error(e, can_gc);
},
- Some(ReaderType::BYOB(ref reader)) => {
+ Some(ReaderType::BYOB(reader)) => {
let Some(reader) = reader.get() else {
// If reader is undefined, return.
return;
@@ -460,7 +460,7 @@ impl ReadableStream {
/// <https://streams.spec.whatwg.org/#readable-stream-default-controller-close>
pub(crate) fn controller_close_native(&self, can_gc: CanGc) {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => {
+ Some(ControllerType::Default(controller)) => {
let _ = controller
.get()
.expect("Stream should have controller.")
@@ -476,7 +476,7 @@ impl ReadableStream {
/// Useful for native source integration only.
pub(crate) fn in_memory(&self) -> bool {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => controller
+ Some(ControllerType::Default(controller)) => controller
.get()
.expect("Stream should have controller.")
.in_memory(),
@@ -492,7 +492,7 @@ impl ReadableStream {
/// Useful for native source integration only.
pub(crate) fn get_in_memory_bytes(&self) -> Option<Vec<u8>> {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => controller
+ Some(ControllerType::Default(controller)) => controller
.get()
.expect("Stream should have controller.")
.get_in_memory_bytes(),
@@ -536,7 +536,7 @@ impl ReadableStream {
pub(crate) fn get_default_controller(&self) -> DomRoot<ReadableStreamDefaultController> {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => {
+ Some(ControllerType::Default(controller)) => {
controller.get().expect("Stream should have controller.")
},
_ => {
@@ -549,9 +549,7 @@ impl ReadableStream {
pub(crate) fn get_default_reader(&self) -> DomRoot<ReadableStreamDefaultReader> {
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
- reader.get().expect("Stream should have reader.")
- },
+ Some(ReaderType::Default(reader)) => reader.get().expect("Stream should have reader."),
_ => {
unreachable!("Getting default reader for a stream with a non-default reader")
},
@@ -565,7 +563,7 @@ impl ReadableStream {
/// <https://streams.spec.whatwg.org/#readable-stream-default-reader-read>
pub(crate) fn read_a_chunk(&self, can_gc: CanGc) -> Rc<Promise> {
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
let Some(reader) = reader.get() else {
unreachable!(
"Attempt to read stream chunk without having first acquired a reader."
@@ -587,7 +585,7 @@ impl ReadableStream {
let reader_ref = self.reader.borrow();
match reader_ref.as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
let Some(reader) = reader.get() else {
unreachable!("Attempt to stop reading without having first acquired a reader.");
};
@@ -604,8 +602,8 @@ impl ReadableStream {
/// <https://streams.spec.whatwg.org/#is-readable-stream-locked>
pub(crate) fn is_locked(&self) -> bool {
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => reader.get().is_some(),
- Some(ReaderType::BYOB(ref reader)) => reader.get().is_some(),
+ Some(ReaderType::Default(reader)) => reader.get().is_some(),
+ Some(ReaderType::BYOB(reader)) => reader.get().is_some(),
None => false,
}
}
@@ -632,21 +630,21 @@ impl ReadableStream {
pub(crate) fn has_default_reader(&self) -> bool {
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => reader.get().is_some(),
+ Some(ReaderType::Default(reader)) => reader.get().is_some(),
_ => false,
}
}
pub(crate) fn has_byob_reader(&self) -> bool {
match self.reader.borrow().as_ref() {
- Some(ReaderType::BYOB(ref reader)) => reader.get().is_some(),
+ Some(ReaderType::BYOB(reader)) => reader.get().is_some(),
_ => false,
}
}
pub(crate) fn has_byte_controller(&self) -> bool {
match self.controller.borrow().as_ref() {
- Some(ControllerType::Byte(ref controller)) => controller.get().is_some(),
+ Some(ControllerType::Byte(controller)) => controller.get().is_some(),
_ => false,
}
}
@@ -654,7 +652,7 @@ impl ReadableStream {
/// <https://streams.spec.whatwg.org/#readable-stream-get-num-read-requests>
pub(crate) fn get_num_read_requests(&self) -> usize {
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
let reader = reader
.get()
.expect("Stream must have a reader when getting the number of read requests.");
@@ -671,7 +669,7 @@ impl ReadableStream {
assert!(self.has_byob_reader());
match self.reader.borrow().as_ref() {
- Some(ReaderType::BYOB(ref reader)) => {
+ Some(ReaderType::BYOB(reader)) => {
let Some(reader) = reader.get() else {
unreachable!(
"Stream must have a reader when get num read into requests is called into."
@@ -694,7 +692,7 @@ impl ReadableStream {
assert!(self.has_default_reader());
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
// step 2 - Let reader be stream.[[reader]].
let reader = reader
.get()
@@ -735,7 +733,7 @@ impl ReadableStream {
// Let reader be stream.[[reader]].
match self.reader.borrow().as_ref() {
- Some(ReaderType::BYOB(ref reader)) => {
+ Some(ReaderType::BYOB(reader)) => {
let Some(reader) = reader.get() else {
unreachable!(
"Stream must have a reader when a read into request is fulfilled."
@@ -776,7 +774,7 @@ impl ReadableStream {
self.state.set(ReadableStreamState::Closed);
// Let reader be stream.[[reader]].
match self.reader.borrow().as_ref() {
- Some(ReaderType::Default(ref reader)) => {
+ Some(ReaderType::Default(reader)) => {
let Some(reader) = reader.get() else {
// If reader is undefined, return.
return;
@@ -784,7 +782,7 @@ impl ReadableStream {
// step 5 & 6
reader.close(can_gc);
},
- Some(ReaderType::BYOB(ref reader)) => {
+ Some(ReaderType::BYOB(reader)) => {
let Some(reader) = reader.get() else {
// If reader is undefined, return.
return;
@@ -823,7 +821,7 @@ impl ReadableStream {
self.close(can_gc);
// If reader is not undefined and reader implements ReadableStreamBYOBReader,
- if let Some(ReaderType::BYOB(ref reader)) = self.reader.borrow().as_ref() {
+ if let Some(ReaderType::BYOB(reader)) = self.reader.borrow().as_ref() {
if let Some(reader) = reader.get() {
// step 6.1, 6.2 & 6.3 of https://streams.spec.whatwg.org/#readable-stream-cancel
reader.cancel(can_gc);
@@ -833,11 +831,11 @@ impl ReadableStream {
// Let sourceCancelPromise be ! stream.[[controller]].[[CancelSteps]](reason).
let source_cancel_promise = match self.controller.borrow().as_ref() {
- Some(ControllerType::Default(ref controller)) => controller
+ Some(ControllerType::Default(controller)) => controller
.get()
.expect("Stream should have controller.")
.perform_cancel_steps(reason, can_gc),
- Some(ControllerType::Byte(ref controller)) => controller
+ Some(ControllerType::Byte(controller)) => controller
.get()
.expect("Stream should have controller.")
.perform_cancel_steps(reason, can_gc),
diff --git a/components/script/dom/servoparser/html.rs b/components/script/dom/servoparser/html.rs
index 5bbee966995..1a25d71058b 100644
--- a/components/script/dom/servoparser/html.rs
+++ b/components/script/dom/servoparser/html.rs
@@ -159,7 +159,7 @@ struct SerializationIterator {
stack: Vec<SerializationCommand>,
}
-fn rev_children_iter(n: &Node, can_gc: CanGc) -> impl Iterator<Item = DomRoot<Node>> {
+fn rev_children_iter(n: &Node, can_gc: CanGc) -> impl Iterator<Item = DomRoot<Node>> + use<'_> {
if n.downcast::<Element>().is_some_and(|e| e.is_void()) {
return Node::new_document_node().rev_children();
}
diff --git a/components/script/dom/servoparser/mod.rs b/components/script/dom/servoparser/mod.rs
index fd6c39c7f97..16501234b66 100644
--- a/components/script/dom/servoparser/mod.rs
+++ b/components/script/dom/servoparser/mod.rs
@@ -192,7 +192,7 @@ impl ServoParser {
context: &Element,
input: DOMString,
can_gc: CanGc,
- ) -> impl Iterator<Item = DomRoot<Node>> {
+ ) -> impl Iterator<Item = DomRoot<Node>> + use<'_> {
let context_node = context.upcast::<Node>();
let context_document = context_node.owner_doc();
let window = context_document.window();
diff --git a/components/script/dom/webglframebuffer.rs b/components/script/dom/webglframebuffer.rs
index b73c5101efb..7eb5f565b89 100644
--- a/components/script/dom/webglframebuffer.rs
+++ b/components/script/dom/webglframebuffer.rs
@@ -65,11 +65,11 @@ impl WebGLFramebufferAttachment {
}
fn root(&self) -> WebGLFramebufferAttachmentRoot {
- match *self {
- WebGLFramebufferAttachment::Renderbuffer(ref rb) => {
+ match self {
+ WebGLFramebufferAttachment::Renderbuffer(rb) => {
WebGLFramebufferAttachmentRoot::Renderbuffer(DomRoot::from_ref(rb))
},
- WebGLFramebufferAttachment::Texture { ref texture, .. } => {
+ WebGLFramebufferAttachment::Texture { texture, .. } => {
WebGLFramebufferAttachmentRoot::Texture(DomRoot::from_ref(texture))
},
}
@@ -78,7 +78,7 @@ impl WebGLFramebufferAttachment {
fn detach(&self) {
match self {
WebGLFramebufferAttachment::Renderbuffer(rb) => rb.detach_from_framebuffer(),
- WebGLFramebufferAttachment::Texture { ref texture, .. } => {
+ WebGLFramebufferAttachment::Texture { texture, .. } => {
texture.detach_from_framebuffer()
},
}
@@ -271,11 +271,11 @@ impl WebGLFramebuffer {
) -> Result<(), u32> {
// Get the size of this attachment.
let (format, size) = match attachment {
- Some(WebGLFramebufferAttachment::Renderbuffer(ref att_rb)) => {
+ Some(WebGLFramebufferAttachment::Renderbuffer(att_rb)) => {
(Some(att_rb.internal_format()), att_rb.size())
},
Some(WebGLFramebufferAttachment::Texture {
- texture: ref att_tex,
+ texture: att_tex,
level,
}) => match att_tex.image_info_at_face(0, *level as u32) {
Some(info) => (
diff --git a/components/script/dom/webgpu/gpudevice.rs b/components/script/dom/webgpu/gpudevice.rs
index aa2a0d6a9c8..681d90e40e3 100644
--- a/components/script/dom/webgpu/gpudevice.rs
+++ b/components/script/dom/webgpu/gpudevice.rs
@@ -239,7 +239,7 @@ impl GPUDevice {
&self,
layout: &GPUPipelineLayoutOrGPUAutoLayoutMode,
) -> PipelineLayout {
- if let GPUPipelineLayoutOrGPUAutoLayoutMode::GPUPipelineLayout(ref layout) = layout {
+ if let GPUPipelineLayoutOrGPUAutoLayoutMode::GPUPipelineLayout(layout) = layout {
PipelineLayout::Explicit(layout.id().0)
} else {
let layout_id = self.global().wgpu_id_hub().create_pipeline_layout_id();
diff --git a/components/script/dom/window.rs b/components/script/dom/window.rs
index 29a8eb60865..6689cd87d1a 100644
--- a/components/script/dom/window.rs
+++ b/components/script/dom/window.rs
@@ -3097,8 +3097,10 @@ fn is_named_element_with_id_attribute(elem: &Element) -> bool {
}
#[allow(unsafe_code)]
-#[no_mangle]
+#[unsafe(no_mangle)]
/// Helper for interactive debugging sessions in lldb/gdb.
unsafe extern "C" fn dump_js_stack(cx: *mut RawJSContext) {
- DumpJSStack(cx, true, false, false);
+ unsafe {
+ DumpJSStack(cx, true, false, false);
+ }
}
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-25 14:55:08 +0000