[Findit] Add cracas_crash_analysis

appengine/findit/model/crash/cracas_crash_analysis.py

Index: appengine/findit/model/crash/cracas_crash_analysis.py
diff --git a/appengine/findit/model/crash/cracas_crash_analysis.py b/appengine/findit/model/crash/cracas_crash_analysis.py
new file mode 100644
index 0000000000000000000000000000000000000000..908f4da9f6910edcfb6f0afaa67ba891f8d50108
--- /dev/null
+++ b/appengine/findit/model/crash/cracas_crash_analysis.py
@@ -0,0 +1,19 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import hashlib
+import json
+
+from google.appengine.ext import ndb
+
+from model.crash.chrome_crash_analysis import ChromeCrashAnalysis
+
+
+class CracasCrashAnalysis(ChromeCrashAnalysis):
+  """Represents an analysis of a Chrome crash on Cracas."""
+
+  @staticmethod
+  def _CreateKey(crash_identifiers):
+    return ndb.Key('CracasCrashAnalysis', hashlib.sha1(

[Martin Barbella, 2016/09/19 20:24:36]

We're using hashlib.sha1(...).hexdigest() in a few places, it seems. Could you
create a helper for this?

Abstracting it a bit should make it more straightforward for new people jumping
into this code, or adding a new analysis type later. They won't need to know
it's a sha1 hash, only that some helper is creating the key we're interested in
and can check the implementation if curious.

[Sharu Jiang, 2016/09/19 22:19:06]

I moved this to the base class CrashAnalysis, because Fracas/Cracas/Clusterfuzz
analysis should use the same _CreateKey function :)

+        json.dumps(crash_identifiers, sort_keys=True)).hexdigest())