summaryrefslogtreecommitdiffstats
path: root/base/file_util_win.cc
diff options
context:
space:
mode:
authorbrettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-10-25 02:12:09 +0000
committerbrettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-10-25 02:12:09 +0000
commitda7d86e75fe4a37ef12f3ae4bd28b043932d2aa0 (patch)
tree5e8bfb0503ab08a672054fa34e04ef7cce3d0d2f /base/file_util_win.cc
parent6c293a7baba7aa1d998e2680492246f239a7015d (diff)
downloadchromium_src-da7d86e75fe4a37ef12f3ae4bd28b043932d2aa0.zip
chromium_src-da7d86e75fe4a37ef12f3ae4bd28b043932d2aa0.tar.gz
chromium_src-da7d86e75fe4a37ef12f3ae4bd28b043932d2aa0.tar.bz2
Replace most LOG/CHECK statements with DLOG/DCHECK statements in base.
I tried hard not to change CHECKs that had side effects. I kept fatal checks that seemed security or debugging-info (in crash reports) sensitive, and ones that seems particularly well-conceived. Review URL: http://codereview.chromium.org/8368009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107042 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/file_util_win.cc')
-rw-r--r--base/file_util_win.cc28
1 files changed, 14 insertions, 14 deletions
diff --git a/base/file_util_win.cc b/base/file_util_win.cc
index 19e15fc..8d9fbdef 100644
--- a/base/file_util_win.cc
+++ b/base/file_util_win.cc
@@ -44,7 +44,7 @@ bool DevicePathToDriveLetterPath(const FilePath& device_path,
const int kDriveMappingSize = 1024;
wchar_t drive_mapping[kDriveMappingSize] = {'\0'};
if (!::GetLogicalDriveStrings(kDriveMappingSize - 1, drive_mapping)) {
- LOG(ERROR) << "Failed to get drive mapping.";
+ DLOG(ERROR) << "Failed to get drive mapping.";
return false;
}
@@ -603,13 +603,13 @@ bool CreateTemporaryFileInDir(const FilePath& dir,
wchar_t temp_name[MAX_PATH + 1];
if (!GetTempFileName(dir.value().c_str(), L"", 0, temp_name)) {
- PLOG(WARNING) << "Failed to get temporary file name in " << dir.value();
+ DPLOG(WARNING) << "Failed to get temporary file name in " << dir.value();
return false;
}
DWORD path_len = GetLongPathName(temp_name, temp_name, MAX_PATH);
if (path_len > MAX_PATH + 1 || path_len == 0) {
- PLOG(WARNING) << "Failed to get long path name for " << temp_name;
+ DPLOG(WARNING) << "Failed to get long path name for " << temp_name;
return false;
}
@@ -667,8 +667,8 @@ bool CreateDirectory(const FilePath& full_path) {
<< "directory already exists.";
return true;
}
- LOG(WARNING) << "CreateDirectory(" << full_path_str << "), "
- << "conflicts with existing file.";
+ DLOG(WARNING) << "CreateDirectory(" << full_path_str << "), "
+ << "conflicts with existing file.";
return false;
}
@@ -695,8 +695,8 @@ bool CreateDirectory(const FilePath& full_path) {
// race to create the same directory.
return true;
} else {
- LOG(WARNING) << "Failed to create directory " << full_path_str
- << ", last error is " << error_code << ".";
+ DLOG(WARNING) << "Failed to create directory " << full_path_str
+ << ", last error is " << error_code << ".";
return false;
}
} else {
@@ -773,8 +773,8 @@ int WriteFile(const FilePath& filename, const char* data, int size) {
0,
NULL));
if (!file) {
- LOG(WARNING) << "CreateFile failed for path " << filename.value()
- << " error code=" << GetLastError();
+ DLOG(WARNING) << "CreateFile failed for path " << filename.value()
+ << " error code=" << GetLastError();
return -1;
}
@@ -785,12 +785,12 @@ int WriteFile(const FilePath& filename, const char* data, int size) {
if (!result) {
// WriteFile failed.
- LOG(WARNING) << "writing file " << filename.value()
- << " failed, error code=" << GetLastError();
+ DLOG(WARNING) << "writing file " << filename.value()
+ << " failed, error code=" << GetLastError();
} else {
// Didn't write all the bytes.
- LOG(WARNING) << "wrote" << written << " bytes to " <<
- filename.value() << " expected " << size;
+ DLOG(WARNING) << "wrote" << written << " bytes to "
+ << filename.value() << " expected " << size;
}
return -1;
}
@@ -967,7 +967,7 @@ bool MemoryMappedFile::InitializeAsImageSection(const FilePath& file_name) {
NULL, NULL);
if (file_ == base::kInvalidPlatformFileValue) {
- LOG(ERROR) << "Couldn't open " << file_name.value();
+ DLOG(ERROR) << "Couldn't open " << file_name.value();
return false;
}