Use Unthrottled instead of DOMManipulation in HTMLLinkElement/HTMLStyleElement

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

[hiroshige, 2016-12-22 18:37:23]

Description was changed from

==========
[TEST] Use timerTaskRunner instead of DOMManipulation in
HTMLLinkElement::scheduleEvent()

BUG=
==========

to

==========
[TEST] Use timerTaskRunner instead of DOMManipulation in
HTMLLinkElement::scheduleEvent()

BUG=649942
==========

[hiroshige, 2017-01-18 19:37:34]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-18 19:37:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hiroshige, 2017-01-18 19:42:29]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-18 19:43:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hiroshige, 2017-01-18 19:48:26]

Description was changed from

==========
[TEST] Use timerTaskRunner instead of DOMManipulation in
HTMLLinkElement::scheduleEvent()

BUG=649942
==========

to

==========
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 posted and
executed before the popup windows become visible.

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
==========

[hiroshige, 2017-01-18 19:48:27]

hiroshige@chromium.org changed reviewers:
+ alexclarke@chromium.org, haraken@chromium.org, japhet@chromium.org

[hiroshige, 2017-01-18 19:50:03]

Description was changed from

==========
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 posted and
executed before the popup windows become visible.

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
==========

to

==========
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
==========

[hiroshige, 2017-01-18 20:08:43]

hiroshige@chromium.org changed reviewers:
- japhet@chromium.org

[commit-bot: I haz the power, 2017-01-18 21:13:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-18 21:13:50]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-01-18 22:01:35]

PTAL.

I locally tested this CL on Linux and the issue 649942 was fixed even without
https://codereview.chromium.org/2620743002/.

[haraken, 2017-01-18 23:41:34]

LGTM

[hiroshige, 2017-01-19 00:10:48]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2017-01-19 00:11:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-19 00:16:41]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484784648359250,
"parent_rev": "96b890e58809d310e0b70029913e0f72a00a4414", "commit_rev":
"812f90689e6637a5a4f13151dba661464716e29e"}

[commit-bot: I haz the power, 2017-01-19 00:17:16]

Description was changed from

==========
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
==========

to

==========
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/+/812f90689e6637a5a4f13151dba6...
==========

[commit-bot: I haz the power, 2017-01-19 00:17:17]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/812f90689e6637a5a4f13151dba6...

[hiroshige, 2017-01-19 21:48:59]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2644103005/ by hiroshige@chromium.org.

The reason for reverting is: This CL is not needed for M-57 and thus
reverting before M-57 branch cut as planned.
.