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

Unified Diff: chrome/app/chrome_exe_main_win.cc

Issue 2088133002: Switch chrome_elf exception handling from breakpad to crashpad. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments Created 4 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: chrome/app/chrome_exe_main_win.cc
diff --git a/chrome/app/chrome_exe_main_win.cc b/chrome/app/chrome_exe_main_win.cc
index 3c6e9798fb78c8967ab34ea3d2361ec01f6f7050..2a71c57d178d1b7874ac32854af7d3b5f8f8b224 100644
--- a/chrome/app/chrome_exe_main_win.cc
+++ b/chrome/app/chrome_exe_main_win.cc
@@ -42,12 +42,6 @@
namespace {
-base::LazyInstance<ChromeCrashReporterClient>::Leaky g_chrome_crash_client =
- LAZY_INSTANCE_INITIALIZER;
-
-base::LazyInstance<std::vector<crash_reporter::Report>>::Leaky g_crash_reports =
- LAZY_INSTANCE_INITIALIZER;
-
// List of switches that it's safe to rendezvous early with. Fast start should
// not be done if a command line contains a switch not in this set.
// Note this is currently stored as a list of two because it's probably faster
@@ -196,18 +190,6 @@ bool RemoveAppCompatFlagsEntry() {
} // namespace
-// This helper is looked up in the browser to retrieve the crash reports. See
-// CrashUploadListCrashpad. Note that we do not pass an std::vector here,
-// because we do not want to allocate/free in different modules. The returned
-// pointer is read-only.
-extern "C" __declspec(dllexport) void GetCrashReportsImpl(
- const crash_reporter::Report** reports,
- size_t* report_count) {
- crash_reporter::GetReports(g_crash_reports.Pointer());
- *reports = g_crash_reports.Pointer()->data();
- *report_count = g_crash_reports.Pointer()->size();
-}
-
#if !defined(WIN_CONSOLE_APP)
int APIENTRY wWinMain(HINSTANCE instance, HINSTANCE prev, wchar_t*, int) {
#else
@@ -237,10 +219,6 @@ int main() {
*base::CommandLine::ForCurrentProcess());
}
- crash_reporter::SetCrashReporterClient(g_chrome_crash_client.Pointer());
- crash_reporter::InitializeCrashpadWithEmbeddedHandler(process_type.empty(),
- process_type);
-
startup_metric_utils::RecordExeMainEntryPointTime(base::Time::Now());
// Signal Chrome Elf that Chrome has begun to start.

Powered by Google App Engine
This is Rietveld 408576698