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

Unified Diff: chromecast/app/linux/cast_crash_reporter_client.cc

Issue 1154383006: Adding crash utilities to chromecast/crash. (Closed) Base URL: https://eureka-internal.googlesource.com/chromium/src@master
Patch Set: cast_shell_unittests not built for android Created 5 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
Index: chromecast/app/linux/cast_crash_reporter_client.cc
diff --git a/chromecast/app/linux/cast_crash_reporter_client.cc b/chromecast/app/linux/cast_crash_reporter_client.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0bf2877057e649a8a26dff9319a41dfef57a983e
--- /dev/null
+++ b/chromecast/app/linux/cast_crash_reporter_client.cc
@@ -0,0 +1,77 @@
+// Copyright 2014 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 "chromecast/app/linux/cast_crash_reporter_client.h"
+
+#include "base/time/time.h"
+#include "chromecast/base/error_codes.h"
+#include "chromecast/crash/linux/crash_util.h"
+#include "components/crash/app/breakpad_linux.h"
+#include "content/public/common/content_switches.h"
+
+namespace chromecast {
+
+namespace {
+
+std::string* g_process_type = nullptr;
+uint64_t g_process_start_time_ms = 0u;
+
+} // namespace
+
+// static
+void CastCrashReporterClient::InitCrashReporter(
+ const std::string& process_type) {
+ DCHECK(!g_process_type);
+ g_process_start_time_ms =
+ (base::TimeTicks::Now() - base::TimeTicks()).InMilliseconds();
+
+ // Save the process type (leaked).
+ g_process_type = new std::string(process_type);
+
+ // Start up breakpad for this process, if applicable.
+ breakpad::InitCrashReporter(process_type);
+}
+
+// static
+const char* CastCrashReporterClient::GetProcessType() {
+ return g_process_type ? g_process_type->c_str() : nullptr;
+}
+
+// static
+uint64_t CastCrashReporterClient::GetProcessStartTime() {
+ return g_process_start_time_ms;
+}
+
+CastCrashReporterClient::CastCrashReporterClient() {
+}
+CastCrashReporterClient::~CastCrashReporterClient() {
+}
+
+bool CastCrashReporterClient::EnableBreakpadForProcess(
+ const std::string& process_type) {
+ return process_type == switches::kRendererProcess ||
+ process_type == switches::kZygoteProcess ||
+ process_type == switches::kGpuProcess;
+}
+
+bool CastCrashReporterClient::HandleCrashDump(const char* crashdump_filename) {
+ // Set the initial error code to ERROR_WEB_CONTENT_RENDER_VIEW_GONE to show
+ // an error message on next cast_shell run. Though the error code is for
+ // renderer process crash, the actual messages can be used for browser process
+ // as well.
+ if (!GetProcessType() || !strcmp(GetProcessType(), ""))
+ SetInitialErrorCode(ERROR_WEB_CONTENT_RENDER_VIEW_GONE);
+
+ // Upload crash dump. If user didn't opt-in crash report, minidump writer
+ // instantiated within CrashUtil::RequestUploadCrashDump() does nothing.
+ CrashUtil::RequestUploadCrashDump(crashdump_filename,
+ GetProcessType() ? GetProcessType() : "",
+ GetProcessStartTime());
+
+ // Always return true to indicate that this crash dump has been processed,
+ // so that it won't fallback to use chrome's default uploader.
+ return true;
+}
+
+} // namespace chromecast
« no previous file with comments | « chromecast/app/linux/cast_crash_reporter_client.h ('k') | chromecast/app/linux/cast_crash_reporter_client_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698