[webkit-reviews] review requested: [Bug 8177] Javascript search incredibly slow : [Attachment 8882] proposed patch

[bugzilla-request-daemon at opendarwin.org]

Alexey Proskuryakov <ap at nypop.com> has asked  for review:
Bug 8177: Javascript search incredibly slow
http://bugzilla.opendarwin.org/show_bug.cgi?id=8177

Attachment 8882: proposed patch
http://bugzilla.opendarwin.org/attachment.cgi?id=8882&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at nypop.com>
Use shared CollectionInfos for HTMLOptionsCollection and HTMLNameCollection,
too.

The test runs perfectly fast now. A test with 2000 options also runs fast (no
visible delay, a nice change from tens of seconds), but it still takes several
seconds to open, because the DOM version keeps changing, which defeats caching:

   for (var i=0; i < 2000; i++)
	document.fmain.user.options[i] = new Option("User " + i, i);

I couldn't understand why the other collections (images, applets, all etc.)
were returned from Document, but were only shared for HTMLDocuments. I just did
the same.