RESOLVED FIXED 121216
[mac] Public header WKBase.h imports a private header, WKBaseMac.h 
https://bugs.webkit.org/show_bug.cgi?id=121216
Summary [mac] Public header WKBase.h imports a private header, WKBaseMac.h
mitz
Reported 2013-09-11 23:55:22 PDT
With a fix for bug 121215 in place, WebKit2 fails the include consistency check because WKBase.h is a public header, yet it include WKBaseMac.h, which is a private header.
Attachments
Possible fix (69.51 KB, patch)
2013-09-13 01:24 PDT, mitz 		no flags
DetailsFormatted DiffDiff
Move public parts of WKBase.h into new public headers, make WKBase.h private (72.73 KB, patch)
2013-09-14 00:58 PDT, mitz 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2013-09-12 00:01:46 PDT
<rdar://problem/14973091>
mitz
Comment 2 2013-09-13 01:24:04 PDT
Created attachment 211519 [details] Possible fix
Build Bot
Comment 3 2013-09-13 14:07:32 PDT
Comment on attachment 211519 [details] Possible fix Attachment 211519 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1817149
mitz
Comment 4 2013-09-13 14:36:30 PDT
(In reply to comment #3) > (From update of attachment 211519 [details]) > Attachment 211519 [details] did not pass mac-ews (mac): > Output: http://webkit-queues.appspot.com/results/1817149 This is just due to a stale WKBase.h in Headers in addition to the one newly installed in PrivateHeaders.
mitz
Comment 5 2013-09-14 00:58:05 PDT
Created attachment 211632 [details] Move public parts of WKBase.h into new public headers, make WKBase.h private
mitz
Comment 6 2013-09-14 10:06:43 PDT
Fixed in <http://trac.webkit.org/r155770>. I had to add #includes in two or three more headers before landing so that WebKit2 clients will build.
Note You need to log in before you can comment on or make changes to this bug.