Bug 202243 – Update availability annotations to match the macOS 10.15 and iOS 13.0 GM SDKs

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit API

[Keith Rollin]

Changed WK_MAC_TBA and WK_IOS_TBA to 10.15 and 13.0, respectively, in all declarations that appear in the GM SDKs.

[Radar WebKit Bug Importer]

<rdar://problem/55720747>

[Keith Rollin]

<rdar://problem/55633104>

[Keith Rollin]

Created attachment 379583 [details]
Patch

[Keith Rollin]

Bump.

[Keith Rollin]

Bump.

[Keith Rollin]

Comment on attachment 379583 [details]
Patch

Brent Fulgham r+'d in Radar.

[Keith Rollin]

I cq+'s too early. I need to add Brent to the ChangeLog.

[Brent Fulgham]

Comment on attachment 379583 [details]
Patch

Here! I'll r+ for real.

[Keith Rollin]

Comment on attachment 379583 [details]
Patch

Yay!

[Wenson Hsieh]

Comment on attachment 379583 [details]
Patch

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

> Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h:129
> +@property (nonatomic, setter=_setShouldAllowDesignSystemUIFonts:) BOOL _shouldAllowDesignSystemUIFonts WK_API_AVAILABLE(macos(10.15), ios(13.0));

I don’t think this one is present in 10.15/13.0 SDKs, since it landed quite recently. (That said, perhaps we should simply delete this one, per https://bugs.webkit.org/show_bug.cgi?id=203549#c4)

[WebKit Commit Bot]

Comment on attachment 379583 [details]
Patch

Rejecting attachment 379583 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 379583, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 5000 characters of output:
webkit.org/attachment.cgi?id=379583
Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Brent Fulgham']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Parsed 43 diffs from patch file(s).
patching file Source/WebKit/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Source/WebKit/Configurations/WebKit.xcconfig
Hunk #1 succeeded at 156 (offset -7 lines).
patching file Source/WebKit/Shared/API/Cocoa/WKMain.h
patching file Source/WebKit/UIProcess/API/Cocoa/NSAttributedString.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKContextMenuElementInfo.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKError.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKErrorPrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKNavigation.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKNavigationActionPrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKNavigationDelegate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKNavigationDelegatePrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKNavigationResponsePrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKPreferences.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h
Hunk #3 succeeded at 153 with fuzz 2.
Hunk #4 succeeded at 205 with fuzz 2 (offset 2 lines).
patching file Source/WebKit/UIProcess/API/Cocoa/WKPreviewActionItem.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKPreviewActionItemIdentifiers.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKPreviewElementInfo.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKProcessPoolPrivate.h
Hunk #1 FAILED at 89.
Hunk #2 FAILED at 111.
Hunk #3 succeeded at 125 with fuzz 2 (offset -1 lines).
2 out of 3 hunks FAILED -- saving rejects to file Source/WebKit/UIProcess/API/Cocoa/WKProcessPoolPrivate.h.rej
patching file Source/WebKit/UIProcess/API/Cocoa/WKSnapshotConfiguration.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKUIDelegate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKUIDelegatePrivate.h
Hunk #1 succeeded at 113 (offset 1 line).
Hunk #2 succeeded at 146 (offset 3 lines).
Hunk #3 succeeded at 201 (offset 3 lines).
patching file Source/WebKit/UIProcess/API/Cocoa/WKURLSchemeTaskPrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKWebViewPrivate.h
Hunk #2 succeeded at 139 (offset 3 lines).
Hunk #3 succeeded at 179 (offset 3 lines).
Hunk #4 succeeded at 234 (offset 3 lines).
Hunk #5 succeeded at 275 (offset 3 lines).
Hunk #6 succeeded at 288 (offset 3 lines).
Hunk #7 succeeded at 363 (offset 3 lines).
Hunk #8 succeeded at 376 (offset 3 lines).
Hunk #9 succeeded at 417 (offset 3 lines).
Hunk #10 succeeded at 432 (offset 3 lines).
Hunk #11 succeeded at 470 (offset 3 lines).
Hunk #12 succeeded at 501 (offset 3 lines).
Hunk #13 succeeded at 582 with fuzz 2 (offset 8 lines).
patching file Source/WebKit/UIProcess/API/Cocoa/WKWebpagePreferences.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataRecordPrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStorePrivate.h
Hunk #2 FAILED at 67.
1 out of 2 hunks FAILED -- saving rejects to file Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStorePrivate.h.rej
patching file Source/WebKit/UIProcess/API/Cocoa/_WKActivatedElementInfo.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKAutomationSessionDelegate.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKContentRuleListAction.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKCustomHeaderFields.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKDiagnosticLoggingDelegate.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKElementAction.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKInspectorWindow.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKProcessPoolConfiguration.h
Hunk #3 FAILED at 70.
1 out of 3 hunks FAILED -- saving rejects to file Source/WebKit/UIProcess/API/Cocoa/_WKProcessPoolConfiguration.h.rej
patching file Source/WebKit/UIProcess/API/Cocoa/_WKTextInputContext.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKUserContentExtensionStorePrivate.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.h
Hunk #1 FAILED at 33.
Hunk #2 succeeded at 71 with fuzz 1 (offset 5 lines).
1 out of 2 hunks FAILED -- saving rejects to file Source/WebKit/UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.h.rej
patching file Source/WebKit/UIProcess/API/Cocoa/_WKWebsiteDataStoreDelegate.h
patching file Source/WebKit/UIProcess/API/Cocoa/_WKWebsitePolicies.h
patching file Source/WebKit/UIProcess/_WKTouchEventGenerator.h
patching file Source/WebKit/WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextControllerPrivate.h

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Brent Fulgham']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: https://webkit-queues.webkit.org/results/13219455

[Keith Rollin]

I think I need to rebase. I'll also double-check Wenson's comment.

[Keith Rollin]

(In reply to Wenson Hsieh from comment #10)
> Comment on attachment 379583 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=379583&action=review
> 
> > Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h:129
> > +@property (nonatomic, setter=_setShouldAllowDesignSystemUIFonts:) BOOL _shouldAllowDesignSystemUIFonts WK_API_AVAILABLE(macos(10.15), ios(13.0));
> 
> I don’t think this one is present in 10.15/13.0 SDKs, since it landed quite
> recently. (That said, perhaps we should simply delete this one, per
> https://bugs.webkit.org/show_bug.cgi?id=203549#c4)

I checked in Xcode11M382v_m18A391011_m19A583_i17A878_t17J586_w17R605_b17P572_FastSim_Boost_47GB, and see it in the macOS 10.15 and iOS 13 SDKs.

[Keith Rollin]

Created attachment 382967 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 382967 [details]
Patch

Rejecting attachment 382967 [details] from commit-queue.

New failing tests:
webgl/1.0.3/conformance/extensions/webgl-draw-buffers.html
Full output: https://webkit-queues.webkit.org/results/13220896

[WebKit Commit Bot]

Created attachment 383001 [details]
Archive of layout-test-results from webkit-cq-03 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: webkit-cq-03  Port: mac-highsierra  Platform: Mac OS X 10.13.6

[WebKit Commit Bot]

Comment on attachment 382967 [details]
Patch

Clearing flags on attachment: 382967

Committed r252174: <https://trac.webkit.org/changeset/252174>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Alex Christensen]

Comment on attachment 382967 [details]
Patch

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

> Source/WebKit/UIProcess/API/Cocoa/NSAttributedString.h:46
> -    NS_SWIFT_NAME(readAccessURL) WK_API_AVAILABLE(macos(WK_MAC_TBA), ios(WK_IOS_TBA));
> +    NS_SWIFT_NAME(readAccessURL) WK_API_AVAILABLE(macos(10.15), ios(13.0));

This seems to have broken the Catalina build.

[Alex Christensen]

It only broke the incremental build.  Clean build fixes it.