AU: Resume interrupted update attempts.

AU: Resume interrupted update attempts.

BUG=7390, 7520
TEST=unit tests

Change-Id: I9baf72aa444dd855409f865f03fb665e91f8d03d

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=9b23057

[petkov, 2010-10-07 21:04:17]

This change enables resuming of interrupted updates by loading the appropriate
state in DeltaPerformer and skipping the FilesystemCopier action. It's not ready
yet -- I'm sending it out early so that you can review and patch it into your
client, if needed. This change needs at least two things to be ready:

- DownloadAction needs to be constructed with an appropriate HTTP fetcher that
supports resume (dependent on your changes).
- testing

[adlr, 2010-10-08 02:23:52]

LGTM

http://codereview.chromium.org/3620013/diff/4001/5008
File update_attempter.cc (right):

http://codereview.chromium.org/3620013/diff/4001/5008#newcode260
update_attempter.cc:260: DeltaPerformer::ResetUpdateProgress(prefs_, false);
should we maybe update prefs files before telling the user to reboot?

[petkov, 2010-10-08 06:02:13]

Addressed comment, rebased and integrated with the multi fetcher. Unit tests
pass. I need to test, maybe tomorrow.

PTAL if you get a chance.

http://codereview.chromium.org/3620013/diff/4001/5008
File update_attempter.cc (right):

http://codereview.chromium.org/3620013/diff/4001/5008#newcode260
update_attempter.cc:260: DeltaPerformer::ResetUpdateProgress(prefs_, false);
On 2010/10/08 02:23:52, adlr wrote:
> should we maybe update prefs files before telling the user to reboot?

Done.

[adlr, 2010-10-08 06:25:22]

LGTM! woohoo!

[adlr, 2010-10-08 17:19:14]

LGTM

[petkov, 2010-10-08 17:21:04]

On 2010/10/08 17:19:14, adlr wrote:
> LGTM

Pushed... Let's test it now :-)