The WebContent process no longer has access to the PostScript conversion service, so there is no need to keep this code.
<rdar://88172449>
Created attachment 450411 [details] Patch
Comment on attachment 450411 [details] Patch r=me
Comment on attachment 450411 [details] Patch Thanks for reviewing!
Committed r288825 (246595@main): <https://commits.webkit.org/246595@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 450411 [details].
Reopening to attach new patch.
Created attachment 450472 [details] Patch
Comment on attachment 450472 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=450472&action=review > Source/WebCore/loader/cache/CachedImage.cpp:530 > if (!isPostScriptResource()) I expect there’s a slight bit more simplification we can do here
Created attachment 450550 [details] Patch
(In reply to Tim Horton from comment #8) > Comment on attachment 450472 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=450472&action=review > > > Source/WebCore/loader/cache/CachedImage.cpp:530 > > if (!isPostScriptResource()) > > I expect there’s a slight bit more simplification we can do here Ah, good catch! I updated the patch. Thanks for reviewing!
Committed r288922 (246657@main): <https://commits.webkit.org/246657@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 450550 [details].
(In reply to EWS from comment #11) > Committed r288922 (246657@main): <https://commits.webkit.org/246657@main> > > All reviewed patches have been landed. Closing bug and clearing flags on > attachment 450550 [details]. Per Arne just removed my very first patch for WebKit for Bug 3527 that landed in r12679! Bug 3527: Allow Safari to open postscript files in browser windows as well <https://bugs.webkit.org/show_bug.cgi?id=3527> <https://commits.webkit.org/r12679>