Commit d366a9da authored by tzik@chromium.org's avatar tzik@chromium.org

Sync FileSystem: Update metadata database even when the file is not modified.

For current impl, Resource ID of a file is not updated for this sequence:
 (1) Add a file with resource_id_1
-- wait for synced --
 (2) Delete the file
 (3) Add a file with resource_id_2, and same content with the first file.

If the (2) and (3) come together, (2) is squashed into (3). And then (3) becomes nop since the content is not modified.

As the result, the metadata of the file has wrong Resource ID and all other change is ignored.

(notry, since most of try jobs are passed, and failed try looks not related to the patch)

BUG=175667
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/12223088

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182204 0039d316-1c4b-4281-b951-d872f2087c98
parent 34bd7325
...@@ -1545,7 +1545,7 @@ void DriveFileSyncService::DidDownloadFileForRemoteSync( ...@@ -1545,7 +1545,7 @@ void DriveFileSyncService::DidDownloadFileForRemoteSync(
const std::string& md5_checksum) { const std::string& md5_checksum) {
if (error == google_apis::HTTP_NOT_MODIFIED) { if (error == google_apis::HTTP_NOT_MODIFIED) {
param->sync_action = fileapi::SYNC_ACTION_NONE; param->sync_action = fileapi::SYNC_ACTION_NONE;
CompleteRemoteSync(param.Pass(), fileapi::SYNC_STATUS_OK); DidApplyRemoteChange(param.Pass(), fileapi::SYNC_STATUS_OK);
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment