Summary: | [Win][Direct2D] Clean up Bitmap Decoder Implementation | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, bfulgham, commit-queue, dino, pvollan, simon.fraser | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 162761, 163349 | ||||||
Bug Blocks: | 161817 | ||||||
Attachments: |
|
Description
Brent Fulgham
2016-10-04 18:55:10 PDT
Created attachment 291685 [details]
Cleanups to make this code less gross.
This patch implements a set of improvements Said suggested in a prior review (made after the original patch landed).
Comment on attachment 291685 [details] Cleanups to make this code less gross. Rejecting attachment 291685 [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-01', 'apply-attachment', '--no-update', '--non-interactive', 291685, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: ex: platform/graphics/win/ImageDecoderDirect2D.cpp |=================================================================== |--- platform/graphics/win/ImageDecoderDirect2D.cpp (revision 207357) |+++ platform/graphics/win/ImageDecoderDirect2D.cpp (working copy) -------------------------- No file to patch. Skipping patch. 2 out of 2 hunks ignored Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Dean Jackson']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/2287417 Comment on attachment 291685 [details] Cleanups to make this code less gross. View in context: https://bugs.webkit.org/attachment.cgi?id=291685&action=review > platform/graphics/win/ImageDecoderDirect2D.cpp:175 > + auto frameSize = frameSizeAtIndex(index, subsamplingLevel); > + return frameSize.area() * 4; Would have made this a one-liner. Landed in r207365. <https://trac.webkit.org/changeset/207365> |