Issue 11416339: Sync FileSystem: Update polling delay when remote change-list did not update change_queue.

Issue 11416339: Sync FileSystem: Update polling delay when remote change-list did not update change_queue. (Closed)

Created:
8 years ago by tzik

Modified:
8 years ago

Reviewers:
kinuko

CC:
chromium-reviews, tzik+watch_chromium.org, kinuko+watch

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Sync FileSystem: Update polling delay when remote change-list did not update change_queue. BUG=156041 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170961

Patch Set 3 : '' #

Created: 8 years ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+17 lines, -12 lines)			Patch
	M	chrome/browser/sync_file_system/drive_file_sync_service.h	View	1 2	2 chunks	+3 lines, -1 line	0 comments	Download
	M	chrome/browser/sync_file_system/drive_file_sync_service.cc	View	1 2	6 chunks	+14 lines, -11 lines	0 comments	Download

Messages

Total messages: 5 (0 generated)

Expand Messages | Collapse Messages

kinuko

lgtm https://codereview.chromium.org/11416339/diff/2001/chrome/browser/sync_file_system/drive_file_sync_service.h File chrome/browser/sync_file_system/drive_file_sync_service.h (right): https://codereview.chromium.org/11416339/diff/2001/chrome/browser/sync_file_system/drive_file_sync_service.h#newcode283 chrome/browser/sync_file_system/drive_file_sync_service.h:283: bool AppendNewRemoteChange(const GURL& origin, Can you add a ...

8 years ago (2012-12-04 10:23:27 UTC) #2

tzik

Thanks! https://codereview.chromium.org/11416339/diff/2001/chrome/browser/sync_file_system/drive_file_sync_service.h File chrome/browser/sync_file_system/drive_file_sync_service.h (right): https://codereview.chromium.org/11416339/diff/2001/chrome/browser/sync_file_system/drive_file_sync_service.h#newcode283 chrome/browser/sync_file_system/drive_file_sync_service.h:283: bool AppendNewRemoteChange(const GURL& origin, On 2012/12/04 10:23:27, kinuko ...

8 years ago (2012-12-04 10:32:03 UTC) #3

commit-bot: I haz the power

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

8 years ago (2012-12-04 12:05:42 UTC) #4

Message was sent while issue was closed.

Change committed as 170961

Expand Messages | Collapse Messages