Chromium Code Reviews Chromium Code Reviews
Issue 343103003:
Flexbox: Allow intrinsic aspect ratios to inform main-size calculation (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/css3/flexbox/aspect-ratios-main-size-calculation.html |
| diff --git a/LayoutTests/css3/flexbox/aspect-ratios-main-size-calculation.html b/LayoutTests/css3/flexbox/aspect-ratios-main-size-calculation.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..2850153abbee8ff176db0a31857e9db4aff45be8 |
| --- /dev/null |
| +++ b/LayoutTests/css3/flexbox/aspect-ratios-main-size-calculation.html |
| @@ -0,0 +1,22 @@ |
| +<!DOCTYPE html> |
| +<style> |
| +.flexbox { |
| + display: flex; |
| + background-color: #aaa; |
| + position: relative; |
| +} |
| + |
| +.flexitem { |
| + background-color:red; |
| + -webkit-aspect-ratio: 1 / 2; |
|
tony
2014/06/23 16:50:57
We should have a test case where the flex item is
harpreet.sk
2014/06/26 14:41:48
I have added the test case where the flex item is
|
| +} |
| +</style> |
| +<script src="../../resources/check-layout.js"></script> |
| +<body onload="checkLayout('.flexbox')"> |
| +<p>Test for crbug.com/249112: In Flexbox, allow intrinsic aspect ratios to inform the main-size calculation</p> |
| +<div class="flexbox"> |
| + <div class="flexitem" style="height:100px" data-expected-width="50" data-expected-height="100"></div> |
|
tony
2014/06/23 16:50:57
We should also have a test case where the height i
harpreet.sk
2014/06/26 14:41:48
I have also added this test case in the new patch.
|
| +</div> |
| +<div class="flexbox" style="height:200px"> |
| + <div class="flexitem" style="height:50%" data-expected-width="50" data-expected-height="100"></div> |
| +</div> |
