Deflake images/cross-fade-invalidation.html

Deflake images/cross-fade-invalidation.html

The test was using a 200ms timeout in order to allow the SVG
image to animate (which should only take 10ms). I suspect that
load time was impacting things, so try moving the test logic to
window.onload and reduce the timeout to 100ms. It may be that
the SVG is not starting at all in some cases, which would be
a more serious issue.

TBR=fmalita@chromium.org
BUG=665776
Committed: https://crrev.com/5beba54f865425e7e25f598505d72315afd9ca15
Cr-Commit-Position: refs/heads/master@{#433201}

[Stephen Chennney, 2016-11-17 22:07:33]

The CQ bit was checked by schenney@chromium.org

[commit-bot: I haz the power, 2016-11-17 22:09:29]

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-11-18 00:06:31]

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

[commit-bot: I haz the power, 2016-11-18 00:06:32]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Stephen Chennney, 2016-11-18 14:24:54]

Description was changed from

==========
Deflake images/cross-fade-invalidation.html

The test was using an arbitrary 200ms timeout for some reason.
Try switching to runAfterLayoutAndPaint.

TBR=fmalita@chromium.org
BUG=665776
==========

to

==========
Deflake images/cross-fade-invalidation.html

The test was using a 200ms timeout in order to allow the SVG
image to animate (which should only take 10ms). I suspect that
load time was impacting things, so try moving the test logic to
window.onload and reduce the timeout to 100ms. It may be that
the SVG is not starting at all in some cases, which would be
a more serious issue.

TBR=fmalita@chromium.org
BUG=665776
==========

[Stephen Chennney, 2016-11-18 14:24:58]

The CQ bit was checked by schenney@chromium.org

[commit-bot: I haz the power, 2016-11-18 14:25:17]

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-11-18 15:33:16]

Description was changed from

==========
Deflake images/cross-fade-invalidation.html

The test was using a 200ms timeout in order to allow the SVG
image to animate (which should only take 10ms). I suspect that
load time was impacting things, so try moving the test logic to
window.onload and reduce the timeout to 100ms. It may be that
the SVG is not starting at all in some cases, which would be
a more serious issue.

TBR=fmalita@chromium.org
BUG=665776
==========

to

==========
Deflake images/cross-fade-invalidation.html

The test was using a 200ms timeout in order to allow the SVG
image to animate (which should only take 10ms). I suspect that
load time was impacting things, so try moving the test logic to
window.onload and reduce the timeout to 100ms. It may be that
the SVG is not starting at all in some cases, which would be
a more serious issue.

TBR=fmalita@chromium.org
BUG=665776
==========

[commit-bot: I haz the power, 2016-11-18 15:33:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-18 15:35:47]

Description was changed from

==========
Deflake images/cross-fade-invalidation.html

The test was using a 200ms timeout in order to allow the SVG
image to animate (which should only take 10ms). I suspect that
load time was impacting things, so try moving the test logic to
window.onload and reduce the timeout to 100ms. It may be that
the SVG is not starting at all in some cases, which would be
a more serious issue.

TBR=fmalita@chromium.org
BUG=665776
==========

to

==========
Deflake images/cross-fade-invalidation.html

The test was using a 200ms timeout in order to allow the SVG
image to animate (which should only take 10ms). I suspect that
load time was impacting things, so try moving the test logic to
window.onload and reduce the timeout to 100ms. It may be that
the SVG is not starting at all in some cases, which would be
a more serious issue.

TBR=fmalita@chromium.org
BUG=665776

Committed: https://crrev.com/5beba54f865425e7e25f598505d72315afd9ca15
Cr-Commit-Position: refs/heads/master@{#433201}
==========

[commit-bot: I haz the power, 2016-11-18 15:35:48]

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