Boris Zbarsky [:bzbarsky] Assignee
	Description • 20 years ago

Some restyles can cause frame reconstructs, whicg may in some cases lead to
restyle events.  We want to make sure to not add entries to the hashtable while
it's being enumerated; see
http://lxr.mozilla.org/seamonkey/source/xpcom/ds/pldhash.h#544

Current plan:  when processing pending restyles, first make a list (on the
stack) of all the restyles to be processed, then clear the hashtable.  Then
process the list we built up.  This will keep us from modifying the hashtable
while we enumerate it.  That should also fix bug 256242, since by the time we're
processing the restyle flushing restyles again won't try to process it again.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 2 • 20 years ago

Comment on attachment 158556 [details] [diff] [review]
Something like this

I went with heap-allocation of the array because the hashtable can end up
containing several hundred elements on some pages (esp. "DHTML" ones).

	David Baron :dbaron:
	Comment 3 • 20 years ago

Comment on attachment 158556 [details] [diff] [review]
Something like this

r+sr=dbaron (I'm assuming the big chunk of code was just moved)

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 4 • 20 years ago

Yeah, the big chunk was just moved.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 5 • 20 years ago

Fix checked in.