[Retry][SyncFS] Implement LargestKnownChangeID handling

[Retry][SyncFS] Implement LargestKnownChangeID handling

BUG=240165
R=kinuko@chromium.org, nhiroki@chromium.org
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=235709

[tzik, 2013-11-18 06:33:48]

PTL

[kinuko, 2013-11-18 06:57:11]

rubber-stamp lgtm

https://codereview.chromium.org/62583003/diff/40001/chrome/browser/sync_file_...
File chrome/browser/sync_file_system/drive_backend/metadata_database_unittest.cc
(right):

https://codereview.chromium.org/62583003/diff/40001/chrome/browser/sync_file_...
chrome/browser/sync_file_system/drive_backend/metadata_database_unittest.cc:755:
ApplyNoopChangeToMetadata(&noop_file.metadata);
Care to add a comment why we need it?

[nhiroki, 2013-11-18 07:06:47]

lgtm

[tzik, 2013-11-18 07:17:13]

https://codereview.chromium.org/62583003/diff/40001/chrome/browser/sync_file_...
File chrome/browser/sync_file_system/drive_backend/metadata_database_unittest.cc
(right):

https://codereview.chromium.org/62583003/diff/40001/chrome/browser/sync_file_...
chrome/browser/sync_file_system/drive_backend/metadata_database_unittest.cc:755:
ApplyNoopChangeToMetadata(&noop_file.metadata);
On 2013/11/18 06:57:11, kinuko wrote:
> Care to add a comment why we need it?

Done.

[commit-bot: I haz the power, 2013-11-18 07:17:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/62583003/20003

[commit-bot: I haz the power, 2013-11-18 11:28:08]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-18 11:38:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/62583003/20003

[commit-bot: I haz the power, 2013-11-18 11:45:22]

Change committed as 235709