[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