New issue
Advanced search Search tips

Issue 641398 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Layout test svg/animations/use-animate-width-and-height.html is Slow

Project Member Reported by schenney@chromium.org, Aug 26 2016

Issue description

The test svg/animations/use-animate-width-and-height.html should be moved to Slow, as it is just plain slow.

 

Comment 1 by f...@opera.com, Aug 26 2016

This test is borked due to the fix that added the line to TestExpectations. I'll upload a potential fix.
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6a43874e66bd2f5511a09028013481d653cbf86f

commit 6a43874e66bd2f5511a09028013481d653cbf86f
Author: fs <fs@opera.com>
Date: Fri Aug 26 19:26:32 2016

Move onclick handler in svg/animations/use-animate-width-and-height.html

We can no longer listen after events on something that is an instance in
a <use>. Move the executeTest() call to the <use> element instead.

Since the mentioned file is almost a stub, the actual change is in
svg/animations/script-tests/use-animate-width-and-height.js.

BUG= 641398 

Review-Url: https://codereview.chromium.org/2283893002
Cr-Commit-Position: refs/heads/master@{#414776}

[modify] https://crrev.com/6a43874e66bd2f5511a09028013481d653cbf86f/third_party/WebKit/LayoutTests/svg/animations/script-tests/use-animate-width-and-height.js

Comment 3 by f...@opera.com, Aug 26 2016

I'll remove the expectation line.
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a4feb3540ebcfcae979ff8c9307f323f7797e11d

commit a4feb3540ebcfcae979ff8c9307f323f7797e11d
Author: fs <fs@opera.com>
Date: Fri Aug 26 20:47:27 2016

svg/animations/use-animate-width-and-height.html no longer time out

TBR=schenney@chromium.org
NOTRY=true
BUG= 641398 

Review-Url: https://codereview.chromium.org/2279263003
Cr-Commit-Position: refs/heads/master@{#414801}

[modify] https://crrev.com/a4feb3540ebcfcae979ff8c9307f323f7797e11d/third_party/WebKit/LayoutTests/TestExpectations

Comment 5 by f...@opera.com, Aug 26 2016

Status: Fixed (was: Assigned)

Sign in to add a comment