Bug 16061 – no-svg build broken after r27278

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[David Kilzer (:ddkilzer)]

I broke the no-svg build after r27278 because neither DOMHTMLEmbedElementPrivate.h nor DOMHTMLObjectElementPrivate.h are created when SVG is disabled.

I'm not sure how to best fix this.  At first glance, either both header files always need to be generated (even with empty categories), or else Xcode needs to conditionalize them.  (Or I just need to remove them from Xcode.)  Note that they also need to be conditionalized in WebKit/MigrateHeaders.make.

It's sad that this wasn't noticed (probably because the no-svg buildbot is always red--Bug 12402 was filed to address that).

[David Kilzer (:ddkilzer)]

Created attachment 17412 [details]
Patch v1

Possible solution (always generate these two files so that they exist on the no-svg build).

[Eric Seidel (no email)]

Comment on attachment 17412 [details]
Patch v1

I think this hack is fine if you add a comment explaining what you're doing in CodeGeneratorObjC.pm.

[David Kilzer (:ddkilzer)]

(In reply to comment #2)
> (From update of attachment 17412 [details] [edit])
> I think this hack is fine if you add a comment explaining what you're doing in
> CodeGeneratorObjC.pm.

Thanks, Eric!  I changed the code to be the comment.

Committed r27923

http://trac.webkit.org/projects/webkit/changeset/27923

[David Kilzer (:ddkilzer)]

And fixed a typo:

Committed r27924