This is the sixth level of the unzipping process described in https://bugs.webkit.org/show_bug.cgi?id=66567.
Created attachment 107121 [details] Patch
Comment on attachment 107121 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=107121&action=review From review discussion with Geoff. > Source/JavaScriptCore/ChangeLog:13 > + into the constructors of the subclasses of the second level of the hierarchy Not true. Needs to be fifth level instead of second. > Source/JavaScriptCore/ChangeLog:20 > + File names truncated for brevity. Need more details on why this is done. > Source/WebCore/ChangeLog:24 > + Changed the DOM bindings generation script to move the calls to finishCreation into the > + create methods of all the generated classes except for children of JSDOMWindowBase and > + JSWorkerContextBase. Because JSDOMWindowBase and JSWorkerContextBase are the next level down. However, so are some of the classes that get changed with this patch, but it's just easier to change all of them at once. I added generation of finishCreation methods for all of these classes
Comment on attachment 107121 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=107121&action=review review+ assuming you do the run-bindings-tests thing I requested >> Source/JavaScriptCore/ChangeLog:20 >> + File names truncated for brevity. > > Need more details on why this is done. Please do not do that. We do want you to include the filenames and in most cases, the function names too. > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:733 > + push(@headerContent, " $className* ptr = new (JSC::allocateCell<$className>(globalData.heap)) ${className}(globalData, structure, impl, windowShell);\n"); Since you are changing the code generator, you need to run "run-bindings-tests --reset-results" and check in the changes to the output of the bindings code generation tests.
Created attachment 107262 [details] Fixing issues
Comment on attachment 107262 [details] Fixing issues This fixes Darin's comments, so r=me.
Comment on attachment 107262 [details] Fixing issues Clearing flags on attachment: 107262 Committed r95108: <http://trac.webkit.org/changeset/95108>
All reviewed patches have been landed. Closing bug.
Broke some tests on Qt. Working on a fix.
Build issue has been resolved.