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

Unified Diff: gm/gm_expectations.cpp

Issue 17365002: GM: add --writeChecksumBasedFilenames option (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: sync_to_r9681 Created 7 years, 6 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 | « gm/gm_expectations.h ('k') | gm/gmmain.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gm/gm_expectations.cpp
===================================================================
--- gm/gm_expectations.cpp (revision 9681)
+++ gm/gm_expectations.cpp (working copy)
@@ -109,7 +109,24 @@
return jsonTypeValuePair;
}
+ SkString GmResultDigest::getHashType() const {
+ // TODO(epoger): The current implementation assumes that the
+ // result digest is always of type kJsonKey_Hashtype_Bitmap_64bitMD5
+ return SkString(kJsonKey_Hashtype_Bitmap_64bitMD5);
+ }
+ SkString GmResultDigest::getDigestValue() const {
+ // TODO(epoger): The current implementation assumes that the
+ // result digest is always of type kJsonKey_Hashtype_Bitmap_64bitMD5
+ //
+ // TODO(epoger): If SkString had an appendU64() method, it would be
+ // more efficient to use that here... but it doesn't.
+ SkString retval;
+ retval.printf("%lu", fHashDigest);
+ return retval;
+ }
+
+
// Expectations class...
Expectations::Expectations(bool ignoreFailure) {
« no previous file with comments | « gm/gm_expectations.h ('k') | gm/gmmain.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698