drive: Move RefreshDirectory from ResourceMetadata to ChangeListLoader

chrome/browser/chromeos/drive/resource_metadata.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 "chrome/browser/chromeos/drive/resource_metadata.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "base/sys_info.h"
 #include "chrome/browser/chromeos/drive/drive.pb.h"
@@ skipping 80 matching lines @@
   DCHECK(metadata);
   DCHECK(out_file_path);
   FileError error = metadata->AddEntry(entry);
   if (error == FILE_ERROR_OK)
     *out_file_path = metadata->GetFilePath(entry.resource_id());
   return error;
 }
 
 }  // namespace
 
-std::string DirectoryFetchInfo::ToString() const {
-  return ("resource_id: " + resource_id_ +
-          ", changestamp: " + base::Int64ToString(changestamp_));
-}
-
 EntryInfoResult::EntryInfoResult() : error(FILE_ERROR_FAILED) {
 }
 
 EntryInfoResult::~EntryInfoResult() {
 }
 
 EntryInfoPairResult::EntryInfoPairResult() {
 }
 
 EntryInfoPairResult::~EntryInfoPairResult() {
@@ skipping 338 matching lines @@
 
   if (!new_parent.file_info().is_directory())
     return FILE_ERROR_NOT_A_DIRECTORY;
 
   // Remove from the old parent and add it to the new parent with the new data.
   if (!PutEntryUnderDirectory(entry))
     return FILE_ERROR_FAILED;
   return FILE_ERROR_OK;
 }
 
-void ResourceMetadata::RefreshDirectoryOnUIThread(
-    const DirectoryFetchInfo& directory_fetch_info,
-    const ResourceEntryMap& entry_map,
-    const FileMoveCallback& callback) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  DCHECK(!callback.is_null());
-
-  PostFileMoveTask(blocking_task_runner_.get(),
-                   base::Bind(&ResourceMetadata::RefreshDirectory,
-                              base::Unretained(this),
-                              directory_fetch_info,
-                              entry_map),
-                   callback);
-}
-
 void ResourceMetadata::GetChildDirectories(
     const std::string& resource_id,
     std::set<base::FilePath>* child_directories) {
   DCHECK(blocking_task_runner_->RunsTasksOnCurrentThread());
 
   std::vector<std::string> children;
   storage_->GetChildren(resource_id, &children);
   for (size_t i = 0; i < children.size(); ++i) {
     ResourceEntry entry;
     if (storage_->GetEntry(children[i], &entry) &&
@@ skipping 125 matching lines @@
   GetResourceEntryByPathOnUIThread(
       first_path,
       base::Bind(
           &ResourceMetadata::GetResourceEntryPairByPathsOnUIThreadAfterGetFirst,
           weak_ptr_factory_.GetWeakPtr(),
           first_path,
           second_path,
           callback));
 }
 
-FileError ResourceMetadata::RefreshDirectory(
-    const DirectoryFetchInfo& directory_fetch_info,
-    const ResourceEntryMap& entry_map,
-    base::FilePath* out_file_path) {
-  DCHECK(blocking_task_runner_->RunsTasksOnCurrentThread());
-  DCHECK(!directory_fetch_info.empty());
-
-  if (!EnoughDiskSpaceIsAvailableForDBOperation(storage_->directory_path()))
-    return FILE_ERROR_NO_LOCAL_SPACE;
-
-  ResourceEntry directory;
-  if (!storage_->GetEntry(directory_fetch_info.resource_id(), &directory))
-    return FILE_ERROR_NOT_FOUND;
-
-  if (!directory.file_info().is_directory())
-    return FILE_ERROR_NOT_A_DIRECTORY;
-
-  // Go through the entry map. Handle existing entries and new entries.
-  for (ResourceEntryMap::const_iterator it = entry_map.begin();
-       it != entry_map.end(); ++it) {
-    if (!EnoughDiskSpaceIsAvailableForDBOperation(storage_->directory_path()))
-      return FILE_ERROR_NO_LOCAL_SPACE;
-
-    const ResourceEntry& entry = it->second;
-    // Skip if the parent resource ID does not match. This is needed to
-    // handle entries with multiple parents. For such entries, the first
-    // parent is picked and other parents are ignored, hence some entries may
-    // have a parent resource ID which does not match the target directory's.
-    //
-    // TODO(satorux): Move the filtering logic to somewhere more appropriate.
-    // crbug.com/193525.
-    if (entry.parent_local_id() != directory_fetch_info.resource_id()) {
-      DVLOG(1) << "Wrong-parent entry rejected: " << entry.resource_id();
-      continue;
-    }
-
-    if (!PutEntryUnderDirectory(entry))
-      return FILE_ERROR_FAILED;
-  }
-
-  directory.mutable_directory_specific_info()->set_changestamp(
-      directory_fetch_info.changestamp());
-  if (!storage_->PutEntry(directory.resource_id(), directory))
-    return FILE_ERROR_FAILED;
-
-  if (out_file_path)
-    *out_file_path = GetFilePath(directory.resource_id());
-
-  return FILE_ERROR_OK;
-}
-
 void ResourceMetadata::GetResourceEntryPairByPathsOnUIThreadAfterGetFirst(
     const base::FilePath& first_path,
     const base::FilePath& second_path,
     const GetResourceEntryPairCallback& callback,
     FileError error,
     scoped_ptr<ResourceEntry> entry) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!callback.is_null());
 
   scoped_ptr<EntryInfoPairResult> result(new EntryInfoPairResult);
@@ skipping 85 matching lines @@
     for (size_t i = 0; i < children.size(); ++i) {
       if (!RemoveEntryRecursively(children[i]))
         return false;
     }
   }
   return storage_->RemoveEntry(resource_id);
 }
 
 }  // namespace internal
 }  // namespace drive