Ignore symlinks in Drag-and-drop'ed folder

webkit/fileapi/isolated_file_util.cc

 // Copyright (c) 2012 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 "webkit/fileapi/isolated_file_util.h"
 
 #include <string>
 #include <vector>
 
 #include "webkit/blob/shareable_file_reference.h"
@@ skipping 35 matching lines @@
   virtual base::Time LastModifiedTime() OVERRIDE {
     return file_info_.last_modified;
   }
 
  private:
   std::vector<FileInfo> files_;
   std::vector<FileInfo>::const_iterator file_iter_;
   base::PlatformFileInfo file_info_;
 };
 
-// Recursively enumerate each path from a given paths set.
-class RecursiveSetFileEnumerator
-    : public FileSystemFileUtil::AbstractFileEnumerator {
- public:
-  explicit RecursiveSetFileEnumerator(const std::vector<FileInfo>& files)
-      : files_(files) {
-    file_iter_ = files_.begin();
-    current_enumerator_.reset(new SetFileEnumerator(files));
-  }
-  virtual ~RecursiveSetFileEnumerator() {}
-
-  // AbstractFileEnumerator overrides.
-  virtual base::FilePath Next() OVERRIDE;
-  virtual int64 Size() OVERRIDE {
-    DCHECK(current_enumerator_.get());
-    return current_enumerator_->Size();
-  }
-  virtual bool IsDirectory() OVERRIDE {
-    DCHECK(current_enumerator_.get());
-    return current_enumerator_->IsDirectory();
-  }
-  virtual base::Time LastModifiedTime() OVERRIDE {
-    DCHECK(current_enumerator_.get());
-    return current_enumerator_->LastModifiedTime();
-  }
-
- private:
-  std::vector<FileInfo> files_;
-  std::vector<FileInfo>::iterator file_iter_;
-  scoped_ptr<FileSystemFileUtil::AbstractFileEnumerator> current_enumerator_;
-};
-
-base::FilePath RecursiveSetFileEnumerator::Next() {
-  if (current_enumerator_.get()) {
-    base::FilePath path = current_enumerator_->Next();
-    if (!path.empty())
-      return path;
-  }
-
-  // We reached the end.
-  if (file_iter_ == files_.end())
-    return base::FilePath();
-
-  // Enumerates subdirectories of the next path.
-  FileInfo& next_file = *file_iter_++;
-  current_enumerator_ = NativeFileUtil::CreateFileEnumerator(
-      next_file.path, true /* recursive */);
-  DCHECK(current_enumerator_.get());
-  return current_enumerator_->Next();
-}
-
 }  // namespace
 
 //-------------------------------------------------------------------------
 
 IsolatedFileUtil::IsolatedFileUtil() {}
 
 PlatformFileError IsolatedFileUtil::GetLocalFilePath(
     FileSystemOperationContext* context,
     const FileSystemURL& url,
     base::FilePath* local_file_path) {
@@ skipping 39 matching lines @@
   if (error == base::PLATFORM_FILE_OK)
     *platform_path = url.path();
   return error;
 }
 
 scoped_ptr<FileSystemFileUtil::AbstractFileEnumerator>
     DraggedFileUtil::CreateFileEnumerator(
         FileSystemOperationContext* context,
         const FileSystemURL& root,
         bool recursive) {
+  if (recursive) {
+    NOTREACHED() << "Recursive enumeration not supported.";
+    return scoped_ptr<FileSystemFileUtil::AbstractFileEnumerator>(
+        new EmptyFileEnumerator);
+  }
   DCHECK(root.is_valid());
   if (!root.path().empty())
-    return NativeFileUtil::CreateFileEnumerator(root.path(), recursive);
+    return LocalFileUtil::CreateFileEnumerator(context, root, recursive);
 
   // Root path case.
   std::vector<FileInfo> toplevels;
   IsolatedContext::GetInstance()->GetDraggedFileInfo(
       root.filesystem_id(), &toplevels);
-  if (!recursive) {
-    return make_scoped_ptr(new SetFileEnumerator(toplevels))
-        .PassAs<FileSystemFileUtil::AbstractFileEnumerator>();
-  }
-  return make_scoped_ptr(new RecursiveSetFileEnumerator(toplevels))
-      .PassAs<FileSystemFileUtil::AbstractFileEnumerator>();
+  return scoped_ptr<AbstractFileEnumerator>(new SetFileEnumerator(toplevels));
 }
 
 }  // namespace fileapi