DescriptionRevert of [cherrypick] Fix computation of runtime for throttled tasks (patchset #2 id:20001 of https://codereview.chromium.org/1730153005/ )
Reason for revert:
The reason for reverting is: Speculative revert for Compile failure on Win, Mac
& Precise 64 Beta Bots (crbug.com/590286)
Original issue's description:
> [cherrypick] Fix computation of runtime for throttled tasks
>
> The computation of the delayed task runtime was computing the (virtual)
> TimeDomain's now + delay. That doesn't take into account the skew
> between the real time and the virtual time, which meant timers could
> run sooner than expected for background tabs. Timers associated with
> foreground tabs are unaffected.
>
> This patch makes the computation of delayed runtime the responsibility
> of the TimeDomain and introduces a ThrottledTimeDomain which computes
> the delayed runtime as realtime + delay.
>
> This means timers will no longer run sooner than expected. Under normal
> circumstancs this only timers whose delay modulo 1000ms is less than 100
> ms are less likely to be affected. Timers whose whose delay modulo
> 1000ms is greater than 900 are highly likely to be affected (there's a
> good chance their delay was too small before).
>
> BUG=587074
>
> Review URL: https://codereview.chromium.org/1718233002
>
> Cr-Commit-Position: refs/heads/master@{#376993}
> (cherry picked from commit b56bbe4ce3e7276a130a00d83cf00e5672cd6be1)
>
> R=skyostil@chromium.org
>
> Committed: https://chromium.googlesource.com/chromium/src/+/fd563254490ae93382cb88a1e9a1c1e54815c97b
TBR=skyostil@chromium.org,alexclarke@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=587074
Patch Set 1 #Messages
Total messages: 6 (3 generated)
|