diff options
Diffstat (limited to 'components/script')
-rw-r--r-- | components/script/devtools.rs | 4 | ||||
-rw-r--r-- | components/script/dom/document.rs | 4 | ||||
-rw-r--r-- | components/script/dom/htmlformelement.rs | 2 | ||||
-rw-r--r-- | components/script/dom/htmlinputelement.rs | 5 | ||||
-rw-r--r-- | components/script/dom/node.rs | 5 | ||||
-rw-r--r-- | components/script/dom/performance.rs | 4 | ||||
-rw-r--r-- | components/script/dom/webglprogram.rs | 2 | ||||
-rw-r--r-- | components/script/dom/webglrenderingcontext.rs | 2 | ||||
-rw-r--r-- | components/script/dom/websocket.rs | 4 | ||||
-rw-r--r-- | components/script/dom/writablestreamdefaultwriter.rs | 2 | ||||
-rw-r--r-- | components/script/stylesheet_loader.rs | 4 | ||||
-rw-r--r-- | components/script/textinput.rs | 2 | ||||
-rw-r--r-- | components/script/xpath/parser.rs | 2 |
13 files changed, 18 insertions, 24 deletions
diff --git a/components/script/devtools.rs b/components/script/devtools.rs index 66e22fc7b4e..d657e6ffd06 100644 --- a/components/script/devtools.rs +++ b/components/script/devtools.rs @@ -144,9 +144,7 @@ pub(crate) fn handle_get_children( Some(parent) => { let is_whitespace = |node: &NodeInfo| { node.node_type == NodeConstants::TEXT_NODE && - node.node_value - .as_ref() - .map_or(true, |v| v.trim().is_empty()) + node.node_value.as_ref().is_none_or(|v| v.trim().is_empty()) }; let inline: Vec<_> = parent diff --git a/components/script/dom/document.rs b/components/script/dom/document.rs index 5f8388066fc..e9b96fc1061 100644 --- a/components/script/dom/document.rs +++ b/components/script/dom/document.rs @@ -1114,7 +1114,7 @@ impl Document { if implicit_transaction { self.begin_focus_transaction(); } - if elem.map_or(true, |e| e.is_focusable_area()) { + if elem.is_none_or(|e| e.is_focusable_area()) { *self.focus_transaction.borrow_mut() = FocusTransaction::InTransaction(elem.map(Dom::from_ref)); } @@ -1795,7 +1795,7 @@ impl Document { let target_has_changed = prev_mouse_over_target .get() .as_ref() - .map_or(true, |old_target| old_target != &new_target); + .is_none_or(|old_target| old_target != &new_target); // Here we know the target has changed, so we must update the state, // dispatch mouseout to the previous one, mouseover to the new one. diff --git a/components/script/dom/htmlformelement.rs b/components/script/dom/htmlformelement.rs index a19eb97d843..3a7a56ffaf4 100644 --- a/components/script/dom/htmlformelement.rs +++ b/components/script/dom/htmlformelement.rs @@ -1639,7 +1639,7 @@ pub(crate) trait FormControl: DomObject { let has_form_attr = elem.has_attribute(&local_name!("form")); let same_subtree = self .form_owner() - .map_or(true, |form| elem.is_in_same_home_subtree(&*form)); + .is_none_or(|form| elem.is_in_same_home_subtree(&*form)); self.unregister_if_necessary(); diff --git a/components/script/dom/htmlinputelement.rs b/components/script/dom/htmlinputelement.rs index 7ea5586bc7b..eedd9134966 100644 --- a/components/script/dom/htmlinputelement.rs +++ b/components/script/dom/htmlinputelement.rs @@ -837,10 +837,7 @@ impl HTMLInputElement { }, // https://html.spec.whatwg.org/multipage/#file-upload-state-(type%3Dfile)%3Asuffering-from-being-missing InputType::File => { - self.Required() && - self.filelist - .get() - .map_or(true, |files| files.Length() == 0) + self.Required() && self.filelist.get().is_none_or(|files| files.Length() == 0) }, // https://html.spec.whatwg.org/multipage/#the-required-attribute%3Asuffering-from-being-missing _ => { diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index eef14ef6fa2..8055425976a 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -1317,7 +1317,7 @@ impl Node { } pub(crate) fn is_display_none(&self) -> bool { - self.style_data.borrow().as_ref().map_or(true, |data| { + self.style_data.borrow().as_ref().is_none_or(|data| { data.element_data .borrow() .styles @@ -2184,8 +2184,7 @@ impl Node { ) { node.owner_doc().add_script_and_layout_blocker(); debug_assert!(*node.owner_doc() == *parent.owner_doc()); - debug_assert!(child.map_or(true, |child| Some(parent) == - child.GetParentNode().as_deref())); + debug_assert!(child.is_none_or(|child| Some(parent) == child.GetParentNode().as_deref())); // Step 1. let count = if node.is::<DocumentFragment>() { diff --git a/components/script/dom/performance.rs b/components/script/dom/performance.rs index c19adceaca1..cd537642bb3 100644 --- a/components/script/dom/performance.rs +++ b/components/script/dom/performance.rs @@ -78,10 +78,10 @@ impl PerformanceEntryList { .entries .iter() .filter(|e| { - name.as_ref().map_or(true, |name_| *e.name() == *name_) && + name.as_ref().is_none_or(|name_| *e.name() == *name_) && entry_type .as_ref() - .map_or(true, |type_| *e.entry_type() == *type_) + .is_none_or(|type_| *e.entry_type() == *type_) }) .cloned() .collect::<Vec<DomRoot<PerformanceEntry>>>(); diff --git a/components/script/dom/webglprogram.rs b/components/script/dom/webglprogram.rs index 562b7620038..5b101430c6a 100644 --- a/components/script/dom/webglprogram.rs +++ b/components/script/dom/webglprogram.rs @@ -432,7 +432,7 @@ impl WebGLProgram { let (size, type_) = { let (base_name, array_index) = match parse_uniform_name(&name) { - Some((name, index)) if index.map_or(true, |i| i >= 0) => (name, index), + Some((name, index)) if index.is_none_or(|i| i >= 0) => (name, index), _ => return Ok(None), }; diff --git a/components/script/dom/webglrenderingcontext.rs b/components/script/dom/webglrenderingcontext.rs index 337ba8415e0..ecccecffb3b 100644 --- a/components/script/dom/webglrenderingcontext.rs +++ b/components/script/dom/webglrenderingcontext.rs @@ -670,7 +670,7 @@ impl WebGLRenderingContext { // or UNSIGNED_SHORT_5_5_5_1, a Uint16Array must be supplied. // or FLOAT, a Float32Array must be supplied. // If the types do not match, an INVALID_OPERATION error is generated. - let data_type_matches = data.as_ref().map_or(true, |buffer| { + let data_type_matches = data.as_ref().is_none_or(|buffer| { Some(data_type.sized_data_type()) == array_buffer_type_to_sized_type(buffer.get_array_type()) && data_type.required_webgl_version() <= self.webgl_version() diff --git a/components/script/dom/websocket.rs b/components/script/dom/websocket.rs index f00b3ef6aea..363ccb2f761 100644 --- a/components/script/dom/websocket.rs +++ b/components/script/dom/websocket.rs @@ -349,7 +349,7 @@ impl WebSocketMethods<crate::DomTypeHolder> for WebSocket { // https://html.spec.whatwg.org/multipage/#dom-websocket-send fn Send(&self, data: USVString) -> ErrorResult { - let data_byte_len = data.0.as_bytes().len() as u64; + let data_byte_len = data.0.len() as u64; let send_data = self.send_impl(data_byte_len)?; if send_data { @@ -417,7 +417,7 @@ impl WebSocketMethods<crate::DomTypeHolder> for WebSocket { } } if let Some(ref reason) = reason { - if reason.0.as_bytes().len() > 123 { + if reason.0.len() > 123 { //reason cannot be larger than 123 bytes return Err(Error::Syntax); } diff --git a/components/script/dom/writablestreamdefaultwriter.rs b/components/script/dom/writablestreamdefaultwriter.rs index ab26a752804..2c6358e96ac 100644 --- a/components/script/dom/writablestreamdefaultwriter.rs +++ b/components/script/dom/writablestreamdefaultwriter.rs @@ -286,7 +286,7 @@ impl WritableStreamDefaultWriter { if !self .stream .get() - .map_or(false, |current_stream| current_stream == stream) + .is_some_and(|current_stream| current_stream == stream) { let promise = Promise::new(global, can_gc); promise.reject_error(Error::Type( diff --git a/components/script/stylesheet_loader.rs b/components/script/stylesheet_loader.rs index 80a3f878160..94e6aa299e6 100644 --- a/components/script/stylesheet_loader.rs +++ b/components/script/stylesheet_loader.rs @@ -188,7 +188,7 @@ impl FetchResponseListener for StylesheetContext { // else we risk applying the wrong stylesheet when responses come out-of-order. let is_stylesheet_load_applicable = self .request_generation_id - .map_or(true, |gen| gen == link.get_request_generation_id()); + .is_none_or(|gen| gen == link.get_request_generation_id()); if is_stylesheet_load_applicable { let shared_lock = document.style_shared_lock().clone(); let sheet = Arc::new(Stylesheet::from_bytes( @@ -375,7 +375,7 @@ impl StyleStylesheetLoader for StylesheetLoader<'_> { layer: ImportLayer, ) -> Arc<Locked<ImportRule>> { // Ensure the supports conditions for this @import are true, if not, refuse to load - if !supports.as_ref().map_or(true, |s| s.enabled) { + if !supports.as_ref().is_none_or(|s| s.enabled) { return Arc::new(lock.wrap(ImportRule { url, stylesheet: ImportSheet::new_refused(), diff --git a/components/script/textinput.rs b/components/script/textinput.rs index 1a72c0d26f8..64f4d75d8ef 100644 --- a/components/script/textinput.rs +++ b/components/script/textinput.rs @@ -1001,7 +1001,7 @@ impl<T: ClipboardProvider> TextInput<T> { /// Whether the content is empty. pub(crate) fn is_empty(&self) -> bool { - self.lines.len() <= 1 && self.lines.first().map_or(true, |line| line.is_empty()) + self.lines.len() <= 1 && self.lines.first().is_none_or(|line| line.is_empty()) } /// The length of the content in bytes. diff --git a/components/script/xpath/parser.rs b/components/script/xpath/parser.rs index cec3148bc70..b1a4bfcc42d 100644 --- a/components/script/xpath/parser.rs +++ b/components/script/xpath/parser.rs @@ -352,7 +352,7 @@ impl CoreFunction { let min = self.min_args(); let max = self.max_args(); - num_args >= min && max.map_or(true, |max| num_args <= max) + num_args >= min && max.is_none_or(|max| num_args <= max) } } |