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

Unified Diff: chromecast/shell/browser/cast_browser_main_parts.cc

Issue 620673003: Chromecast: adds crash handling for Android build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: style nits Created 6 years, 3 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/shell/browser/cast_browser_main_parts.cc
diff --git a/chromecast/shell/browser/cast_browser_main_parts.cc b/chromecast/shell/browser/cast_browser_main_parts.cc
index a8f53430c86fae699a27744064dc1c34973c53c3..a9000dc5a12f5ba3f8f2562417a22e367890c9df 100644
--- a/chromecast/shell/browser/cast_browser_main_parts.cc
+++ b/chromecast/shell/browser/cast_browser_main_parts.cc
@@ -22,6 +22,8 @@
#include "media/base/media_switches.h"
#if defined(OS_ANDROID)
+#include "chromecast/crash/android/crash_handler.h"
+#include "components/crash/browser/crash_dump_manager_android.h"
#include "net/android/network_change_notifier_factory_android.h"
#endif // defined(OS_ANDROID)
@@ -106,6 +108,16 @@ void CastBrowserMainParts::PreMainMessageLoopRun() {
content::BrowserThread::GetBlockingPool(),
ChromecastConfig::GetInstance()->pref_service(),
cast_browser_process_->browser_context()->GetRequestContext()));
+
+#if defined(OS_ANDROID)
+ base::FilePath crash_dumps_dir;
+ if (!chromecast::CrashHandler::GetCrashDumpLocation(&crash_dumps_dir)) {
+ LOG(ERROR) << "Could not find crash dump location.";
+ }
+ cast_browser_process_->SetCrashDumpManager(
+ new breakpad::CrashDumpManager(crash_dumps_dir));
+#endif
+
cast_browser_process_->SetRemoteDebuggingServer(new RemoteDebuggingServer());
InitializeWebUI();

Powered by Google App Engine
This is Rietveld 408576698