Bug 7511 – make win32 DumpRenderTree run without crashing

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Maciej Stachowiak]

make win32 DumpRenderTree run without crashing

[Maciej Stachowiak]

Created attachment 6772 [details]
patch to fix remaining crashes

[Darin Adler]

Comment on attachment 6772 [details]
patch to fix remaining crashes

I'd prefer that the macro names like QNAME_DEFAULT_CONSTRUCTOR and DOM_HTMLNAMES_HIDE_GLOBALS match more closely -- either both should use a prefix or neither.

It seems to me that the !AVOID_STATIC_CONSTRUCTORS case is now a bit broken, so I'm not sure why AVOID_STATIC_CONSTRUCTORS still exists. It seems we now have a new technique for avoiding static constructors that is more portable, so we can do it everywhere.

I'm not happy with all the casts to void*, although I suppose they are necessary.

I don't understand why the changes here for static constructors are not also in JavaScriptCore.

Instead of __VA_ARGS__ the DEFINE_GLOBAL macro could be changed to take a parenthesized list of constructor parameters as a single macro argument, and hence work around a problem you were addressing here.

The patch has tabs in it, in the DocumentImpl.cpp part.

r=me

[Maciej Stachowiak]

The !AVOID case still generates static constructors, it just does its best to only make ones that won't do anything interesting. I believe it would still result in generating framework init functions.