New issue
Advanced search Search tips

Issue 625231 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Depopulate fast/svg/

Project Member Reported by f...@opera.com, Jul 1 2016

Issue description

This location for tests never got too popular, so there's only a few handfuls of tests here. Move them to svg/ instead.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 1 2016

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

commit 9021fe63f554128b2a26db6486e862c8fb10fc6f
Author: fs <fs@opera.com>
Date: Fri Jul 01 20:10:15 2016

Fold fast/svg/script-tests/animation-events.js into the test using it

There's only one test using this "framework", so fold it into the test
itself. Move the result to svg/animation/.

Straight copy of the .js file into the .html file. Whitespace is adjusted
and some redundant things are removed (';' and <div>.)

BUG= 625231 

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

[delete] https://crrev.com/5ab2a857673322907256218e9b5eeda80114e419/third_party/WebKit/LayoutTests/fast/svg/animation-events.html
[rename] https://crrev.com/9021fe63f554128b2a26db6486e862c8fb10fc6f/third_party/WebKit/LayoutTests/svg/animations/animation-events.html

Project Member

Comment 2 by bugdroid1@chromium.org, Jul 1 2016

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

commit 2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037
Author: fs <fs@opera.com>
Date: Fri Jul 01 20:20:02 2016

Relocate tests from fast/svg/ to svg/

Some files are renamed to avoid name clashes on case-insensitive
file-systems:

 svgangle.html  => svgangle-units.html
 svglength.html => svglength-units.html
 svglist.html   => svglist-basic-interface.html

(The last one did not actually clash, but was renamed for easier
 differentiation.)

BUG= 625231 

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

[modify] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/custom/no-fixed-in-svg-assert-expected.txt
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/custom/no-fixed-in-svg-assert.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/SVGStyleElement.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/getScreenCTM.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/getbbox.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/href-baseval-animval.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/linear-gradient-default-length.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/svgangle-units.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/svglength-units.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/svglist-basic-interface.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/dom/tabindex-focus.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/foreign-object-case-sensitivity-expected.txt
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/foreign-object-case-sensitivity.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/resources/whitespace-helper.js
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-angle-expected.txt
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-angle.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-integer-expected.txt
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-integer.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-length-expected.txt
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-length.html
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-number-expected.txt
[rename] https://crrev.com/2bd73d9eff173e4f9c1ecaf99a3c9a3a5aa40037/third_party/WebKit/LayoutTests/svg/parser/whitespace-number.html

Comment 3 by f...@opera.com, Jul 1 2016

Status: Fixed (was: Started)

Sign in to add a comment