[Webkit-unassigned] [Bug 56286] layout test checksums should be embedded in the png as a comment

Mon Mar 14 12:49:34 PDT 2011

https://bugs.webkit.org/show_bug.cgi?id=56286

--- Comment #2 from Tony Chang <tony at chromium.org>  2011-03-14 12:49:34 PST ---
(In reply to comment #1)
> (In reply to comment #0)
> > - harder to view/manually edit the checksum, e.g. it won't be in the diff.  I'm not sure anyone ever actually does this.
> 
> I've used this to quickly determine whether chromium-mac baselines should be moved to mac-leopard (i.e. if a test fails on Mac Leopard, I've run it with --reset-results and checked whether the new checksum was the same as the one that was in the Chromium directory). However, I think I could use ImageDiff for this too.

I could also write some tools to help with this.  E.g., it seems like PrettyPatch.rb could know how to pull the checksum out of the png or another script.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.