Chromium Code Reviews Chromium Code Reviews
Issue 1145773002:
Setting -webkit-column-count to auto is the same as not setting it. (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| Index: LayoutTests/fast/multicol/explicit-columns-auto.html |
| diff --git a/LayoutTests/fast/multicol/explicit-columns-auto.html b/LayoutTests/fast/multicol/explicit-columns-auto.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..34aa97a59f2d5d312e87ebe681a1b4d539d4b363 |
| --- /dev/null |
| +++ b/LayoutTests/fast/multicol/explicit-columns-auto.html |
| @@ -0,0 +1,23 @@ |
| +<!DOCTYPE html> |
| +<script src="../../resources/js-test.js"></script> |
| +<style> |
| + .explicitlyNoColumns { -webkit-columns:auto; } |
|
rune
2015/05/19 09:18:15
Inconsistent indentation. 2 spaces here and 4 spac
mstensho (USE GERRIT)
2015/05/19 09:37:17
Done.
|
| +</style> |
| + |
| +<div id="elm"></div> |
| + |
| +<script> |
| + description("Test that setting columns to auto is the same as not setting it at all."); |
| + onload = function() { |
| + document.body.offsetTop; |
|
rune
2015/05/19 09:18:15
The offsetTop forced style/layout should suffice.
mstensho (USE GERRIT)
2015/05/19 09:37:17
Done.
|
| + document.getElementById('elm').className = "explicitlyNoColumns"; |
| + if (window.internals) { |
| + // Need to call updateStyleAndReturnAffectedElementCount() first, to get style recalculated, |
| + // so that the necessary objects get marked for layout (which, if the test passes, should be 0). |
| + shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "1"); |
| + shouldBe("internals.needsLayoutCount()", "0"); |
| + } else { |
| + testFailed("There's no web exposed API usable for this test. The bug doesn't affect layout in a reliable way. Please rerun inside the layout test framework."); |
| + } |
| + } |
| +</script> |
