Do not call stat() when reading directories via File API.

chrome/browser/media_galleries/fileapi/itunes_file_util.cc

 // Copyright 2013 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 "chrome/browser/media_galleries/fileapi/itunes_file_util.h"
 
 #include <set>
 #include <string>
 #include <vector>
 
@@ skipping 177 matching lines @@
     storage::FileSystemOperationContext* context,
     const storage::FileSystemURL& url,
     EntryList* file_list) {
   DCHECK(file_list->empty());
   std::vector<std::string> components = GetVirtualPathComponents(url);
 
   if (components.size() == 0) {
     base::File::Info xml_info;
     if (!base::GetFileInfo(GetDataProvider()->library_path(), &xml_info))
       return base::File::FILE_ERROR_IO;
-    file_list->push_back(DirectoryEntry(kITunesLibraryXML,
-                                        DirectoryEntry::FILE,
-                                        xml_info.size, xml_info.last_modified));
-    file_list->push_back(DirectoryEntry(kITunesMediaDir,
-                                        DirectoryEntry::DIRECTORY,
-                                        0, base::Time()));
+    file_list->push_back(
+        DirectoryEntry(kITunesLibraryXML, DirectoryEntry::FILE));
+    file_list->push_back(
+        DirectoryEntry(kITunesMediaDir, DirectoryEntry::DIRECTORY));
     return base::File::FILE_OK;
   }
 
   if (components.size() == 1 && components[0] == kITunesLibraryXML)
     return base::File::FILE_ERROR_NOT_A_DIRECTORY;
 
   if (components[0] != kITunesMediaDir || components.size() > 5)
     return base::File::FILE_ERROR_NOT_FOUND;
 
   if (components.size() == 1) {
     if (!GetDataProvider()->auto_add_path().empty()) {
-      file_list->push_back(DirectoryEntry(kITunesAutoAddDir,
-                                          DirectoryEntry::DIRECTORY,
-                                          0, base::Time()));
+      file_list->push_back(
+          DirectoryEntry(kITunesAutoAddDir, DirectoryEntry::DIRECTORY));
     }
-    file_list->push_back(DirectoryEntry(kITunesMusicDir,
-                                        DirectoryEntry::DIRECTORY,
-                                        0, base::Time()));
+    file_list->push_back(
+        DirectoryEntry(kITunesMusicDir, DirectoryEntry::DIRECTORY));
     return base::File::FILE_OK;
   }
 
   if (components[1] == kITunesAutoAddDir &&
       !GetDataProvider()->auto_add_path().empty()) {
     return NativeMediaFileUtil::ReadDirectorySync(context, url, file_list);
   }
 
   if (components[1] != kITunesMusicDir)
     return base::File::FILE_ERROR_NOT_FOUND;
 
   if (components.size() == 2) {
     std::set<ITunesDataProvider::ArtistName> artists =
         GetDataProvider()->GetArtistNames();
     std::set<ITunesDataProvider::ArtistName>::const_iterator it;
     for (it = artists.begin(); it != artists.end(); ++it)
-      file_list->push_back(DirectoryEntry(*it, DirectoryEntry::DIRECTORY,
-                                          0, base::Time()));
+      file_list->push_back(DirectoryEntry(*it, DirectoryEntry::DIRECTORY));
     return base::File::FILE_OK;
   }
 
   if (components.size() == 3) {
     std::set<ITunesDataProvider::AlbumName> albums =
         GetDataProvider()->GetAlbumNames(components[2]);
     if (albums.size() == 0)
       return base::File::FILE_ERROR_NOT_FOUND;
     std::set<ITunesDataProvider::AlbumName>::const_iterator it;
     for (it = albums.begin(); it != albums.end(); ++it)
-      file_list->push_back(DirectoryEntry(*it, DirectoryEntry::DIRECTORY,
-                                          0, base::Time()));
+      file_list->push_back(DirectoryEntry(*it, DirectoryEntry::DIRECTORY));
     return base::File::FILE_OK;
   }
 
   if (components.size() == 4) {
     ITunesDataProvider::Album album =
         GetDataProvider()->GetAlbum(components[2], components[3]);
     if (album.size() == 0)
       return base::File::FILE_ERROR_NOT_FOUND;
     ITunesDataProvider::Album::const_iterator it;
     for (it = album.begin(); it != album.end(); ++it) {
       base::File::Info file_info;
       if (media_path_filter()->Match(it->second) &&
           base::GetFileInfo(it->second, &file_info)) {
-        file_list->push_back(DirectoryEntry(it->first, DirectoryEntry::FILE,
-                                            file_info.size,
-                                            file_info.last_modified));
+        file_list->push_back(DirectoryEntry(it->first, DirectoryEntry::FILE));
       }
     }
     return base::File::FILE_OK;
   }
 
   // At this point, the only choice is one of two errors, but figuring out
   // which one is required.
   DCHECK_EQ(4UL, components.size());
   base::FilePath location;
   location = GetDataProvider()->GetTrackLocation(components[1], components[2],
@@ skipping 135 matching lines @@
                                                             callback);
 }
 
 ITunesDataProvider* ITunesFileUtil::GetDataProvider() {
   if (!imported_registry_)
     imported_registry_ = ImportedMediaGalleryRegistry::GetInstance();
   return imported_registry_->ITunesDataProvider();
 }
 
 }  // namespace itunes