Bug 67692 – Unzip initialization lists and constructors in JSCell hierarchy (6/7)

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Mark Hahnenberg]

This is the sixth level of the unzipping process described in https://bugs.webkit.org/show_bug.cgi?id=66567.

[Mark Hahnenberg]

Created attachment 107121 [details]
Patch

[Mark Hahnenberg]

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

[Darin Adler]

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.

[Mark Hahnenberg]

Created attachment 107262 [details]
Fixing issues

[Geoffrey Garen]

Comment on attachment 107262 [details]
Fixing issues

This fixes Darin's comments, so r=me.

[WebKit Review Bot]

Comment on attachment 107262 [details]
Fixing issues

Clearing flags on attachment: 107262

Committed r95108: <http://trac.webkit.org/changeset/95108>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Mark Hahnenberg]

Broke some tests on Qt.  Working on a fix.

[Mark Hahnenberg]

Build issue has been resolved.