[Media Router WebUI] Run afterNextRender() before tests using media-router-container.

[Media Router WebUI] Run afterNextRender() before tests using media-router-container.

There's a race condition where afterNextRender() is not always called before setTimeout(). This causes flaky tests. All tests that use media-router-container, which uses afterNextRender() as part of its setup, should be run after an afterNextRender() call rather than setTimeout(). This will ensure setup completion, including registering listeners.

BUG=610547
Committed: https://crrev.com/6101899d9c383af3e47801dc11dc594df43555c1
Cr-Commit-Position: refs/heads/master@{#392668}

[apacible, 2016-05-10 01:09:58]

Description was changed from

==========
adf

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

to

==========
[Media Router WebUI] Run afterNextRender() before tests using
media-router-container.

There's a race condition where afterNextRender() is not always called before
setTimeout(). This causes flaky tests. All tests that use
media-router-container, which uses afterNextRender() as part of its setup,
should be run after an afterNextRender() call rather than setTimeout(). This
will ensure setup completion, including registering listeners.

BUG=610547
==========

[apacible, 2016-05-10 03:14:07]

Patchset #1 (id:1) has been deleted

[apacible, 2016-05-10 03:23:01]

apacible@chromium.org changed reviewers:
+ imcheng@chromium.org

[apacible, 2016-05-10 03:23:12]

Patchset #1 (id:20001) has been deleted

[apacible, 2016-05-10 03:23:20]

PTAL, thanks!

[imcheng, 2016-05-10 18:19:35]

lgtm

[apacible, 2016-05-10 18:28:45]

The CQ bit was checked by apacible@chromium.org

[commit-bot: I haz the power, 2016-05-10 18:29:10]

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

[commit-bot: I haz the power, 2016-05-10 19:04:41]

Description was changed from

==========
[Media Router WebUI] Run afterNextRender() before tests using
media-router-container.

There's a race condition where afterNextRender() is not always called before
setTimeout(). This causes flaky tests. All tests that use
media-router-container, which uses afterNextRender() as part of its setup,
should be run after an afterNextRender() call rather than setTimeout(). This
will ensure setup completion, including registering listeners.

BUG=610547
==========

to

==========
[Media Router WebUI] Run afterNextRender() before tests using
media-router-container.

There's a race condition where afterNextRender() is not always called before
setTimeout(). This causes flaky tests. All tests that use
media-router-container, which uses afterNextRender() as part of its setup,
should be run after an afterNextRender() call rather than setTimeout(). This
will ensure setup completion, including registering listeners.

BUG=610547
==========

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

Committed patchset #1 (id:40001)

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

Description was changed from

==========
[Media Router WebUI] Run afterNextRender() before tests using
media-router-container.

There's a race condition where afterNextRender() is not always called before
setTimeout(). This causes flaky tests. All tests that use
media-router-container, which uses afterNextRender() as part of its setup,
should be run after an afterNextRender() call rather than setTimeout(). This
will ensure setup completion, including registering listeners.

BUG=610547
==========

to

==========
[Media Router WebUI] Run afterNextRender() before tests using
media-router-container.

There's a race condition where afterNextRender() is not always called before
setTimeout(). This causes flaky tests. All tests that use
media-router-container, which uses afterNextRender() as part of its setup,
should be run after an afterNextRender() call rather than setTimeout(). This
will ensure setup completion, including registering listeners.

BUG=610547

Committed: https://crrev.com/6101899d9c383af3e47801dc11dc594df43555c1
Cr-Commit-Position: refs/heads/master@{#392668}
==========

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

Patchset 1 (id:??) landed as
https://crrev.com/6101899d9c383af3e47801dc11dc594df43555c1
Cr-Commit-Position: refs/heads/master@{#392668}