Chromium Code Reviews
DescriptionRevert of Use Unthrottled instead of DOMManipulation in HTMLLinkElement/HTMLStyleElement (patchset #3 id:40001 of https://codereview.chromium.org/2596343002/ )
Reason for revert:
This CL is not needed for M-57 and thus
reverting before M-57 branch cut as planned.
Original issue's description:
> Use Unthrottled instead of DOMManipulation in HTMLLinkElement/HTMLStyleElement
>
> The async tasks for HTMLLinkElement/HTMLStyleElement::dispatchPendingEvent()
> are on the critical paths to show popup windows for extensions, but
> the DOMManipulation task runner is throttled because the tasks are executed
> before the popup windows become visible, causing delay in showing popups.
>
> The scheduler is fixed by [1], but [1] is not merged to M-56.
> This CL makes a mergeable workaround by using TaskType::Unthrottled for these
> tasks to fix the issue on M-56.
>
> [1] https://codereview.chromium.org/2620743002
>
> BUG=649942
>
> Review-Url: https://codereview.chromium.org/2596343002
> Cr-Commit-Position: refs/heads/master@{#444559}
> Committed: https://chromium.googlesource.com/chromium/src/+/812f90689e6637a5a4f13151dba661464716e29e
TBR=haraken@chromium.org,alexclarke@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=649942
Review-Url: https://codereview.chromium.org/2644103005
Cr-Commit-Position: refs/heads/master@{#444851}
Committed: https://chromium.googlesource.com/chromium/src/+/1a7fcf622020ce0685a1a81a227d6b67650f8251
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|
||||||||||||||||||||||||||||