IntersectionObserver: use post task inside of resume().

IntersectionObserver: use post task inside of resume().

Rather than running callbacks immediately, post a task to run the
callbacks, to avoid conflicts while undeferring a page.

R=dcheng@chromium.org
BUG=670092
Committed: https://crrev.com/389cf84d79f9dd6d2d7ad49cce34e9e38b578804
Cr-Commit-Position: refs/heads/master@{#438043}

[dcheng, 2016-12-09 23:03:11]

Change looks reasonable to me; is there any way to test it?

[szager1, 2016-12-13 01:15:26]

On 2016/12/09 23:03:11, dcheng wrote:
> Change looks reasonable to me; is there any way to test it?

Added test, PTAL.

[dcheng, 2016-12-13 01:35:37]

lgtm

https://codereview.chromium.org/2565863002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/web/tests/IntersectionObserverTest.cpp (right):

https://codereview.chromium.org/2565863002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/web/tests/IntersectionObserverTest.cpp:93:
testing::runPendingTasks();
Just to make sure I understand: is this to try to make sure there are no queued
tasks hanging around in the message loop?

[szager1, 2016-12-13 01:40:44]

https://codereview.chromium.org/2565863002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/web/tests/IntersectionObserverTest.cpp (right):

https://codereview.chromium.org/2565863002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/web/tests/IntersectionObserverTest.cpp:93:
testing::runPendingTasks();
On 2016/12/13 01:35:36, dcheng wrote:
> Just to make sure I understand: is this to try to make sure there are no
queued
> tasks hanging around in the message loop?

Correct.

[szager1, 2016-12-13 01:40:50]

The CQ bit was checked by szager@chromium.org

[commit-bot: I haz the power, 2016-12-13 01:41:56]

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-12-13 03:50:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481593250630310,
"parent_rev": "363e41ace14ed25379a266444c9e9212ae1be4fc", "commit_rev":
"d4382f7cab2bce2d8d552cb171ab9faba976e502"}

[commit-bot: I haz the power, 2016-12-13 03:50:43]

Description was changed from

==========
IntersectionObserver: use post task inside of resume().

Rather than running callbacks immediately, post a task to run the
callbacks, to avoid conflicts while undeferring a page.

R=dcheng@chromium.org
BUG=670092
==========

to

==========
IntersectionObserver: use post task inside of resume().

Rather than running callbacks immediately, post a task to run the
callbacks, to avoid conflicts while undeferring a page.

R=dcheng@chromium.org
BUG=670092

Review-Url: https://codereview.chromium.org/2565863002
==========

[commit-bot: I haz the power, 2016-12-13 03:50:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 03:54:56]

Description was changed from

==========
IntersectionObserver: use post task inside of resume().

Rather than running callbacks immediately, post a task to run the
callbacks, to avoid conflicts while undeferring a page.

R=dcheng@chromium.org
BUG=670092

Review-Url: https://codereview.chromium.org/2565863002
==========

to

==========
IntersectionObserver: use post task inside of resume().

Rather than running callbacks immediately, post a task to run the
callbacks, to avoid conflicts while undeferring a page.

R=dcheng@chromium.org
BUG=670092

Committed: https://crrev.com/389cf84d79f9dd6d2d7ad49cce34e9e38b578804
Cr-Commit-Position: refs/heads/master@{#438043}
==========

[commit-bot: I haz the power, 2016-12-13 03:54:57]

Patchset 2 (id:??) landed as
https://crrev.com/389cf84d79f9dd6d2d7ad49cce34e9e38b578804
Cr-Commit-Position: refs/heads/master@{#438043}