From bf4d73cf1e2ed87fd30c620c1d4de76fc94fd90e Mon Sep 17 00:00:00 2001
From: ldm0 <ldm2993593805@163.com>
Date: Mon, 24 Aug 2026 04:59:16 +0800
Subject: [PATCH] refactor(style): isolate live viewport resolution

---
 .../src/native_bridge/element/styles.rs       | 57 +++++++++++--------
 1 file changed, 33 insertions(+), 24 deletions(-)

diff --git a/moli-renderer-v8/src/native_bridge/element/styles.rs b/moli-renderer-v8/src/native_bridge/element/styles.rs
index ce303b36ab..408eed9f4a 100644
--- a/moli-renderer-v8/src/native_bridge/element/styles.rs
+++ b/moli-renderer-v8/src/native_bridge/element/styles.rs
@@ -340,34 +340,43 @@ fn style_object_computation_context<'s>(
     style: v8::Local<'s, v8::Object>,
 ) -> StyleComputationContext {
     let read_document = style_object_read_document(scope, style);
-    let viewport = bridge_handle_from_object(scope, style)
-        .ok()
-        .and_then(|(runtime_ptr, bridge_handle)| {
-            let runtime = unsafe { &*runtime_ptr };
-            match bridge_handle {
-                BridgeHandle::ComputedStyle(handle, descriptor) => match descriptor.target {
-                    ComputedStyleTargetKey::Dynamic => runtime
-                        .dom_host()
-                        .owner_document_handle(handle)
-                        .or(read_document)
-                        .map(|document| style_viewport_for_document(runtime, document)),
-                    ComputedStyleTargetKey::ChildFrame(frame_handle) => {
-                        iframe_handle_viewport(runtime, frame_handle)
-                    }
-                    ComputedStyleTargetKey::DetachedIframe(_)
-                    | ComputedStyleTargetKey::PopupDocument(_) => None,
-                },
-                BridgeHandle::Window
-                | BridgeHandle::Node(_)
-                | BridgeHandle::ClassList(_, _)
-                | BridgeHandle::Dataset(_)
-                | BridgeHandle::Style(_) => None,
-            }
-        })
+    let viewport = live_computed_style_viewport(scope, style)
         .unwrap_or_else(|| style_object_viewport(scope, style));
     StyleComputationContext::new(viewport).with_read_document(read_document)
 }
 
+fn live_computed_style_viewport<'s>(
+    scope: &mut v8::PinScope<'s, '_>,
+    style: v8::Local<'s, v8::Object>,
+) -> Option<StyleViewport> {
+    let Ok((runtime_ptr, BridgeHandle::ComputedStyle(handle, descriptor))) =
+        bridge_handle_from_object(scope, style)
+    else {
+        return None;
+    };
+    let runtime = unsafe { &*runtime_ptr };
+    match descriptor.target {
+        ComputedStyleTargetKey::Dynamic => {
+            if let Some(frame) = computed_style_target_child_frame_handle(scope, runtime, handle) {
+                return iframe_handle_viewport(runtime, frame);
+            }
+            if !computed_style_target_is_in_flat_tree(runtime, handle) {
+                return None;
+            }
+            let document = runtime.dom_host().owner_document_handle(handle)?;
+            Some(style_viewport_for_document(runtime, document))
+        }
+        ComputedStyleTargetKey::ChildFrame(frame)
+            if child_frame_target_document_is_current(runtime, handle, frame) =>
+        {
+            iframe_handle_viewport(runtime, frame)
+        }
+        ComputedStyleTargetKey::ChildFrame(_)
+        | ComputedStyleTargetKey::DetachedIframe(_)
+        | ComputedStyleTargetKey::PopupDocument(_) => None,
+    }
+}
+
 fn style_object_property_count_with_context<'s>(
     scope: &mut v8::PinScope<'s, '_>,
     style: v8::Local<'s, v8::Object>,