Cleanup in {test_,}file_util_posix.cc

base/test_file_util_posix.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/test_file_util.h"
 
 #include <errno.h>
 #include <fts.h>
 #include <sys/types.h>
 #include <sys/stat.h>
@@ skipping 26 matching lines @@
   }
 
   int error = 0;
   FTSENT* ent;
   while (!error && (ent = fts_read(fts)) != NULL) {
     // ent->fts_path is the source path, including from_path, so paste
     // the suffix after from_path onto to_path to create the target_path.
     std::string suffix(&ent->fts_path[from_path.value().size()]);
     // Strip the leading '/' (if any).
     if (!suffix.empty()) {
-      DCHECK(suffix[0] == '/');
+      DCHECK_EQ('/', suffix[0]);
       suffix.erase(0, 1);
     }
     const FilePath target_path = to_path.Append(suffix);
     switch (ent->fts_info) {
       case FTS_D:  // Preorder directory.
         // Try creating the target dir, continuing on it if it exists already.
+        // Rely on the user's umask to produce correct permissions.
         if (mkdir(target_path.value().c_str(), 0777) != 0) {
           if (errno != EEXIST)
             error = errno;
         }
         break;
       case FTS_F:     // Regular file.
       case FTS_NSOK:  // File, no stat info requested.
         {
           errno = 0;
           FilePath source_path(ent->fts_path);
@@ skipping 16 matching lines @@
           error = errno;
         break;
       case FTS_DNR:  // Directory cannot be read.
       case FTS_ERR:  // Error.
       case FTS_NS:   // Stat failed.
         // Abort with the error.
         error = ent->fts_errno;
         break;
       case FTS_SL:      // Symlink.
       case FTS_SLNONE:  // Symlink with broken target.
-        LOG(WARNING) << "skipping symbolic link.";
+        LOG(WARNING) << "skipping symbolic link: " << ent->fts_path;
         continue;
       case FTS_DEFAULT:  // Some other sort of file.
-        LOG(WARNING) << "skipping weird file.";
+        LOG(WARNING) << "skipping file of unknown type: " << ent->fts_path;
         continue;
       default:
         NOTREACHED();
         continue;  // Hope for the best!
     }
   }
   // fts_read may have returned NULL and set errno to indicate an error.
   if (!error && errno != 0)
     error = errno;
 
   if (!fts_close(fts)) {
     // If we already have an error, let's use that error instead of the error
     // fts_close set.
     if (!error)
       error = errno;
   }
 
   if (error) {
     LOG(ERROR) << strerror(error);
     return false;
   }
   return true;
 }
 
 }  // namespace file_util