drive: Remove MD5 argument from FileCache::MarkDirty

chrome/browser/chromeos/drive/file_cache.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/file_cache.h"
 
 #include <vector>
 
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
@@ skipping 372 matching lines @@
 
   base::PostTaskAndReplyWithResult(
       blocking_task_runner_.get(),
       FROM_HERE,
       base::Bind(
           &FileCache::MarkAsUnmounted, base::Unretained(this), file_path),
       callback);
 }
 
 void FileCache::MarkDirtyOnUIThread(const std::string& resource_id,
-                                    const std::string& md5,
                                     const FileOperationCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!callback.is_null());
 
   base::PostTaskAndReplyWithResult(
       blocking_task_runner_.get(),
       FROM_HERE,
-      base::Bind(
-          &FileCache::MarkDirty, base::Unretained(this), resource_id, md5),
+      base::Bind(&FileCache::MarkDirty, base::Unretained(this), resource_id),
       callback);
 }
 
-FileError FileCache::MarkDirty(const std::string& resource_id,
-                               const std::string& md5) {
+FileError FileCache::MarkDirty(const std::string& resource_id) {
   AssertOnSequencedWorkerPool();
 
-  // If file has already been marked dirty in previous instance of chrome, we
-  // would have lost the md5 info during cache initialization, because the file
-  // would have been renamed to .local extension.
-  // So, search for entry in cache without comparing md5.
-
   // Marking a file dirty means its entry and actual file blob must exist in
   // cache.
   FileCacheEntry cache_entry;
-  if (!GetCacheEntry(resource_id, md5, &cache_entry) ||
+  if (!storage_->GetCacheEntry(resource_id, &cache_entry) ||
       !cache_entry.is_present()) {
-    LOG(WARNING) << "Can't mark dirty a file that wasn't cached: res_id="
-                 << resource_id
-                 << ", md5=" << md5;
+    LOG(WARNING) << "Can't mark dirty a file that wasn't cached: "
+                 << resource_id;
     return FILE_ERROR_NOT_FOUND;
   }
 
   if (cache_entry.is_dirty())
     return FILE_ERROR_OK;
 
   // Now that file operations have completed, update metadata.
   cache_entry.set_is_dirty(true);
   return storage_->PutCacheEntry(resource_id, cache_entry) ?
       FILE_ERROR_OK : FILE_ERROR_FAILED;
@@ skipping 299 matching lines @@
                                     false,  // not recursive
                                     base::FileEnumerator::FILES);
     for (base::FilePath current = enumerator.Next(); !current.empty();
          current = enumerator.Next())
       base::Move(current, current.RemoveExtension());
   }
 }
 
 }  // namespace internal
 }  // namespace drive