Switch IntersectionObserver to postTask.

Switch IntersectionObserver to postTask.

For our own visibility detection in the render pipeline throttling code
we need to have postTask timing instead of idle tasks. If we need that,
then web developers will too. Web developers who care about performance
will need to wrap their callbacks in requestIdleCallback.

Committed: https://crrev.com/31768c865c398d024d8adad8cd5d9f2b182d12c2
Cr-Commit-Position: refs/heads/master@{#417818}

[ojan, 2016-09-08 21:18:42]

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

[commit-bot: I haz the power, 2016-09-08 21:19:19]

Dry run: 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, 2016-09-09 00:11:44]

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

[commit-bot: I haz the power, 2016-09-09 00:11:45]

Dry run: This issue passed the CQ dry run.

[ojan, 2016-09-09 17:28:46]

Description was changed from

==========
WIP. NOT FOR COMMIT. Switch IntersectionObserver to postTask.
==========

to

==========
Switch IntersectionObserver to postTask.

For our own visibility detection in the render pipeline throttling code
we need to have postTask timing instead of idle tasks. If we need that,
then web developers will too. Web developers who care about performance
will need to wrap their callbacks in requestIdleCallback.
==========

[ojan, 2016-09-09 17:37:42]

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

[commit-bot: I haz the power, 2016-09-09 17:38:23]

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

[ojan, 2016-09-09 18:31:53]

ojan@chromium.org changed reviewers:
+ haraken@chromium.org, skyostil@chromium.org, szager@chromium.org

[ojan, 2016-09-09 18:31:54]

Not sure if I'm doing the task posting correctly in this new WeakPtr world.

[haraken, 2016-09-10 01:34:45]

LGTM

[ojan, 2016-09-10 02:19:04]

The CQ bit was checked by ojan@chromium.org

[commit-bot: I haz the power, 2016-09-10 02:19:20]

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, 2016-09-10 03:59:46]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-10 04:01:41]

Description was changed from

==========
Switch IntersectionObserver to postTask.

For our own visibility detection in the render pipeline throttling code
we need to have postTask timing instead of idle tasks. If we need that,
then web developers will too. Web developers who care about performance
will need to wrap their callbacks in requestIdleCallback.
==========

to

==========
Switch IntersectionObserver to postTask.

For our own visibility detection in the render pipeline throttling code
we need to have postTask timing instead of idle tasks. If we need that,
then web developers will too. Web developers who care about performance
will need to wrap their callbacks in requestIdleCallback.

Committed: https://crrev.com/31768c865c398d024d8adad8cd5d9f2b182d12c2
Cr-Commit-Position: refs/heads/master@{#417818}
==========

[commit-bot: I haz the power, 2016-09-10 04:01:42]

Patchset 4 (id:??) landed as
https://crrev.com/31768c865c398d024d8adad8cd5d9f2b182d12c2
Cr-Commit-Position: refs/heads/master@{#417818}

[Sami, 2016-09-12 09:45:02]

lgtm.

[szager1, 2016-09-12 18:07:32]

late lgtm