Bug 33031

Summary: [V8] Generate configuring of all indexers.
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: WebCore JavaScriptAssignee: Dimitri Glazkov (Google) <dglazkov>
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: All   
Bug Depends on: 32996    
Bug Blocks: 32569    
Description Flags
Generate configuring of all indexers, v1. abarth: review+

Description Dimitri Glazkov (Google) 2009-12-29 15:45:42 PST
It's a bit ugly. The biggest issue is that apparently V8 API completely decouples numeric and non-numeric property lookup, which forces us to have extra indexers. The indexers simply convert index value to string and pass it along to named property handling machinery. It's sad. I'll ask V8 folks what's up with that.

Otherwise, the story is very similar to bug 32996
Comment 1 Dimitri Glazkov (Google) 2009-12-29 15:50:12 PST
Created attachment 45624 [details]
Generate configuring of all indexers, v1.
Comment 2 Adam Barth 2009-12-29 18:51:53 PST
Comment on attachment 45624 [details]
Generate configuring of all indexers, v1.

This patch feels like deja vu.  I didn't quite follow all the machinations here, but the net result looks good.  Hopefully our tests are strong enough to catch screw ups.
Comment 3 Dimitri Glazkov (Google) 2009-12-30 08:55:45 PST
Landed as http://trac.webkit.org/changeset/52663.