Chromium Code Reviews Chromium Code Reviews
Issue 2842313002:
Add two large table collapsed border performance tests (Closed)
| Index: third_party/WebKit/PerformanceTests/Mutation/large-table-row-height-change-with-collapsed-border.html |
| diff --git a/third_party/WebKit/PerformanceTests/Mutation/large-table-row-height-change-with-collapsed-border.html b/third_party/WebKit/PerformanceTests/Mutation/large-table-row-height-change-with-collapsed-border.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..af3a09b59c17090a2c07a27821ca3e43b2c6fae1 |
| --- /dev/null |
| +++ b/third_party/WebKit/PerformanceTests/Mutation/large-table-row-height-change-with-collapsed-border.html |
| @@ -0,0 +1,33 @@ |
| +<!DOCTYPE html> |
| +<body> |
| +<script src="../resources/runner.js"></script> |
| +<style>td { border: 1px solid blue }</style> |
| +<script> |
| +function createTable(rows, columns) { |
| + var table = document.createElement("TABLE"); |
| + // Collapsing border is not necessary to see the slowness |
| + // but it makes the painting phase ~2x slower. |
| + table.style.borderCollapse = "collapse"; |
| + for (var i = 0; i < rows; ++i) { |
| + var tr = document.createElement("TR"); |
| + for (var j = 0; j < columns; ++j) { |
| + var td = document.createElement("TD"); |
| + tr.appendChild(td); |
| + } |
| + table.appendChild(tr); |
| + } |
| + return table; |
| +} |
| + |
| +var table = createTable(400, 400); |
| +document.body.appendChild(table); |
| + |
| +var y = 0; |
| +PerfTestRunner.measureFrameTime({ |
| + run: function() { |
|
pdr.
2017/04/27 05:18:56
Optional: can you move the setup into setup() and
Xianzhu
2017/04/27 05:56:00
Good idea! I will address this in a follow-up, inc
|
| + table.childNodes[y].style.height = '20px'; |
| + y++; |
| + }, |
| +}); |
| +</script> |
| +</body> |
