[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..58247bbb322efca9f6e5bebc120c09331df6c2be
--- /dev/null
+++ b/appengine/findit/model/crash/cracas_crash_analysis.py
@@ -0,0 +1,17 @@
+# 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.fracas_crash_analysis import FracasCrashAnalysis
+
+
+class CracasCrashAnalysis(FracasCrashAnalysis):

[Martin Barbella, 2016/09/08 21:15:39]

This doesn't seem clean to me, and I suspect even ClusterFuzz will look very
similar. Could you create a base class for these, and have the rest inherit from
it? Doing so may not even be necessary if we simply include the client in the
model and index on it.

[Sharu Jiang, 2016/09/08 22:18:52]

We already have a base class - CrashAnalysis, the inherited FracasCrashAnalysis
will only add historical_metadata and channel fields (data in customized_data),
since CracasCrashAnalysis will also add these two, so I choose make a copy here
(I do need to overwrite one method though, this would be included in the next
patch.), if duplicating code in FracasCrashAnalysis would seem more clear, I can
do that too.

+  """Represents an analysis of a Chrome crash."""
+  # TODO(katesonia): Deprecate FracasCrashAnalysis after switching from Fracas
+  # to Cracas.
+  pass