Address review comments for 8d0d999

Change-Id: I697a0768d992ffa4ee35dded191960e4adbd69cf
Reviewed-on: https://chromium-review.googlesource.com/922728
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Joshua Peraza <jperaza@chromium.org>
This commit is contained in:
Joshua Peraza 2018-02-15 12:20:48 -08:00 committed by Commit Bot
parent 90cde8e30f
commit 4094c2628d

View File

@ -156,6 +156,8 @@ class ScopedLockFile {
private: private:
ScopedRemoveFile lock_file_; ScopedRemoveFile lock_file_;
DISALLOW_COPY_AND_ASSIGN(ScopedLockFile);
}; };
} // namespace } // namespace
@ -273,8 +275,8 @@ bool CrashReportDatabaseGeneric::Initialize(const base::FilePath& path,
return false; return false;
} }
for (size_t i = 0; i < arraysize(kReportDirectories); ++i) { for (const base::FilePath::CharType* subdir : kReportDirectories) {
if (!LoggingCreateDirectory(base_dir_.Append(kReportDirectories[i]), if (!LoggingCreateDirectory(base_dir_.Append(subdir),
FilePermissions::kOwnerOnly, FilePermissions::kOwnerOnly,
true)) { true)) {
return false; return false;