RESOLVED FIXED 126919
We generate DOM*.h ObjC headers for disabled features
https://bugs.webkit.org/show_bug.cgi?id=126919
Summary We generate DOM*.h ObjC headers for disabled features
Simon Fraser (smfr)
Reported 2014-01-13 11:07:01 PST
Our OS X DerivedSources has files like DOMTouch.h, DOMDeviceMotionEvent.h with fully-formed contents, even though the IDL files are using the appropriate "Conditional=" statements. This seems wrong; we should not generate these headers if the features are disabled.
Attachments
Radar WebKit Bug Importer
Comment 1 2014-01-22 10:13:05 PST
Alexey Proskuryakov
Comment 2 2014-01-22 10:17:06 PST
Generally, I would say that all ports should always generate all headers, so that we wouldn't have to #ifdef at incude sites - ifdefs would be inside the headers.
Simon Fraser (smfr)
Comment 3 2014-01-22 11:08:10 PST
(In reply to comment #2) > Generally, I would say that all ports should always generate all headers, so that we wouldn't have to #ifdef at incude sites - ifdefs would be inside the headers. But we can't put ENABLE() or USE() macros in public headers. Those switches control what features WebKit exposes, so for me it makes sense to not have headers for nonexistent features.
Alexey Proskuryakov
Comment 4 2014-01-22 11:18:48 PST
Would not copying the headers into the framework in production builds also solve this, or is that more involved?
Anders Carlsson
Comment 5 2016-08-04 14:13:44 PDT
This has been fixed now.
Note You need to log in before you can comment on or make changes to this bug.