Bug 149652 – Make WebKit for iOS Simulator build with the public iOS 9 SDK

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Daniel Bates]

When using the public iOS 9 SDK, build-webkit --ios-simulator dies with many errors :( We should support building WebKit with the public iOS 9 SDK.

[Daniel Bates]

Created attachment 262123 [details]
Patch

[WebKit Commit Bot]

Attachment 262123 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/spi/cocoa/PthreadSPI.h:33:  pthread_set_fixedpriority_self is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Simon Fraser (smfr)]

Comment on attachment 262123 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=262123&action=review

> Source/WebCore/ChangeLog:12
> +        * platform/spi/cocoa/PthreadSPI.h: Added.

The uppercase P is odd, given that the header is pthread.h

> Source/WebCore/platform/spi/cocoa/AVKitSPI.h:77
> +#if __IPHONE_OS_VERSION_MIN_REQUIRED < 90000

Is this already inside PLATFORM(IOS)?

> Source/WebCore/platform/spi/cocoa/AVKitSPI.h:104
> +#if __IPHONE_OS_VERSION_MIN_REQUIRED < 90000

Ditto.

> Source/WebKit2/Platform/spi/ios/SafariServicesSPI.h:46
> +@interface SFSafariViewController : UIViewController

Isn't SFSafariViewController public?

[Daniel Bates]

(In reply to comment #3)
> Comment on attachment 262123 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=262123&action=review
> 
> > Source/WebCore/ChangeLog:12
> > +        * platform/spi/cocoa/PthreadSPI.h: Added.
> 
> The uppercase P is odd, given that the header is pthread.h
> 

Will rename file to pthreadSPI.h.

> > Source/WebCore/platform/spi/cocoa/AVKitSPI.h:77
> > +#if __IPHONE_OS_VERSION_MIN_REQUIRED < 90000
> 
> Is this already inside PLATFORM(IOS)?
> 

Yes. We are still interested in building against older iOS SDKs.


> > Source/WebCore/platform/spi/cocoa/AVKitSPI.h:104
> > +#if __IPHONE_OS_VERSION_MIN_REQUIRED < 90000
> 
> Ditto.
> 

See my remark above.

> > Source/WebKit2/Platform/spi/ios/SafariServicesSPI.h:46
> > +@interface SFSafariViewController : UIViewController
> 
> Isn't SFSafariViewController public?

Oops! Will import header <SafariServices/SFSafariViewController.h> to pick up definition of SFSafariViewController.

[Daniel Bates]

Committed r190336: <http://trac.webkit.org/changeset/190336>