Bug 41778

Summary:

Add reverseFind to Vector and deploy in HTML5 parser

Product:

WebKit

Reporter:

Adam Barth <abarth>

Component:

New Bugs

Assignee:

Adam Barth <abarth>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

eric

Priority:

Version:

528+ (Nightly build)

Hardware:

Other

OS:

OS X 10.5

Bug Depends on:

Bug Blocks:

41123

Attachments:

Description	Flags
Patch	sam: review+, abarth: commit-queue+

Description Adam Barth 2010-07-07 11:09:15 PDT

Add reverseFind to Vector and deploy in HTML5 parser

Comment 1 Adam Barth 2010-07-07 11:13:17 PDT

Created attachment 60757 [details]
Patch

Comment 2 Adam Barth 2010-07-07 12:02:30 PDT

Comment on attachment 60757 [details]
Patch

Thanks Sam.

Comment 3 Eric Seidel (no email) 2010-07-07 13:11:18 PDT

Comment on attachment 60757 [details]
Patch

Huh?  I don't understand your comparison change.  How is it possible for find to work with a non-entity type?

What was the speedup of changing the operator?  I guess none, since it was already avoiding the operator in findIndex.

Comment 4 Adam Barth 2010-07-07 14:21:20 PDT

Committed r62697: <http://trac.webkit.org/changeset/62697>

Comment 5 Adam Barth 2010-07-07 15:38:04 PDT

I discussed this with Eric on the phone and he's on board now.