Clean up duplicate tests in web-animations-api

Clean up duplicate tests in web-animations-api

Most of the tests in the LayoutTests/web-animations-api directory have
been upstreamed to the Web Platform Tests and are now redundant with
tests in external/wpt/web-animations. This patch deletes the redundant
test files.

Test files that were added after the push to prepare the tests for
upstreaming, or that were not upstreamed due to a need for further
changes, have been moved to LayoutTests/animations to simplify the test
directory structure.

Some additional references to no-longer-existing tests in the
web-animations-api directory have been removed from FlagExpectations and
SmokeTests. The virtual/stable/web-animations-api tests have not been
touched.

BUG=623437, 591099
Review-Url: https://codereview.chromium.org/2910883002
Cr-Commit-Position: refs/heads/master@{#477224}
Committed: https://chromium.googlesource.com/chromium/src/+/c8516e444e4348c99158fa53ea43f73c8a799d29

[suzyh_UTC10 (ex-contributor), 2017-05-29 01:52:38]

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

[suzyh_UTC10 (ex-contributor), 2017-05-29 01:58:48]

Hi Alan,

I used our upstreaming spreadsheet
https://docs.google.com/spreadsheets/d/14xRMeyf6KujO0HCacnFhaKAU_RMx2lSY65Z5e...
to cross-check the test deletions, but have not updated that spreadsheet with
info about the final status of the PRs to WPT. There is a non-zero chance I've
made an error in checking that the deletions are safe, so please don't just
rubberstamp; I'd like a second pair of eyes across it.

Thanks,
Suzy

[suzyh_UTC10 (ex-contributor), 2017-05-29 01:58:55]

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

[commit-bot: I haz the power, 2017-05-29 01:59:10]

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-05-29 03:18:57]

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

[commit-bot: I haz the power, 2017-05-29 03:18:58]

Dry run: This issue passed the CQ dry run.

[alancutter (OOO until 2018), 2017-06-06 00:15:33]

On 2017/05/29 at 01:58:48, suzyh wrote:
> Hi Alan,
> 
> I used our upstreaming spreadsheet
https://docs.google.com/spreadsheets/d/14xRMeyf6KujO0HCacnFhaKAU_RMx2lSY65Z5e...
to cross-check the test deletions, but have not updated that spreadsheet with
info about the final status of the PRs to WPT. There is a non-zero chance I've
made an error in checking that the deletions are safe, so please don't just
rubberstamp; I'd like a second pair of eyes across it.
> 
> Thanks,
> Suzy

This seems to mostly line up with the current state of WPT pull requests. I
don't see fill-forward-negative-end-delay.html in the spreadsheet though, do you
remember why that one can be deleted?

[suzyh_UTC10 (ex-contributor), 2017-06-06 01:30:58]

On 2017/06/06 at 00:15:33, alancutter wrote:
> On 2017/05/29 at 01:58:48, suzyh wrote:
> > Hi Alan,
> > 
> > I used our upstreaming spreadsheet
https://docs.google.com/spreadsheets/d/14xRMeyf6KujO0HCacnFhaKAU_RMx2lSY65Z5e...
to cross-check the test deletions, but have not updated that spreadsheet with
info about the final status of the PRs to WPT. There is a non-zero chance I've
made an error in checking that the deletions are safe, so please don't just
rubberstamp; I'd like a second pair of eyes across it.
> > 
> > Thanks,
> > Suzy
> 
> This seems to mostly line up with the current state of WPT pull requests. I
don't see fill-forward-negative-end-delay.html in the spreadsheet though, do you
remember why that one can be deleted?

Oh yes, sorry. I thought it was captured by
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/external/...

[alancutter (OOO until 2018), 2017-06-06 05:34:35]

lgtm

[suzyh_UTC10 (ex-contributor), 2017-06-06 05:38:59]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2017-06-06 05:39:19]

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-06-06 05:41:41]

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

[commit-bot: I haz the power, 2017-06-06 05:41:43]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[suzyh_UTC10 (ex-contributor), 2017-06-06 05:47:33]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-06-06 05:47:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/2910883002/#ps20001
(title: "Rebase and remove one more reference to deleted test")

[commit-bot: I haz the power, 2017-06-06 05:47:54]

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-06-06 07:10:37]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496728053796280,
"parent_rev": "d27758dcb7026312da955a8ba8d01fbaf925882a", "commit_rev":
"c8516e444e4348c99158fa53ea43f73c8a799d29"}

[commit-bot: I haz the power, 2017-06-06 07:10:49]

Description was changed from

==========
Clean up duplicate tests in web-animations-api

Most of the tests in the LayoutTests/web-animations-api directory have
been upstreamed to the Web Platform Tests and are now redundant with
tests in external/wpt/web-animations. This patch deletes the redundant
test files.

Test files that were added after the push to prepare the tests for
upstreaming, or that were not upstreamed due to a need for further
changes, have been moved to LayoutTests/animations to simplify the test
directory structure.

Some additional references to no-longer-existing tests in the
web-animations-api directory have been removed from FlagExpectations and
SmokeTests. The virtual/stable/web-animations-api tests have not been
touched.

BUG=623437,591099
==========

to

==========
Clean up duplicate tests in web-animations-api

Most of the tests in the LayoutTests/web-animations-api directory have
been upstreamed to the Web Platform Tests and are now redundant with
tests in external/wpt/web-animations. This patch deletes the redundant
test files.

Test files that were added after the push to prepare the tests for
upstreaming, or that were not upstreamed due to a need for further
changes, have been moved to LayoutTests/animations to simplify the test
directory structure.

Some additional references to no-longer-existing tests in the
web-animations-api directory have been removed from FlagExpectations and
SmokeTests. The virtual/stable/web-animations-api tests have not been
touched.

BUG=623437,591099

Review-Url: https://codereview.chromium.org/2910883002
Cr-Commit-Position: refs/heads/master@{#477224}
Committed:
https://chromium.googlesource.com/chromium/src/+/c8516e444e4348c99158fa53ea43...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/c8516e444e4348c99158fa53ea43...