Bug 33547

Summary: [V8] Remove static const ints from V8CustomBinding.h
Product: WebKit Reporter: Nate Chapin <japhet>
Component: WebCore Misc.Assignee: Nate Chapin <japhet>
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 32638    
Description Flags
patch dglazkov: review+, japhet: commit-queue-

Description Nate Chapin 2010-01-12 12:57:36 PST
This consists of the following:
1. Define all custom fields that need to be attached to V8 DOM objects in CodeGeneratorV8.pm.
2. Generate the indices for those fields in CodeGeneratorV8.pm.
3. Remove const ints from V8Custom
4. Remove special cases for custom field counts from V8DOMWrapper.cpp.

Also, this patch will include some slight reworking of #includes, because there are a couple of files that were piggybacking on V8CustomBinding.h's includes.
Comment 1 Nate Chapin 2010-01-12 13:10:51 PST
Created attachment 46396 [details]

This patch will cause a Chromium canary compile break, so cq-.
Comment 2 Dimitri Glazkov (Google) 2010-01-12 13:16:56 PST
Comment on attachment 46396 [details]


Can you add a FIXME to make an effort to eliminate the hard-codiness in field array composition?
Comment 3 Nate Chapin 2010-01-12 14:14:44 PST