Group all responsive animation tests together

Group all responsive animation tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

The tests in the svg-attribute-responsive directory are also moved
under responsive/interpolation/ in order to avoid creating a copy of
the responsive-test.js with different relative path imports.

BUG=623434
Review-Url: https://codereview.chromium.org/2973013002
Cr-Commit-Position: refs/heads/master@{#485512}
Committed: https://chromium.googlesource.com/chromium/src/+/0a1a9e4d3577a94effac93c81110444f1fb9b7a4

[suzyh_UTC10 (ex-contributor), 2017-07-06 06:38:16]

Description was changed from

==========
Group all responsive tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

A copy of the resources/responsive-test.js file is made for the
svg-attribute-responsive directory, because testharness files are being
imported from relative paths, and that location has now changed for
animations/responsive/interpolation/* tests.

BUG=623434
==========

to

==========
Group all responsive animation tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

A copy of the resources/responsive-test.js file is made for the
svg-attribute-responsive directory, because testharness files are being
imported from relative paths, and that location has now changed for
animations/responsive/interpolation/* tests.

BUG=623434
==========

[suzyh_UTC10 (ex-contributor), 2017-07-06 06:38:17]

suzyh@chromium.org changed reviewers:
+ alancutter@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-07-06 06:39:12]

I'm not sure forking the resources/responsive-test.js file was the way to go,
but it seems messy either way. What do you think?

[suzyh_UTC10 (ex-contributor), 2017-07-06 06:39:21]

The CQ bit was checked by suzyh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-06 06:39:28]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[alancutter (OOO until 2018), 2017-07-06 06:44:25]

On 2017/07/06 at 06:39:12, suzyh wrote:
> I'm not sure forking the resources/responsive-test.js file was the way to go,
but it seems messy either way. What do you think?

Let's move the svg-attribute tests under the responsive/interpolation directory.

[suzyh_UTC10 (ex-contributor), 2017-07-06 07:02:36]

The CQ bit was checked by suzyh@chromium.org to run a CQ dry run

[suzyh_UTC10 (ex-contributor), 2017-07-06 07:02:43]

On 2017/07/06 at 06:44:25, alancutter wrote:
> On 2017/07/06 at 06:39:12, suzyh wrote:
> > I'm not sure forking the resources/responsive-test.js file was the way to
go, but it seems messy either way. What do you think?
> 
> Let's move the svg-attribute tests under the responsive/interpolation
directory.

Done.

[commit-bot: I haz the power, 2017-07-06 07:02:45]

Dry run: 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, 2017-07-06 08:23:49]

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

[commit-bot: I haz the power, 2017-07-06 08:23:50]

Dry run: This issue passed the CQ dry run.

[suzyh_UTC10 (ex-contributor), 2017-07-10 01:36:07]

Ping?

[suzyh_UTC10 (ex-contributor), 2017-07-10 04:35:41]

suzyh@chromium.org changed reviewers:
+ ericwilligers@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-07-10 04:35:56]

+ericwilligers since alancutter is OOO

[suzyh_UTC10 (ex-contributor), 2017-07-10 23:47:01]

The CQ bit was checked by suzyh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-10 23:47:27]

Dry run: 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, 2017-07-11 01:09:11]

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

[commit-bot: I haz the power, 2017-07-11 01:09:12]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2017-07-11 03:18:29]

Why do the following appear as "M" instead of D? 
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-d-responsive.html
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-points-responsive.html
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-tableValues-responsive.html
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-transform-responsive.html
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-x-list-responsive.html

[Eric Willigers, 2017-07-11 03:18:41]

lgtm

[suzyh_UTC10 (ex-contributor), 2017-07-11 03:20:27]

Description was changed from

==========
Group all responsive animation tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

A copy of the resources/responsive-test.js file is made for the
svg-attribute-responsive directory, because testharness files are being
imported from relative paths, and that location has now changed for
animations/responsive/interpolation/* tests.

BUG=623434
==========

to

==========
Group all responsive animation tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

The tests in the svg-attribute-responsive directory are also moved
under responsive/interpolation/ in order to avoid creating a copy of
the responsive-test.js with different relative path imports.

BUG=623434
==========

[suzyh_UTC10 (ex-contributor), 2017-07-11 03:21:47]

On 2017/07/11 at 03:18:30, ericwilligers wrote:
> Why do the following appear as "M" instead of D? 
>
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-d-responsive.html
>
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-points-responsive.html
>
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-tableValues-responsive.html
>
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-transform-responsive.html
>
third_party/WebKit/LayoutTests/animations/svg-attribute-responsive/svg-x-list-responsive.html

I do not know :(

Also I've now updated the CL description, which I failed to do after the change
in the second patchset.

[suzyh_UTC10 (ex-contributor), 2017-07-11 03:22:14]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2017-07-11 03:22:25]

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, 2017-07-11 03:29:07]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1499743334804590,
"parent_rev": "38a3c2ce990afbabf043bf71035e885460389ca6", "commit_rev":
"0a1a9e4d3577a94effac93c81110444f1fb9b7a4"}

[commit-bot: I haz the power, 2017-07-11 03:29:21]

Description was changed from

==========
Group all responsive animation tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

The tests in the svg-attribute-responsive directory are also moved
under responsive/interpolation/ in order to avoid creating a copy of
the responsive-test.js with different relative path imports.

BUG=623434
==========

to

==========
Group all responsive animation tests together

Move tests of animations being responsive to style changes under the
existing animations/responsive test directory. The tests that are
already there are testing responsiveness in interpolations, so they are
moved to an interpolation/ subdirectory.

The tests in the svg-attribute-responsive directory are also moved
under responsive/interpolation/ in order to avoid creating a copy of
the responsive-test.js with different relative path imports.

BUG=623434

Review-Url: https://codereview.chromium.org/2973013002
Cr-Commit-Position: refs/heads/master@{#485512}
Committed:
https://chromium.googlesource.com/chromium/src/+/0a1a9e4d3577a94effac93c81110...
==========

[commit-bot: I haz the power, 2017-07-11 03:29:22]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/0a1a9e4d3577a94effac93c81110...