Following the patch for bug #139652, WTF failed to build with the public iOS SDK since the patch depends on the existence of header AppSupport/CPAggregateDictionary.h, which does not exist in the public iOS SDK. We should create a SPI wrapper header for CPAggregateDictionary.h that includes AppSupport/CPAggregateDictionary.h and forward declares the applicable SPI functions when building with the Apple Internal SDK and public iOS SDK, respectively.
Created attachment 243377 [details] Patch
Comment on attachment 243377 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=243377&action=review > Source/WTF/wtf/ios/FeatureCounter.mm:29 > #if PLATFORM(IOS) Why cannot we just and "&& USE(APPLE_INTERNAL_SDK)" here?
(In reply to comment #2) > Comment on attachment 243377 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=243377&action=review > > > Source/WTF/wtf/ios/FeatureCounter.mm:29 > > #if PLATFORM(IOS) > > Why cannot we just and "&& USE(APPLE_INTERNAL_SDK)" here? We can. I assumed we wanted to count features in WebKit built with the public SDK.
Comment on attachment 243377 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=243377&action=review >>> Source/WTF/wtf/ios/FeatureCounter.mm:29 >>> #if PLATFORM(IOS) >> >> Why cannot we just and "&& USE(APPLE_INTERNAL_SDK)" here? > > We can. I assumed we wanted to count features in WebKit built with the public SDK. I don't believe we want to do feature counting for anything else than Safari / MobileSafari at the moment.
Created attachment 243384 [details] Patch
Comment on attachment 243384 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=243384&action=review > Source/WTF/ChangeLog:9 > + as we only interested in counting features in builds of iOS WebKit that were built we only => we are only
Comment on attachment 243384 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=243384&action=review > Source/WTF/wtf/ios/FeatureCounter.mm:29 > +#if PLATFORM(IOS) && USE(APPLE_INTERNAL_SDK) The #endif comment should be updated. We should also update the #if check in FeatureCounter.cpp otherwise we are going to get linking errors with the public SDK on iOS
Created attachment 243387 [details] Patch
Comment on attachment 243387 [details] Patch r=me, thanks.
Comment on attachment 243387 [details] Patch Clearing flags on attachment: 243387 Committed r177394: <http://trac.webkit.org/changeset/177394>
All reviewed patches have been landed. Closing bug.