Chromium Code Reviews Chromium Code Reviews
Issue 882173002:
When a positioned object's dependence on content height changes we need to layout. (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| Index: LayoutTests/fast/block/positioning/positioned-movement-layout-when-bottom-changes-to-and-from-auto.html |
| diff --git a/LayoutTests/fast/block/positioning/positioned-movement-layout-when-bottom-changes-to-and-from-auto.html b/LayoutTests/fast/block/positioning/positioned-movement-layout-when-bottom-changes-to-and-from-auto.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..e3e81321a56fbbc179ced5fa9544dd5e230e11a4 |
| --- /dev/null |
| +++ b/LayoutTests/fast/block/positioning/positioned-movement-layout-when-bottom-changes-to-and-from-auto.html |
| @@ -0,0 +1,30 @@ |
| +<!DOCTYPE HTML> |
| +<style> |
| + body { |
| + margin: 0; |
| + } |
| + #test { |
| + position: absolute; |
| + background-color: green; |
| + top: 0; |
| + left: 0; |
| + right: 0; |
| + } |
| +</style> |
| +<script src="../../../resources/check-layout.js"></script> |
| +<script> |
| + |
|
mstensho (USE GERRIT)
2015/02/02 10:00:47
Superfluous blank line.
|
| + function runTest() |
| + { |
| + var test = document.getElementById('test'); |
| + document.body.offsetTop; |
| + test.style['bottom'] = '50%'; |
|
mstensho (USE GERRIT)
2015/02/02 10:00:47
Any reason why you shouldn't just let it be '50%'
|
| + document.body.offsetTop; |
| + test.style['bottom'] = 'auto'; |
| + window.checkLayout("#test"); |
| + } |
| + window.onload = runTest; |
| +</script> |
| + <div id="test" data-expected-height=0> |
|
mstensho (USE GERRIT)
2015/02/02 10:00:47
Strange indentation.
Maybe you could also add a c
|
| + </div> |
| +Tests positioned movement layout when its height is changed by top/bottom changing from auto to fixed and back again. |
|
mstensho (USE GERRIT)
2015/02/02 10:00:47
If you agree to make the change that I suggested a
|
