Acknowledge invalidations from DriveFileSyncService.

Acknowledge invalidations from DriveFileSyncService.

In r186079, local invalidations and acknowledgements were fully
implemented. This means that if an InvalidationHandler doesn't
acknowledge invalidations that it receives, it will receive wake up
pings periodically afterwards.

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

[msw, 2013-04-09 18:28:29]

https://codereview.chromium.org/13913018/diff/1/chrome/browser/sync_file_syst...
File chrome/browser/sync_file_system/drive_file_sync_service.cc (right):

https://codereview.chromium.org/13913018/diff/1/chrome/browser/sync_file_syst...
chrome/browser/sync_file_system/drive_file_sync_service.cc:344: // TODO(msw):
Unit tests do not provide profiles; see http://crbug.com/122183.
Please do not add a TODO here for me. Did you look at the bug? It's about
dropping GMock from ChromeToMobile testing.

https://codereview.chromium.org/13913018/diff/1/chrome/browser/sync_file_syst...
chrome/browser/sync_file_system/drive_file_sync_service.cc:2389: // TODO(msw):
Unit tests do not provide profiles; see http://crbug.com/122183.
ditto.

[dcheng, 2013-04-09 18:32:04]

Oops, I should have looked at the actual bug. Sorry.

https://codereview.chromium.org/13913018/diff/1/chrome/browser/sync_file_syst...
File chrome/browser/sync_file_system/drive_file_sync_service.cc (right):

https://codereview.chromium.org/13913018/diff/1/chrome/browser/sync_file_syst...
chrome/browser/sync_file_system/drive_file_sync_service.cc:344: // TODO(msw):
Unit tests do not provide profiles; see http://crbug.com/122183.
On 2013/04/09 18:28:29, msw wrote:
> Please do not add a TODO here for me. Did you look at the bug? It's about
> dropping GMock from ChromeToMobile testing.

Done.

https://codereview.chromium.org/13913018/diff/1/chrome/browser/sync_file_syst...
chrome/browser/sync_file_system/drive_file_sync_service.cc:2389: // TODO(msw):
Unit tests do not provide profiles; see http://crbug.com/122183.
On 2013/04/09 18:28:29, msw wrote:
> ditto.

Done.

[kinuko, 2013-04-10 01:49:26]

lgtm

[commit-bot: I haz the power, 2013-04-10 15:37:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dcheng@chromium.org/13913018/5001

[commit-bot: I haz the power, 2013-04-10 17:35:38]

Change committed as 193410