RESOLVED FIXED 86346
[chromium] Remove ImageDecoderCG.cpp from platform/image-decoders 
https://bugs.webkit.org/show_bug.cgi?id=86346
Summary [chromium] Remove ImageDecoderCG.cpp from platform/image-decoders
noel gordon
Reported 2012-05-14 02:18:45 PDT
[chromium] Remove ImageDecoderCG.cpp from platform/image-decoders
Attachments
Patch (9.19 KB, patch)
2012-05-14 02:20 PDT, noel gordon 		no flags
DetailsFormatted DiffDiff
Patch (9.94 KB, patch)
2012-05-15 02:12 PDT, noel gordon 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
noel gordon
Comment 1 2012-05-14 02:20:54 PDT
Created attachment 141669 [details] Patch
noel gordon
Comment 2 2012-05-14 21:17:39 PDT
Well nothing exploded :) Perhaps I should also remove the ImageSource changes, thoughts? http://trac.webkit.org/changeset/70846/trunk/WebCore/platform/graphics/ImageSource.h
noel gordon
Comment 3 2012-05-15 02:11:32 PDT
Suppose I could just try removing ImageSource changes ...
noel gordon
Comment 4 2012-05-15 02:12:29 PDT
Created attachment 141896 [details] Patch
Peter Kasting
Comment 5 2012-05-15 11:14:53 PDT
Not a reviewer, but LGTM
Adam Barth
Comment 6 2012-05-15 11:20:01 PDT
Thanks Noel!
noel gordon
Comment 7 2012-05-15 16:08:12 PDT
This code served us well Adam, thanks for writing it. @zoltan, this change will bump your work on bug 80400, may I continue?
Zoltan Horvath
Comment 8 2012-05-16 10:03:09 PDT
Thanks for your notice Noel! It does not affect too much on my actual patch, so let's check it in. I'll update my (bug #80400) patch later, since I have to find a solution for jpeglib/pnglib qt-mac dependency.
noel gordon
Comment 9 2012-05-16 14:51:44 PDT
Ok Zoltan thank you, I'll go ahead and submit.
WebKit Review Bot
Comment 10 2012-05-16 15:36:54 PDT
Comment on attachment 141896 [details] Patch Clearing flags on attachment: 141896 Committed r117355: <http://trac.webkit.org/changeset/117355>
WebKit Review Bot
Comment 11 2012-05-16 15:36:59 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.