DriveFileSyncService listens to OnNetworkConnected event to restart synchronization

DriveFileSyncService listens to OnNetworkConnected event to restart synchronization


BUG=161436
TEST=manual (Launch test app -> Disconnect network -> Connect network -> Receive notification)

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

[nhiroki, 2012-12-03 05:31:17]

Could you review this? I manually checked this works well.

(This patch is based on https://codereview.chromium.org/11414253/)

Thanks!

[kinuko, 2012-12-03 06:18:48]

looking good to me

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

https://codereview.chromium.org/11419281/diff/1/chrome/browser/sync_file_syst...
chrome/browser/sync_file_system/drive_file_sync_service.cc:510: state_ =
REMOTE_SERVICE_OK;
as in OnAuthenticated() maybe we should flip this only if the service state was

REMOTE_SERVICE_TEMPORARY_UNAVAILABLE or possibly
REMOTE_SERVICE_AUTHENTICATION_REQUIRED

?

[nhiroki, 2012-12-03 06:48:53]

Updated. Thanks!

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

https://codereview.chromium.org/11419281/diff/1/chrome/browser/sync_file_syst...
chrome/browser/sync_file_system/drive_file_sync_service.cc:510: state_ =
REMOTE_SERVICE_OK;
On 2012/12/03 06:18:48, kinuko wrote:
> as in OnAuthenticated() maybe we should flip this only if the service state
was
> 
> REMOTE_SERVICE_TEMPORARY_UNAVAILABLE or possibly
> REMOTE_SERVICE_AUTHENTICATION_REQUIRED
> 
> ?

Done.

[kinuko, 2012-12-03 06:57:17]

lgtm