WebKit export: RenderReplaced::computeAspectRatioInformationForRenderBox does not need to call into RenderBox::computeReplacedLogicalWidth (#54564)

Ahmad-S792 · sammygill · web-flow · commit f102c090424b · 2025-08-31T16:18:16.000Z
https://bugs.webkit.org/show_bug.cgi?id=252589 Co-authored-by: Sammy Gill <sammy.gill@apple.com>
diff --git a/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/video-default-object-height-constrained-by-max-width.html b/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/video-default-object-height-constrained-by-max-width.html
@@ -0,0 +1,22 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta name="assert" content="Video that has no intrinsic size should have height constrained by max-width">
+<link rel="help" href="https://w3c.github.io/csswg-drafts/css-sizing-4/#aspect-ratio-size-transfers">
+<link author="Sammy Gill" href="mailto:sammy.gill@apple.com">
+<link rel="match" href="/css/reference/ref-filled-green-100px-square.xht">
+<style>
+  video {
+    background-color: green;
+    width: auto;
+    height: auto;
+    max-width: 100px;
+  }
+</style>
+</head>
+<!-- aspect-ratio should be set to: auto 100 / 100-->
+<body>
+<p>Test passes if there is a filled green square.</p>
+<video width="100" height="100" src="/css/support/60x60-green.png">
+</body>
+</html>
diff --git a/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/video-default-object-width-constrained-by-max-height.html b/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/video-default-object-width-constrained-by-max-height.html
@@ -0,0 +1,22 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta name="assert" content="Video that has no intrinsic size should have width constrained by max-height">
+<link rel="help" href="https://w3c.github.io/csswg-drafts/css-sizing-4/#aspect-ratio-size-transfers">
+<link author="Sammy Gill" href="mailto:sammy.gill@apple.com">
+<link rel="match" href="/css/reference/ref-filled-green-100px-square.xht">
+<style>
+  video {
+    background-color: green;
+    width: auto;
+    height: auto;
+    max-height: 100px;
+  }
+</style>
+</head>
+  <!-- aspect-ratio should be set to: auto 100 / 100-->
+<body>
+<p>Test passes if there is a filled green square.</p>
+<video width="100" height="100" src="/css/support/60x60-green.png">
+</body>
+</html>
diff --git a/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/video-inline-size-containment-no-crash.html b/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/video-inline-size-containment-no-crash.html
@@ -0,0 +1,19 @@
+<!DOCTYPE html>
+<html>
+<head>
+<link author="Sammy Gill" href="mailto:sammy.gill@apple.com">
+<meta name="assert" content="Applying inline-size containment to the video should not result in a debug assert being triggered">
+<style>
+video {
+  aspect-ratio: 1;
+  container-type: inline-size;
+  inset: 0 auto;
+  min-width: min-content;
+  position: fixed;
+}
+</style>
+</head>
+<body>
+<video></video>
+</body>
+</html>
