nsHTMLTableRowElement::InsertCell could be faster

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

(Blocks: 1 bug)

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

In particular, we can avoid updating our mCells list unless we have to.  This came up in bug 437733.
Created attachment 387395 [details] [diff] [review]
Like so, say
Attachment #387395 - Flags: superreview?(jst)
Attachment #387395 - Flags: review?(jst)
>+    // -1 means append, so should use null prevSibling

That should be nextSibling, of course.
Blocks: 437733
Comment on attachment 387395 [details] [diff] [review]
Like so, say

r+sr=jst with your own comment addressed :)
Attachment #387395 - Flags: superreview?(jst)
Attachment #387395 - Flags: superreview+
Attachment #387395 - Flags: review?(jst)
Attachment #387395 - Flags: review+
Pushed http://hg.mozilla.org/mozilla-central/rev/8c125dba9db7
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.