Send less title changes to the embedder (#33287)

Instead of sending a title change for each incremental parsing change, we now do it when:
- the element is bound to the tree.
- the parser is done with the element (in pop()).
- the title content changes after the parsing, eg. using document.title .

Signed-off-by: webbeef <me@webbeef.org>
This commit is contained in:
webbeef 2024-09-09 15:41:02 -07:00 committed by GitHub
parent e5150dbda1
commit 193f592617
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -2,6 +2,8 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this * License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at https://mozilla.org/MPL/2.0/. */ * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
use std::cell::Cell;
use dom_struct::dom_struct; use dom_struct::dom_struct;
use html5ever::{LocalName, Prefix}; use html5ever::{LocalName, Prefix};
use js::rust::HandleObject; use js::rust::HandleObject;
@ -19,6 +21,7 @@ use crate::dom::virtualmethods::VirtualMethods;
#[dom_struct] #[dom_struct]
pub struct HTMLTitleElement { pub struct HTMLTitleElement {
htmlelement: HTMLElement, htmlelement: HTMLElement,
popped: Cell<bool>,
} }
impl HTMLTitleElement { impl HTMLTitleElement {
@ -29,6 +32,7 @@ impl HTMLTitleElement {
) -> HTMLTitleElement { ) -> HTMLTitleElement {
HTMLTitleElement { HTMLTitleElement {
htmlelement: HTMLElement::new_inherited(local_name, prefix, document), htmlelement: HTMLElement::new_inherited(local_name, prefix, document),
popped: Cell::new(false),
} }
} }
@ -47,6 +51,13 @@ impl HTMLTitleElement {
proto, proto,
) )
} }
fn notify_title_changed(&self) {
let node = self.upcast::<Node>();
if node.is_in_doc() {
node.owner_doc().title_changed();
}
}
} }
impl HTMLTitleElementMethods for HTMLTitleElement { impl HTMLTitleElementMethods for HTMLTitleElement {
@ -70,9 +81,11 @@ impl VirtualMethods for HTMLTitleElement {
if let Some(s) = self.super_type() { if let Some(s) = self.super_type() {
s.children_changed(mutation); s.children_changed(mutation);
} }
let node = self.upcast::<Node>();
if node.is_in_doc() { // Notify of title changes only after the initial full parsing
node.owner_doc().title_changed(); // of the element.
if self.popped.get() {
self.notify_title_changed();
} }
} }
@ -85,4 +98,16 @@ impl VirtualMethods for HTMLTitleElement {
node.owner_doc().title_changed(); node.owner_doc().title_changed();
} }
} }
fn pop(&self) {
if let Some(s) = self.super_type() {
s.pop();
}
self.popped.set(true);
// Initial notification of title change, once the full text
// is available.
self.notify_title_changed();
}
} }