518836 - Set scrollbar attributes asynchronously

Status: NEW

(Core :: Layout, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: WIP, not properly tested

Doing textarea.value += somevalue + "\n" in a loop causes us to
set scrollbar attribute very often. We should, if possible, update scrollbars
asynchronously.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

I may need to tweak the patch, but does the idea sound reasonable?

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Attachment: safer, async only with text form controls

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

If a script is doing textarea.value += somevalue in a loop, why are we flushing reflows?

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I mean, the patch seems fine, but I don't understand the use-case.

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Editor flushes, but yeah, perhaps I could get rid of that.

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Ok, seems like I can disable the flush.

Comment 7

[Boris Zbarsky [:bzbarsky]]

See bug 259636 for the sordid history of trying to disable the flush.  If the caret issue is fixed now, we can probably reconsider the patch in that bug, right?  Or even bug 174823?  That discussion should happen there, though.