WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2014-01-22 10:13:05 PST
<
rdar://problem/15881908
>
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.
Top of Page
Format For Printing
XML
Clone This Bug