DescriptionRevert of Prevent redundant DoWorks due to canceled delayed tasks (patchset #6 id:100001 of https://codereview.chromium.org/2320403003/ )
Reason for revert:
Suspect that this CL caused breakage in Gmail and Hangouts per http://crbug.com/647484 .
Original issue's description:
> Prevent redundant DoWorks due to canceled delayed tasks
>
> To achieve this we make a few changes:
>
> 1. We only register the next wakeup with the TimeDomain, rather than all
> of them.
> 2. MoveReadyDelayedTasksToDelayedWorkQueue now registers the next
> wakeup (if any). Since it removes all canceled delayed tasks from the
> front of the priority queue this has the effect of not scheduling
> wakeups for cancelled tasks.
> 3. Tweaking the TaskQueueManager level delayed DoWork de-duplication
> logic to only post a delayed DoWork if the task is meant to run before
> any previously registered delayed DoWorks.
>
> BUG=638542, 605718
>
> Committed: https://crrev.com/929cbb9f92b5570867c3842c80778243db81a013
> Cr-Commit-Position: refs/heads/master@{#418556}
TBR=skyostil@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=638542, 605718
Committed: https://crrev.com/04e1e5cab53931ea11d58a578a237d6a5762a054
Cr-Commit-Position: refs/heads/master@{#419542}
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|