Skip to content

Commit 2b23bed

Browse files
kojiishichromium-wpt-export-bot
authored andcommitted
[iframe] Switch the property to frame-sizing
This patch switches the responsive iframe CSS property from `contain-intrinsic-block-size` to `frame-sizing`, as resolved on Jan 27th[1]. Tests are also updated, with two exceptions: * `responsive-iframe-logical.tentative.html` isn't meaningful that it is removed. * The fallback sizing was postponed to resolve, and thus its test `responsive-iframe-fallback.tentative.html` is expected to fail. All changes are under a runtime flag currently experimental. [1] w3c/csswg-drafts#1771 (comment) Bug: 418397278 Change-Id: If0f1db67a9690f9a58392b6a704d3dc405954cfd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7532195 Reviewed-by: Kent Tamura <[email protected]> Auto-Submit: Koji Ishii <[email protected]> Commit-Queue: Kent Tamura <[email protected]> Cr-Commit-Position: refs/heads/main@{#1576859}
1 parent d36712a commit 2b23bed

7 files changed

+9
-40
lines changed

css/css-sizing/responsive-iframe/responsive-iframe-cross-origin.sub.tentative.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
<link rel="match" href="responsive-iframe-ref.html">
66
<style>
77
iframe {
8-
border: 1px solid black;
9-
contain-intrinsic-size: from-element none;
8+
border: 1px solid black;
9+
frame-sizing: content-height;
1010
}
1111
</style>
1212
<iframe frameborder=0 src="http://{{hosts[alt][]}}:{{ports[http][0]}}/css/css-sizing/responsive-iframe/resources/iframe-contents.html">

css/css-sizing/responsive-iframe/responsive-iframe-dom-content-loaded.tentative.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<script src="/resources/testharnessreport.js"></script>
77
<style>
88
iframe {
9-
contain-intrinsic-size: from-element none;
9+
frame-sizing: content-height;
1010
}
1111
</style>
1212
<iframe id="target" frameborder=0 src="resources/iframe-contents-dom-content-loaded.html"></iframe>

css/css-sizing/responsive-iframe/responsive-iframe-icb.tentative.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<style>
88
iframe {
99
border: 1px solid black;
10-
contain-intrinsic-size: from-element none;
10+
frame-sizing: content-height;
1111
}
1212
</style>
1313
<iframe id="target" frameborder=0 src="resources/iframe-contents-icb.html"></iframe>

css/css-sizing/responsive-iframe/responsive-iframe-logical.tentative.html

Lines changed: 0 additions & 31 deletions
This file was deleted.

css/css-sizing/responsive-iframe/responsive-iframe-not-embedded-sized.tentative.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
<link rel="match" href="responsive-iframe-unsized-ref.html">
66
<style>
77
iframe {
8-
border: 1px solid black;
9-
contain-intrinsic-size: from-element none;
8+
border: 1px solid black;
9+
frame-sizing: content-height;
1010
}
1111
</style>
1212
<iframe frameborder=0 scrolling=no src="resources/iframe-contents-unsized.html">

css/css-sizing/responsive-iframe/responsive-iframe-request-resize.tentative.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<style>
77
iframe {
88
border: 1px solid black;
9-
contain-intrinsic-size: from-element none;
9+
frame-sizing: content-height;
1010
}
1111
</style>
1212
<iframe frameborder=0 src="resources/iframe-contents-request-resize.html">

css/css-sizing/responsive-iframe/responsive-iframe.tentative.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
<link rel="match" href="responsive-iframe-ref.html">
66
<style>
77
iframe {
8-
border: 1px solid black;
9-
contain-intrinsic-size: from-element none;
8+
border: 1px solid black;
9+
frame-sizing: content-height;
1010
}
1111
</style>
1212
<iframe frameborder=0 src="resources/iframe-contents.html">

0 commit comments

Comments
 (0)