Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Unified Diff: site/dev/testing/testing.md

Issue 1151153005: Add more detail on how DM checksums are done. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: site/dev/testing/testing.md
diff --git a/site/dev/testing/testing.md b/site/dev/testing/testing.md
index 1ed56235bfef04d39511971f07f7558b21fd3d0f..356c74236ba0230aa061cf0de61bfb56f04ad95e 100644
--- a/site/dev/testing/testing.md
+++ b/site/dev/testing/testing.md
@@ -127,8 +127,13 @@ can be found at dm_output/8888/image/mandrill_132x132_12x12.astc-5-subsets.png.
dm.json is used by our automated testing system, so you can ignore it if you
like. It contains a listing of each test run and a checksum of the image
-generated for that run. (Boring technical detail: it is not a checksum of the
-.png file, but rather a checksum of the raw pixels used to create that .png.)
+generated for that run.
+
+### Detail <a name="digests"></a>
+Boring technical detail: The checksum is not a checksum of the
+.png file, but rather a checksum of the raw pixels used to create that .png.
+That means it is possible for two different configurations to produce
+the same exact .png, but have their checksums differ.
Unit tests don't generally output anything but a status update when they pass.
If a test fails, DM will print out its assertion failures, both at the time
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698