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

Unified Diff: chrome/tools/crash_service/main.cc

Issue 271099: Re-try r29078: Remove some deprecated file_util wstring functions. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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
« no previous file with comments | « chrome/test/mini_installer_test/run_all_unittests.cc ('k') | chrome_frame/chrome_launcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/tools/crash_service/main.cc
===================================================================
--- chrome/tools/crash_service/main.cc (revision 29088)
+++ chrome/tools/crash_service/main.cc (working copy)
@@ -17,11 +17,11 @@
const wchar_t kStandardLogFile[] = L"operation_log.txt";
-bool GetCrashServiceDirectory(std::wstring* dir) {
- std::wstring temp_dir;
+bool GetCrashServiceDirectory(FilePath* dir) {
+ FilePath temp_dir;
if (!file_util::GetTempDir(&temp_dir))
return false;
- file_util::AppendToPath(&temp_dir, L"chrome_crashes");
+ temp_dir = temp_dir.Append(L"chrome_crashes");
if (!file_util::PathExists(temp_dir)) {
if (!file_util::CreateDirectory(temp_dir))
return false;
@@ -40,19 +40,18 @@
CommandLine::Init(0, NULL);
// We use/create a directory under the user's temp folder, for logging.
- std::wstring operating_dir;
+ FilePath operating_dir;
GetCrashServiceDirectory(&operating_dir);
- std::wstring log_file(operating_dir);
- file_util::AppendToPath(&log_file, kStandardLogFile);
+ FilePath log_file = operating_dir.Append(kStandardLogFile);
// Logging to a file with pid, tid and timestamp.
- logging::InitLogging(log_file.c_str(), logging::LOG_ONLY_TO_FILE,
+ logging::InitLogging(log_file.value().c_str(), logging::LOG_ONLY_TO_FILE,
logging::LOCK_LOG_FILE, logging::APPEND_TO_OLD_LOG_FILE);
logging::SetLogItems(true, true, true, false);
LOG(INFO) << "session start. cmdline is [" << cmd_line << "]";
- CrashService crash_service(operating_dir);
+ CrashService crash_service(operating_dir.ToWStringHack());
if (!crash_service.Initialize(::GetCommandLineW()))
return 1;
« no previous file with comments | « chrome/test/mini_installer_test/run_all_unittests.cc ('k') | chrome_frame/chrome_launcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698