[Chromium] Android needs to swizzle pixel dumps in order to get the same hash as other platforms
Created attachment 141057 [details] Patch
Comment on attachment 141057 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=141057&action=review > Tools/DumpRenderTree/chromium/TestShell.cpp:710 > +#if OS(ANDROID) > + webkit_support::EncodeRGBAPNGWithChecksum(reinterpret_cast<const unsigned char*>(sourceBitmap.getPixels()), sourceBitmap.width(), > + sourceBitmap.height(), static_cast<int>(sourceBitmap.rowBytes()), discardTransparency, md5hash, &png); > +#else > webkit_support::EncodeBGRAPNGWithChecksum(reinterpret_cast<const unsigned char*>(sourceBitmap.getPixels()), sourceBitmap.width(), > sourceBitmap.height(), static_cast<int>(sourceBitmap.rowBytes()), discardTransparency, md5hash, &png); > +#endif Did you just add 5 lines of code, or 4 charaters or am blind?
Comment on attachment 141057 [details] Patch This bug looks a dupe of Bug 84682. Anyway, the code is ok.
*** Bug 84682 has been marked as a duplicate of this bug. ***
Comment on attachment 141057 [details] Patch Clearing flags on attachment: 141057 Committed r116588: <http://trac.webkit.org/changeset/116588>
All reviewed patches have been landed. Closing bug.