Separate out some more ICU from base and into base/i18n....

base/file_util_posix.cc

 // Copyright (c) 2006-2008 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/file_util.h"
 
 #include <dirent.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <fnmatch.h>
@@ skipping 17 matching lines @@
 #include "base/eintr_wrapper.h"
 #include "base/file_path.h"
 #include "base/lock.h"
 #include "base/logging.h"
 #include "base/scoped_ptr.h"
 #include "base/singleton.h"
 #include "base/string_util.h"
 #include "base/sys_string_conversions.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
-#include "unicode/coll.h"
-
-
-namespace {
-
-class LocaleAwareComparator {
- public:
-  LocaleAwareComparator() {
-    UErrorCode error_code = U_ZERO_ERROR;
-    // Use the default collator. The default locale should have been properly
-    // set by the time this constructor is called.
-    collator_.reset(icu::Collator::createInstance(error_code));
-    DCHECK(U_SUCCESS(error_code));
-    // Make it case-sensitive.
-    collator_->setStrength(icu::Collator::TERTIARY);
-    // Note: We do not set UCOL_NORMALIZATION_MODE attribute. In other words, we
-    // do not pay performance penalty to guarantee sort order correctness for
-    // non-FCD (http://unicode.org/notes/tn5/#FCD) file names. This should be a
-    // reasonable tradeoff because such file names should be rare and the sort
-    // order doesn't change much anyway.
-  }
-
-  // Note: A similar function is available in l10n_util.
-  // We cannot use it because base should not depend on l10n_util.
-  // TODO(yuzo): Move some of l10n_util to base.
-  int Compare(const string16& a, const string16& b) {
-    // We are not sure if Collator::compare is thread-safe.
-    // Use an AutoLock just in case.
-    AutoLock auto_lock(lock_);
-
-    UErrorCode error_code = U_ZERO_ERROR;
-    UCollationResult result = collator_->compare(
-        static_cast<const UChar*>(a.c_str()),
-        static_cast<int>(a.length()),
-        static_cast<const UChar*>(b.c_str()),
-        static_cast<int>(b.length()),
-        error_code);
-    DCHECK(U_SUCCESS(error_code));
-    return result;
-  }
-
- private:
-  scoped_ptr<icu::Collator> collator_;
-  Lock lock_;
-  friend struct DefaultSingletonTraits<LocaleAwareComparator>;
-
-  DISALLOW_COPY_AND_ASSIGN(LocaleAwareComparator);
-};
-
-}  // namespace
 
 namespace file_util {
 
 #if defined(OS_FREEBSD) || \
     (defined(OS_MACOSX) && \
      MAC_OS_X_VERSION_MIN_REQUIRED < MAC_OS_X_VERSION_10_5)
 typedef struct stat stat_wrapper_t;
 static int CallStat(const char *path, stat_wrapper_t *sb) {
   return stat(path, sb);
 }
@@ skipping 518 matching lines @@
       return FilePath();
 
     root_path_ = pending_paths_.top();
     root_path_ = root_path_.StripTrailingSeparators();
     pending_paths_.pop();
 
     std::vector<DirectoryEntryInfo> entries;
     if (!ReadDirectory(&entries, root_path_, file_type_ & SHOW_SYM_LINKS))
       continue;
 
-    // The API says that order is not guaranteed, but order affects UX
-    std::sort(entries.begin(), entries.end(), CompareFiles);
-
     directory_entries_.clear();
     current_directory_entry_ = 0;
     for (std::vector<DirectoryEntryInfo>::const_iterator
         i = entries.begin(); i != entries.end(); ++i) {
       FilePath full_path = root_path_.Append(i->filename);
       if (ShouldSkip(full_path))
         continue;
 
       if (pattern_.value().size() &&
           fnmatch(pattern_.value().c_str(), full_path.value().c_str(),
@@ skipping 45 matching lines @@
       }
       memset(&info.stat, 0, sizeof(info.stat));
     }
     entries->push_back(info);
   }
 
   closedir(dir);
   return true;
 }
 
-bool FileEnumerator::CompareFiles(const DirectoryEntryInfo& a,
-                                  const DirectoryEntryInfo& b) {
-  // Order lexicographically with directories before other files.
-  if (S_ISDIR(a.stat.st_mode) != S_ISDIR(b.stat.st_mode))
-    return S_ISDIR(a.stat.st_mode);
-
-  // On linux, the file system encoding is not defined. We assume
-  // SysNativeMBToWide takes care of it.
-  //
-  // ICU's collator can take strings in OS native encoding. But we convert the
-  // strings to UTF-16 ourselves to ensure conversion consistency.
-  // TODO(yuzo): Perhaps we should define SysNativeMBToUTF16?
-  return Singleton<LocaleAwareComparator>()->Compare(
-      WideToUTF16(base::SysNativeMBToWide(a.filename.value().c_str())),
-      WideToUTF16(base::SysNativeMBToWide(b.filename.value().c_str()))) < 0;
-}
-
 ///////////////////////////////////////////////
 // MemoryMappedFile
 
 MemoryMappedFile::MemoryMappedFile()
     : file_(-1),
       data_(NULL),
       length_(0) {
 }
 
 bool MemoryMappedFile::MapFileToMemory(const FilePath& file_name) {
@@ skipping 24 matching lines @@
     munmap(data_, length_);
   if (file_ != -1)
     close(file_);
 
   data_ = NULL;
   length_ = 0;
   file_ = -1;
 }
 
 } // namespace file_util