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

Unified Diff: chrome/app/chrome_crash_reporter_client_win.cc

Issue 2487783002: Make Crashpad use the user data dir, rather than always default location (Closed)
Patch Set: install_static for swarming Created 4 years, 1 month 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_crash_reporter_client_win.cc
diff --git a/chrome/app/chrome_crash_reporter_client_win.cc b/chrome/app/chrome_crash_reporter_client_win.cc
index ac089d5373a24ceac726b49b35c4d72ef104e7ea..b44fc84ac5f80e00baf5f73307ae71a5636714bd 100644
--- a/chrome/app/chrome_crash_reporter_client_win.cc
+++ b/chrome/app/chrome_crash_reporter_client_win.cc
@@ -19,6 +19,7 @@
#include "chrome/common/chrome_result_codes.h"
#include "chrome/install_static/install_details.h"
#include "chrome/install_static/install_util.h"
+#include "chrome/install_static/user_data_dir.h"
#include "components/crash/content/app/crashpad.h"
#include "components/crash/core/common/crash_keys.h"
@@ -252,12 +253,12 @@ void ChromeCrashReporterClient::InitializeCrashReportingForProcess() {
instance = new ChromeCrashReporterClient();
ANNOTATE_LEAKING_OBJECT_PTR(instance);
- std::string process_type = install_static::GetSwitchValueFromCommandLine(
- ::GetCommandLineA(), install_static::kProcessType);
+ std::wstring process_type = install_static::GetSwitchValueFromCommandLine(
+ ::GetCommandLine(), install_static::kProcessType);
if (process_type != install_static::kCrashpadHandler) {
crash_reporter::SetCrashReporterClient(instance);
- crash_reporter::InitializeCrashpadWithEmbeddedHandler(process_type.empty(),
- process_type);
+ crash_reporter::InitializeCrashpadWithEmbeddedHandler(
+ process_type.empty(), install_static::UTF16ToUTF8(process_type));
}
}
#endif // NACL_WIN64
@@ -370,14 +371,13 @@ bool ChromeCrashReporterClient::GetCrashDumpLocation(
if (GetAlternativeCrashDumpLocation(crash_dir))
return true;
- // TODO(scottmg): Consider supporting --user-data-dir. See
- // https://crbug.com/565446.
- return install_static::GetDefaultCrashDumpLocation(crash_dir);
+ *crash_dir = install_static::GetCrashDumpLocation();
+ return true;
}
bool ChromeCrashReporterClient::GetCrashMetricsLocation(
base::string16* metrics_dir) {
- return install_static::GetDefaultUserDataDirectory(metrics_dir);
+ return install_static::GetUserDataDirectory(metrics_dir, nullptr);
}
// TODO(ananta)
@@ -401,7 +401,7 @@ bool ChromeCrashReporterClient::GetCollectStatsInSample() {
bool ChromeCrashReporterClient::EnableBreakpadForProcess(
const std::string& process_type) {
- return process_type == install_static::kRendererProcess ||
- process_type == install_static::kPpapiPluginProcess ||
- process_type == install_static::kGpuProcess;
+ // This is not used by Crashpad (at least on Windows).
+ NOTREACHED();
+ return true;
}

Powered by Google App Engine
This is Rietveld 408576698