Bug 152374

Summary: Reduce resource usage when copying image to clipboard
Product: WebKit Reporter: Jeremy Zerfas <WebKit>
Component: ImagesAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, darin
Priority: P2    
Version: WebKit Local Build   
Hardware: Mac   
OS: OS X 10.11   
Attachments:
Description Flags
Patch to reduce resource usage when copying image to clipboard. none

Jeremy Zerfas
Reported 2015-12-16 19:17:18 PST
When copying an image to the clipboard on current versions of WebKit on Mac OS X, there is an unnecessary NSImage allocation and call to TIFFRepresentation. This results in more memory and CPU time being used than is necessary (particularly for animated GIFs). I will be adding a patch for this shortly which will remove the NSImage allocation and TIFFRepresentation call. This is also somewhat related to bug 125868 and bug 131576.
Attachments
Patch to reduce resource usage when copying image to clipboard. (13.71 KB, patch)
2015-12-16 19:35 PST, Jeremy Zerfas 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jeremy Zerfas
Comment 1 2015-12-16 19:35:59 PST
Created attachment 267526 [details] Patch to reduce resource usage when copying image to clipboard.
Darin Adler
Comment 2 2015-12-16 21:27:00 PST
Comment on attachment 267526 [details] Patch to reduce resource usage when copying image to clipboard. Why do I feel like I’ve seen this patch before?
Darin Adler
Comment 3 2015-12-16 21:27:26 PST
Oh, I remember. It was in bug 125868.
WebKit Commit Bot
Comment 4 2015-12-20 20:23:35 PST
Comment on attachment 267526 [details] Patch to reduce resource usage when copying image to clipboard. Clearing flags on attachment: 267526 Committed r194326: <http://trac.webkit.org/changeset/194326>
WebKit Commit Bot
Comment 5 2015-12-20 20:23:39 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.