Initial addition of blimp crash client code.

blimp/engine/app/blimp_engine_crash_reporter_client.cc

+// 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.
+
+#include "blimp/engine/app/blimp_engine_crash_reporter_client.h"
+
+#include "blimp/engine/app/blimp_engine_crash_keys.h"
+#include "components/crash/core/common/crash_keys.h"
+#include "components/version_info/version_info_values.h"
+#include "content/public/common/content_switches.h"
+
+namespace blimp {
+namespace engine {
+
+BlimpEngineCrashReporterClient::BlimpEngineCrashReporterClient() {}
+BlimpEngineCrashReporterClient::~BlimpEngineCrashReporterClient() {}
+
+void BlimpEngineCrashReporterClient::SetCrashReporterClientIdFromGUID(
+    const std::string& client_guid) {
+  ::crash_keys::SetMetricsClientIdFromGUID(client_guid);
+}
+
+void BlimpEngineCrashReporterClient::GetProductNameAndVersion(
+    const char** product_name,
+    const char** version) {
+  *product_name = "Chrome_Blimp_Engine";
+  *version = PRODUCT_VERSION;
+}
+
+size_t BlimpEngineCrashReporterClient::RegisterCrashKeys() {
+  return RegisterEngineCrashKeys();
+}
+
+bool BlimpEngineCrashReporterClient::IsRunningUnattended() {
+  // If this returns "true," crash reports will not be uploaded. For now the
+  // engine will not be running unattended. Eventually when automated testing
+  // harnesses are set up, this should be changed to return "true" in those
+  // cases.
+  return false;
+}
+
+bool BlimpEngineCrashReporterClient::GetCollectStatsConsent() {
+  // Always collect blimp engine crash reports.

[Kevin M, 2016/03/24 20:14:32]

nit: Blimp

[marcinjb, 2016/03/24 20:27:03]

Done.

+  return true;
+}
+
+bool BlimpEngineCrashReporterClient::EnableBreakpadForProcess(
+    const std::string& process_type) {
+  return process_type == ::switches::kRendererProcess ||
+         process_type == ::switches::kPluginProcess ||
+         process_type == ::switches::kPpapiPluginProcess ||

[Kevin M, 2016/03/24 20:14:32]

Are we doing PPAPI?

[marcinjb, 2016/03/24 20:27:03]

¯\_(ツ)_/¯

More seriously, there's no major downside to including this here. If we're not,
this is just one boolean value that won't trip.  If we decide further down the
road that we will, then no changes will be necessary.

If you feel strongly, I can easily remove this line.

+         process_type == ::switches::kZygoteProcess ||
+         process_type == ::switches::kGpuProcess;
+}
+
+}  // namespace engine
+}  // namespace blimp