Bug 155980

Summary: Move the CG ImageDecoder class from ImageSourceCG.cpp to ImageDecoderCG.h and .cpp
Product: WebKit Reporter: Said Abou-Hallawa <sabouhallawa>
Component: ImagesAssignee: Said Abou-Hallawa <sabouhallawa>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Said Abou-Hallawa
Reported 2016-03-29 11:29:28 PDT
This is a clean up step after r198782.
Attachments
Patch (47.95 KB, patch)
2016-03-29 11:32 PDT, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Patch (47.60 KB, patch)
2016-03-29 16:02 PDT, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Patch (47.53 KB, patch)
2016-03-29 16:55 PDT, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Patch (47.68 KB, patch)
2016-03-30 10:39 PDT, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Said Abou-Hallawa
Comment 1 2016-03-29 11:32:20 PDT
Created attachment 275113 [details] Patch
Said Abou-Hallawa
Comment 2 2016-03-29 16:02:39 PDT
Created attachment 275143 [details] Patch
Said Abou-Hallawa
Comment 3 2016-03-29 16:55:16 PDT
Created attachment 275152 [details] Patch
Simon Fraser (smfr)
Comment 4 2016-03-29 18:07:34 PDT
Comment on attachment 275152 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=275152&action=review > Source/WebCore/platform/graphics/cg/ImageDecoderCG.h:55 > + IntSize frameSizeAtIndex(size_t, SubsamplingLevel) const; Original or subsampled size? Maybe we should have typedefs for those.
Said Abou-Hallawa
Comment 5 2016-03-30 10:39:53 PDT
Created attachment 275200 [details] Patch
WebKit Commit Bot
Comment 6 2016-03-30 12:10:16 PDT
Comment on attachment 275200 [details] Patch Clearing flags on attachment: 275200 Committed r198849: <http://trac.webkit.org/changeset/198849>
WebKit Commit Bot
Comment 7 2016-03-30 12:10:19 PDT
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.