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 b44fc84ac5f80e00baf5f73307ae71a5636714bd..ac089d5373a24ceac726b49b35c4d72ef104e7ea 100644 |
--- a/chrome/app/chrome_crash_reporter_client_win.cc |
+++ b/chrome/app/chrome_crash_reporter_client_win.cc |
@@ -19,7 +19,6 @@ |
#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" |
@@ -253,12 +252,12 @@ |
instance = new ChromeCrashReporterClient(); |
ANNOTATE_LEAKING_OBJECT_PTR(instance); |
- std::wstring process_type = install_static::GetSwitchValueFromCommandLine( |
- ::GetCommandLine(), install_static::kProcessType); |
+ std::string process_type = install_static::GetSwitchValueFromCommandLine( |
+ ::GetCommandLineA(), install_static::kProcessType); |
if (process_type != install_static::kCrashpadHandler) { |
crash_reporter::SetCrashReporterClient(instance); |
- crash_reporter::InitializeCrashpadWithEmbeddedHandler( |
- process_type.empty(), install_static::UTF16ToUTF8(process_type)); |
+ crash_reporter::InitializeCrashpadWithEmbeddedHandler(process_type.empty(), |
+ process_type); |
} |
} |
#endif // NACL_WIN64 |
@@ -371,13 +370,14 @@ |
if (GetAlternativeCrashDumpLocation(crash_dir)) |
return true; |
- *crash_dir = install_static::GetCrashDumpLocation(); |
- return true; |
+ // TODO(scottmg): Consider supporting --user-data-dir. See |
+ // https://crbug.com/565446. |
+ return install_static::GetDefaultCrashDumpLocation(crash_dir); |
} |
bool ChromeCrashReporterClient::GetCrashMetricsLocation( |
base::string16* metrics_dir) { |
- return install_static::GetUserDataDirectory(metrics_dir, nullptr); |
+ return install_static::GetDefaultUserDataDirectory(metrics_dir); |
} |
// TODO(ananta) |
@@ -401,7 +401,7 @@ |
bool ChromeCrashReporterClient::EnableBreakpadForProcess( |
const std::string& process_type) { |
- // This is not used by Crashpad (at least on Windows). |
- NOTREACHED(); |
- return true; |
-} |
+ return process_type == install_static::kRendererProcess || |
+ process_type == install_static::kPpapiPluginProcess || |
+ process_type == install_static::kGpuProcess; |
+} |