Convert IntersectionObserver tests to use testRunner.runIdleTasks.

Convert IntersectionObserver tests to use testRunner.runIdleTasks.

BUG=606491
R=ojan@chromium.org,skyostil@chromium.org

Committed: https://crrev.com/923374c640e686109bf0bbfe917fa95f66f0d00a
Cr-Commit-Position: refs/heads/master@{#394711}

[Sami, 2016-05-18 10:07:23]

lgtm (thanks for the great comment too.)

https://codereview.chromium.org/1983383002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/intersection-observer/iframe-cross-origin.html
(right):

https://codereview.chromium.org/1983383002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/intersection-observer/iframe-cross-origin.html:20:
})
nit: missing ';' (in other copy too)

I guess there's no easy way to avoid having this copy of the function?

[szager1, 2016-05-18 21:25:28]

https://codereview.chromium.org/1983383002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/intersection-observer/iframe-cross-origin.html
(right):

https://codereview.chromium.org/1983383002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/intersection-observer/iframe-cross-origin.html:20:
})
On 2016/05/18 10:07:23, Sami wrote:
> nit: missing ';' (in other copy too)

Done.
 
> I guess there's no easy way to avoid having this copy of the function?

I went ahead and move helper-functions.js into LayoutTests/resources, so it can
be used from http/tests, so no more redundant copies of that stuff.

[szager1, 2016-05-19 05:31:36]

The CQ bit was checked by szager@chromium.org

[szager1, 2016-05-19 05:31:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/1983383002/#ps20001
(title: "Move helper-functions.js into resources")

[commit-bot: I haz the power, 2016-05-19 05:32:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1983383002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1983383002/20001

[commit-bot: I haz the power, 2016-05-19 05:42:51]

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

[commit-bot: I haz the power, 2016-05-19 05:42:52]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[szager1, 2016-05-19 06:30:38]

The CQ bit was checked by szager@chromium.org

[szager1, 2016-05-19 06:30:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/1983383002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-19 06:31:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1983383002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1983383002/40001

[commit-bot: I haz the power, 2016-05-19 08:06:52]

Committed patchset #3 (id:40001)

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

Description was changed from

==========
Convert IntersectionObserver tests to use testRunner.runIdleTasks.

BUG=606491
R=ojan@chromium.org,skyostil@chromium.org
==========

to

==========
Convert IntersectionObserver tests to use testRunner.runIdleTasks.

BUG=606491
R=ojan@chromium.org,skyostil@chromium.org

Committed: https://crrev.com/923374c640e686109bf0bbfe917fa95f66f0d00a
Cr-Commit-Position: refs/heads/master@{#394711}
==========

[commit-bot: I haz the power, 2016-05-19 08:08:22]

Patchset 3 (id:??) landed as
https://crrev.com/923374c640e686109bf0bbfe917fa95f66f0d00a
Cr-Commit-Position: refs/heads/master@{#394711}