[SyncFS] Run RemoteToLocalSyncer as a background task

[SyncFS] Run RemoteToLocalSyncer as a background task

BUG=344769
Committed: https://crrev.com/638d2c191d132640597d03926d6a7735f3f85a6f
Cr-Commit-Position: refs/heads/master@{#296356}

[tzik, 2014-09-12 05:15:55]

tzik@chromium.org changed reviewers:
+ nhiroki@chromium.org

[tzik, 2014-09-12 05:15:56]

PTL

[nhiroki, 2014-09-16 08:42:27]

LGTM with nits

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
File chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc
(right):

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc:335: // 
 FileMetadata.  When it updates FileMetadata or FileDetais in FileTracker,
s/FileDetais/FileDetails/

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc:340: //
- For UnininstallAppTask, it may delete FileMetadata and FileTracker.
s/UnininstallAppTask/UninstallAppTask/

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc:376:
latest_file_metadata.details().change_id()) {
nit: You may need a 4-space indent.

[peria, 2014-09-23 12:48:28]

peria@chromium.org changed reviewers:
+ peria@chromium.org

[peria, 2014-09-23 12:48:29]

lgtm

[tzik, 2014-09-24 06:01:49]

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
File chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc
(right):

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc:335: // 
 FileMetadata.  When it updates FileMetadata or FileDetais in FileTracker,
On 2014/09/16 08:42:27, nhiroki wrote:
> s/FileDetais/FileDetails/

Done.

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc:340: //
- For UnininstallAppTask, it may delete FileMetadata and FileTracker.
On 2014/09/16 08:42:27, nhiroki wrote:
> s/UnininstallAppTask/UninstallAppTask/

Done.

https://codereview.chromium.org/554213002/diff/120001/chrome/browser/sync_fil...
chrome/browser/sync_file_system/drive_backend/remote_to_local_syncer.cc:376:
latest_file_metadata.details().change_id()) {
On 2014/09/16 08:42:27, nhiroki wrote:
> nit: You may need a 4-space indent.

Done.
I introduced local variable for them.

[tzik, 2014-09-24 06:02:13]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-09-24 06:03:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/554213002/160001

[commit-bot: I haz the power, 2014-09-24 06:38:03]

Committed patchset #9 (id:160001) as 09ef77b3f082aa0c64d50145c605a417edb27447

[commit-bot: I haz the power, 2014-09-24 06:38:41]

Patchset 9 (id:??) landed as
https://crrev.com/638d2c191d132640597d03926d6a7735f3f85a6f
Cr-Commit-Position: refs/heads/master@{#296356}