AU: Throttle notifications based on progress; still notify if slow progress.

AU: Throttle notifications based on progress; still notify if slow progress.

This switches time based notifications to monotonic TimeTicks to avoid
system clock jump issues.

BUG=5885
TEST=unit tests, gmerged on device and watched notifications

Change-Id: Ia19a7bd9c4dc206f0d793a83eff64e0562af0803

[adlr, 2010-08-20 20:17:16]

http://codereview.chromium.org/3187012/diff/2001/3001
File update_attempter.cc (right):

http://codereview.chromium.org/3187012/diff/2001/3001#newcode339
update_attempter.cc:339: bytes_received >= total ||
should this be ==? what if a server reports a smaller total size than it serves?
we don't want to spam listeners w/ updates once we cross the "total" size.

[petkov, 2010-08-20 20:29:31]

PTAL

http://codereview.chromium.org/3187012/diff/2001/3001
File update_attempter.cc (right):

http://codereview.chromium.org/3187012/diff/2001/3001#newcode339
update_attempter.cc:339: bytes_received >= total ||
On 2010/08/20 20:17:16, adlr wrote:
> should this be ==? what if a server reports a smaller total size than it
serves?
> we don't want to spam listeners w/ updates once we cross the "total" size.

Done.

You're right -- given that we have the time-based notifications still in, the
check for > is not necessary.

[adlr, 2010-08-20 21:46:25]

LGTM