Bug 233213

Summary: [iOS] use PhotosUIPrivate instead of PhotosUI
Product: WebKit Reporter: Devin Rousso <hi>
Component: New BugsAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: akeerthi, benjamin, cdumez, cmarcelo, ews-watchlist, hi, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
akeerthi: review+
Patch none

Description Devin Rousso 2021-11-16 14:18:08 PST 
.
Comment 1 Devin Rousso 2021-11-16 14:18:31 PST 
<rdar://problem/85300427>
Comment 2 Devin Rousso 2021-11-16 14:19:06 PST 
Created attachment 444429 [details]
Patch
Comment 3 Devin Rousso 2021-11-16 14:44:33 PST 
Created attachment 444435 [details]
Patch
Comment 4 Aditya Keerthi 2021-11-16 14:50:25 PST 
Comment on attachment 444435 [details]
Patch

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

r=me

> Source/WebKit/Platform/spi/ios/PhotosUISPI.h:30
> +#else

I think this should be `#elif HAVE(PHOTOS_UI)`, so we don't try to import on watchOS/tvOS.
Comment 5 Devin Rousso 2021-11-16 14:58:22 PST 
Created attachment 444440 [details]
Patch
Comment 6 EWS 2021-11-16 18:39:54 PST 
Committed r285910 (244319@main): <https://commits.webkit.org/244319@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 444440 [details].