Chromium Code Reviews
DescriptionRevert of [scheduler] Change TaskQueue observer call mechanism. (patchset #6 id:100001 of https://codereview.chromium.org/2808843003/ )
Reason for revert:
This looks causing failure of loading pages when launching the process. For example, twitter lite cannot be open from the home screen because the hosting service worker probably cannot run on a process.
See also https://crbug.com/725337.
Original issue's description:
> [scheduler] Change TaskQueue observer call mechanism.
>
> Instead of manually calling TaskQueue observer we can rely on TimeDomain
> calling SetScheduledTimeDomainWakeUp and we can call the observer from
> there.
>
> This patch also fixes the problem where pages were stuck due to a missing
> notification. Test coverage is improved and HasPendingImmediateWork renamed
> to HasTaskToRunImmediately to better reflect its meaning.
>
> R=alexclarke@chromium.org,skyostil@chromium.org
> BUG=699541, 710095
>
> Review-Url: https://codereview.chromium.org/2808843003
> Cr-Commit-Position: refs/heads/master@{#470957}
> Committed: https://chromium.googlesource.com/chromium/src/+/0f67a0c1b7f0e461114771b4297f5fc0a0202cee
TBR=alexclarke@chromium.org,skyostil@chromium.org,altimin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=699541, 710095
Patch Set 1 #Messages
Total messages: 6 (3 generated)
|