From 5dac7467d462c7aceddc4d2463ef915f414a96cb Mon Sep 17 00:00:00 2001
From: Aliaksandr Kalenik <kalenik.aliaksandr@gmail.com>
Date: Tue, 11 Aug 2026 11:25:08 +0100
Subject: [PATCH] LibWeb: Stop refreshing SVG payloads from records at run
 close

The close-time refresh compensated for the pass-global used-values
store: SVG resource subtrees are laid out once per consumer, and a
later consumer's layout rewrote shared records after an earlier
placement had already snapshotted them into fragments. Now that runs
own their used-values records, each consumer's resource box carries
its own slots and every SVG payload write lands before the owning
box's placement snapshots it, so the walk never found anything to
patch. Delete the refresh and the flag that gated it.
---
 .../LibWeb/Rust/src/layout/fragment_tree.rs   | 32 -------------------
 .../Rust/src/layout/svg_formatting_context.rs |  9 ------
 2 files changed, 41 deletions(-)

diff --git a/Libraries/LibWeb/Rust/src/layout/fragment_tree.rs b/Libraries/LibWeb/Rust/src/layout/fragment_tree.rs
index 4fe0a55eb063f..ff6186bd856fb 100644
--- a/Libraries/LibWeb/Rust/src/layout/fragment_tree.rs
+++ b/Libraries/LibWeb/Rust/src/layout/fragment_tree.rs
@@ -304,7 +304,6 @@ pub(crate) struct RunFragmentBuilder {
     root_node: crate::layout::node_data::NodeSlotId,
     root_containing_block_slot: Option<u32>,
     is_entry_accumulator: bool,
-    saw_svg_payload_write: Cell<bool>,
     inner: std::cell::RefCell<RunFragmentBuilderInner>,
 }
 
@@ -351,7 +350,6 @@ impl RunFragmentBuilder {
             root_node,
             root_containing_block_slot: root_containing_block.map(|node| node.slot_index()),
             is_entry_accumulator: false,
-            saw_svg_payload_write: Cell::new(false),
             inner: std::cell::RefCell::new(RunFragmentBuilderInner::default()),
         }
     }
@@ -361,7 +359,6 @@ impl RunFragmentBuilder {
             root_node,
             root_containing_block_slot: None,
             is_entry_accumulator: true,
-            saw_svg_payload_write: Cell::new(false),
             inner: std::cell::RefCell::new(RunFragmentBuilderInner::default()),
         }
     }
@@ -370,10 +367,6 @@ impl RunFragmentBuilder {
         self.root_node
     }
 
-    pub(crate) fn note_svg_payload_write(&self) {
-        self.saw_svg_payload_write.set(true);
-    }
-
     pub(crate) fn register_pending_abspos(
         &self,
         coordinate_space_box: crate::layout::node_data::NodeSlotId,
@@ -719,31 +712,6 @@ impl RunFragmentBuilder {
         for candidate in &mut propagated_anchor_candidates {
             propagate_payload_toward_run_root_space(candidate, self.root_node, records, callbacks);
         }
-        if self.saw_svg_payload_write.take() {
-            refresh_svg_payloads_from_records(&mut inner.top_scope_links, records);
-        }
         (inner.top_scope_links, propagated_pending_abspos, propagated_anchor_candidates)
     }
 }
-
-fn refresh_svg_payloads_from_records(links: &mut [FragmentLink], records: &RunRecords) {
-    for link in links {
-        let Some(owned_record) = records.used_values_if_owned(link.fragment.node) else {
-            continue;
-        };
-        let fragment = &mut *link.fragment;
-        if let Some(rare_cell) = owned_record.rare_data.get() {
-            let mut rare = rare_cell.borrow_mut();
-            if rare.computed_svg_transforms.is_some() {
-                fragment.computed_svg_transforms = rare.computed_svg_transforms;
-            }
-            if rare.svg_viewport_size.is_some() {
-                fragment.svg_viewport_size = rare.svg_viewport_size;
-            }
-            if let Some(path) = rare.computed_svg_path.take() {
-                fragment.computed_svg_path.set(Some(path));
-            }
-        }
-        refresh_svg_payloads_from_records(&mut fragment.children, records);
-    }
-}
diff --git a/Libraries/LibWeb/Rust/src/layout/svg_formatting_context.rs b/Libraries/LibWeb/Rust/src/layout/svg_formatting_context.rs
index 5f635e3e1cce7..4a21d809e7f9f 100644
--- a/Libraries/LibWeb/Rust/src/layout/svg_formatting_context.rs
+++ b/Libraries/LibWeb/Rust/src/layout/svg_formatting_context.rs
@@ -515,19 +515,11 @@ impl SvgFormattingContext {
         has_transforms.then_some(transforms)
     }
 
-    fn note_svg_payload_write(&self) {
-        if let Some(fragments) = &self.fragments {
-            fragments.note_svg_payload_write();
-        }
-    }
-
     fn set_computed_transforms(&self, node: Node, transforms: FfiSvgComputedTransforms) {
-        self.note_svg_payload_write();
         self.used_values(node).rare_data_mut().computed_svg_transforms = Some(transforms);
     }
 
     fn set_svg_viewport_size(&self, node: Node, viewport_size: FfiCssPixelSize) {
-        self.note_svg_payload_write();
         self.used_values(node).rare_data_mut().svg_viewport_size = Some(viewport_size);
     }
 
@@ -975,7 +967,6 @@ impl SvgFormattingContext {
         let used = &used_pointer;
         used.set_content_inline_size(transformed_bounding_box.width);
         used.set_content_block_size(transformed_bounding_box.height);
-        self.note_svg_payload_write();
         self.used_values(graphics_box).rare_data_mut().computed_svg_path = Some(path);
         self.place_child(graphics_box, transformed_bounding_box.x, transformed_bounding_box.y);
         used.has_definite_inline_size.set(true);