Bug 225709

Summary: Plumb some more metadata to QuickLook when revealing images
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: PlatformAssignee: Wenson Hsieh <wenson_hsieh>
Status: RESOLVED FIXED    
Severity: Normal CC: drousso, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
drousso: review+
Patch for landing none

Description Wenson Hsieh 2021-05-12 11:58:50 PDT
rdar://77864866
Comment 1 Wenson Hsieh 2021-05-12 12:19:46 PDT
Created attachment 428401 [details]
Patch
Comment 2 Devin Rousso 2021-05-12 12:31:52 PDT
Comment on attachment 428401 [details]
Patch

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

r=me

> Source/WebKit/ChangeLog:26
> +        Rename the `URL` argument to `fileURL`, to avoid confusion with the new `imageURL` that's passed in. Note that
> +        this is temporary, since we'll adopt new QuickLook SPI on macOS once the fix for rdar://74299451 is available.

NIT: Should we put `FIXME` comments in the actual code for <rdar://74299451> so somebody else knows where to look if needed?
Comment 3 Wenson Hsieh 2021-05-12 12:44:12 PDT
Thanks for the review!

(In reply to Devin Rousso from comment #2)
> Comment on attachment 428401 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=428401&action=review
> 
> r=me
> 
> > Source/WebKit/ChangeLog:26
> > +        Rename the `URL` argument to `fileURL`, to avoid confusion with the new `imageURL` that's passed in. Note that
> > +        this is temporary, since we'll adopt new QuickLook SPI on macOS once the fix for rdar://74299451 is available.
> 
> NIT: Should we put `FIXME` comments in the actual code for <rdar://74299451>
> so somebody else knows where to look if needed?

Makes sense — I'll add an inline FIXME referencing this radar in `WKImageExtractionPreviewController.mm`.
Comment 4 Wenson Hsieh 2021-05-12 13:09:21 PDT
Created attachment 428406 [details]
Patch for landing
Comment 5 EWS 2021-05-12 14:08:43 PDT
Committed r277389 (237645@main): <https://commits.webkit.org/237645@main>

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