[Webkit-unassigned] [Bug 119026] New: Table DOM mutation methods (insertRow, insertCell) should lazy attach
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 23 16:58:18 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=119026
Summary: Table DOM mutation methods (insertRow, insertCell)
should lazy attach
Product: WebKit
Version: 528+ (Nightly build)
Platform: Unspecified
OS/Version: Unspecified
Status: NEW
Keywords: BlinkMergeCandidate
Severity: Normal
Priority: P2
Component: Layout and Rendering
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: rniwa at webkit.org
CC: bdakin at apple.com, koivisto at iki.fi,
simon.fraser at apple.com, dino at apple.com,
robert at webkit.org, akling at apple.com
Consider merging https://chromium.googlesource.com/chromium/blink/+/74ae9c918ea8b6b1a41871b826ec3f2a5737061c
Doing thead.insertRow, table.insertRow or tr.insertCell would sync attach, this
changes them to do lazyAttach just like doing appendChild() or insertBefore()
would have.
In a basic benchmark:
table.offsetTop;
var t = Date.now();
for (var i = 0; i < 200; i++) {
var row = table.insertRow(0);
for (var j = 0; j < 200; j++)
row.insertCell(0).textContent = "foo bar";
}
table.offsetTop;
document.body.textContent = Date.now() - t;
This cuts the time from 840ms down to 550ms for a 35% speed improvement.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list