Upstream chromium's image decoders From here: http://src.chromium.org/viewvc/chrome/trunk/deps/third_party/WebKit/WebCore/platform/image-decoders/skia/ My plan is to attach two patches. The first will copy the existing PNG, JPEG, and GIF decoders. Then, I will do a subsequent change, which applies our changes as well as adds BMP, ICO, and XBM decoders.
Created attachment 26975 [details] patch: part 1 - copy existing files
Comment on attachment 26975 [details] patch: part 1 - copy existing files rs=me
Created attachment 26976 [details] patch: part 2 - copy ImageDecoder.h as well
Created attachment 26977 [details] patch: part 3 - apply skia changes on top
Comment on attachment 26976 [details] patch: part 2 - copy ImageDecoder.h as well rs=me
Comment on attachment 26977 [details] patch: part 3 - apply skia changes on top This is a bit hard to review at this size. It is possible to split out the adds into a new patch? Or somehow make this one smaller?
Comment on attachment 26977 [details] patch: part 3 - apply skia changes on top Gonna rubber stamp this. I've reviewed this already when you submitted it to our local branch. My understanding of the "grand plan" is that Hyatt was interested in replacing the shared decoders with Chromium's decoders. Peter Kasting intends to take care of that merging/replacement work.
http://trac.webkit.org/changeset/40165 http://trac.webkit.org/changeset/40166 http://trac.webkit.org/changeset/40173