Chromium Code Reviews Chromium Code Reviews
Issue 1695193006:
Performance test for deeply nested tables inside multicol. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/PerformanceTests/Layout/multicol/deeply-nested-tables.html |
| diff --git a/third_party/WebKit/PerformanceTests/Layout/multicol/deeply-nested-tables.html b/third_party/WebKit/PerformanceTests/Layout/multicol/deeply-nested-tables.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..93d06252345bebb14bf925e02378e2c1553f43e7 |
| --- /dev/null |
| +++ b/third_party/WebKit/PerformanceTests/Layout/multicol/deeply-nested-tables.html |
| @@ -0,0 +1,63 @@ |
| +<!DOCTYPE html> |
| +<script src="../../resources/runner.js"></script> |
| +<style> |
| + #target div { display:table; } |
| +</style> |
| +<pre id="log"></pre> |
| +<div id="target" style="display:none; columns:3; column-fill:auto; width:40em; height:41.5em; line-height:2em;"> |
|
rune
2016/10/19 23:03:00
41.5em seems arbitrary, or is it picked to fit the
mstensho (USE GERRIT)
2016/10/20 09:43:02
Pretty arbitrary, although it's kind of nice that
|
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + <div> |
| + content content content |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| + </div> |
| +</div> |
| +<script> |
| + var target = document.getElementById("target"); |
| + var style = target.style; |
| + |
| + function test() { |
| + style.display = "block"; |
| + PerfTestRunner.forceLayout(); |
| + style.display = "none"; |
| + PerfTestRunner.forceLayout(); |
| + } |
| + |
| + PerfTestRunner.measureRunsPerSecond({ |
| + description: "Measures performance of multicol layout in deeply nested tables.", |
|
rune
2016/10/19 23:03:00
Is the time spent in layout tree building a lot sm
mstensho (USE GERRIT)
2016/10/20 09:43:02
Tree building should be unaffected by my fix.
rune
2016/10/20 09:58:22
My point was if the test really measures layout tr
|
| + run: test |
| + }); |
| +</script> |
