| Summary: | [chromium] Remove ImageDecoderCG.cpp from platform/image-decoders | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | noel gordon <noel.gordon> | ||||||
| Component: | New Bugs | Assignee: | noel gordon <noel.gordon> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | abarth, eric, pkasting, tpayne, webkit.review.bot, zoltan | ||||||
| Priority: | P2 | ||||||||
| Version: | 528+ (Nightly build) | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Bug Depends on: | |||||||||
| Bug Blocks: | 80400 | ||||||||
| Attachments: |
|
||||||||
|
Description
noel gordon
2012-05-14 02:18:45 PDT
Created attachment 141669 [details]
Patch
Well nothing exploded :) Perhaps I should also remove the ImageSource changes, thoughts? http://trac.webkit.org/changeset/70846/trunk/WebCore/platform/graphics/ImageSource.h Suppose I could just try removing ImageSource changes ... Created attachment 141896 [details]
Patch
Not a reviewer, but LGTM Thanks Noel! This code served us well Adam, thanks for writing it. @zoltan, this change will bump your work on bug 80400, may I continue? 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. Ok Zoltan thank you, I'll go ahead and submit. Comment on attachment 141896 [details] Patch Clearing flags on attachment: 141896 Committed r117355: <http://trac.webkit.org/changeset/117355> All reviewed patches have been landed. Closing bug. |