Summary: | innerHTML updates layout every time (hurts celtickane JS benchmark) | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Maciej Stachowiak <mjs> | ||||
Component: | HTML Editing | Assignee: | Maciej Stachowiak <mjs> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P2 | ||||||
Version: | 523.x (Safari 3) | ||||||
Hardware: | All | ||||||
OS: | OS X 10.4 | ||||||
Attachments: |
|
Description
Maciej Stachowiak
2007-09-30 02:19:30 PDT
Created attachment 16467 [details]
don't update layout for innerHTML
Comment on attachment 16467 [details]
don't update layout for innerHTML
I also looked, it appears that createMarkup(Node*... does not actually touch the render tree (renderer()) so I agree, this looks safe. r=me
Incidentally this patch is a 65% speedup on that one test in the benchmark. |