Some usability ideas around SkTHash.

dm/DM.cpp

 #include "CrashHandler.h"
 #include "DMJsonWriter.h"
 #include "DMSrcSink.h"
 #include "DMSrcSinkAndroid.h"
 #include "OverwriteLine.h"
 #include "ProcStats.h"
 #include "SkBBHFactory.h"
 #include "SkChecksum.h"
 #include "SkCommonFlags.h"
 #include "SkForceLinking.h"
@@ skipping 86 matching lines @@
 /*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~*/
 
 struct Gold : public SkString {
     Gold() : SkString("") {}
     Gold(ImplicitString sink, ImplicitString src, ImplicitString name, ImplicitString md5)
         : SkString("") {
         this->append(sink);
         this->append(src);
         this->append(name);
         this->append(md5);
-        while (this->size() % 4) {
-            this->append("!");  // Pad out if needed so we can pass this to Murmur3.
-        }
     }
-    static uint32_t Hash(const Gold& g) {
-        return SkChecksum::Murmur3((const uint32_t*)g.c_str(), g.size());
-    }
+    static uint32_t Hash(const Gold& g) { return SkGoodHash((const SkString&)g); }
 };
 static SkTHashSet<Gold, Gold::Hash> gGold;
 
 static void add_gold(JsonWriter::BitmapResult r) {
     gGold.add(Gold(r.config, r.sourceType, r.name, r.md5));
 }
 
 static void gather_gold() {
     if (!FLAGS_readPath.isEmpty()) {
         SkString path(FLAGS_readPath[0]);
@@ skipping 518 matching lines @@
     }
     return 0;
 }
 
 #if !defined(SK_BUILD_FOR_IOS) && !defined(SK_BUILD_FOR_NACL)
 int main(int argc, char** argv) {
     SkCommandLineFlags::Parse(argc, argv);
     return dm_main();
 }
 #endif