Bug 33031

Summary: [V8] Generate configuring of all indexers.
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: WebCore JavaScriptAssignee: Dimitri Glazkov (Google) <dglazkov>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: All   
Bug Depends on: 32996    
Bug Blocks: 32569    
Attachments:
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.