Rename TimedItem => ComputedTimingProperties and Timing => AnimationEffectTiming in test

Rename TimedItem => ComputedTimingProperties and Timing => AnimationEffectTiming in test

BUG=623437
Committed: https://crrev.com/1121153c958ea45a0bdadcdbb127a0b1e51daa5f
Cr-Commit-Position: refs/heads/master@{#404791}

[nainar, 2016-07-12 05:01:18]

Description was changed from

==========
just some random renames

BUG=
==========

to

==========
Rename TimedItem => ComputedTimingProperties and Timing => AnimationEffectTiming
in test

BUG=623437
==========

[nainar, 2016-07-12 05:02:07]

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

[nainar, 2016-07-12 05:02:07]

The only other test for getComputedTiming() is
https://github.com/w3c/web-platform-tests/blob/fa55abc4c96aa63a39229edd2fb469...
and it doesnt cover these cases ->hence retaining and jsut renaming

[suzyh_UTC10 (ex-contributor), 2016-07-12 05:18:06]

Can you recommend a new filename for the test and its path in the
web-platform-tests repository?

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/web-animations-api/timed-item.html (right):

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:6: <script
src="../imported/wpt/web-animations/testcommon.js"></script>
Are you using this testcommon script?

[nainar, 2016-07-12 05:30:33]

Suggested folder: interfaces/KeyframeEffect (mentioned in the sheet to now)
Suggested name: effect-of-keyframeeffect-on-getComputedTiming ?

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/web-animations-api/timed-item.html (right):

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:3: <link
rel="https://w3c.github.io/web-animations/#the-animation-interface">
Should be:
https://www.w3.org/TR/web-animations/#the-computedtimingproperties-dictionary
oops

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:6: <script
src="../imported/wpt/web-animations/testcommon.js"></script>
should I change the ones that are comparing times to assert_times_equal or
remove this?

[suzyh_UTC10 (ex-contributor), 2016-07-12 05:39:54]

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/web-animations-api/timed-item.html (right):

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:3: <link
rel="https://w3c.github.io/web-animations/#the-animation-interface">
On 2016/07/12 at 05:30:33, nainar wrote:
> Should be:
https://www.w3.org/TR/web-animations/#the-computedtimingproperties-dictionary
oops

Except from a base of https://w3c.github.io/web-animations/

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:6: <script
src="../imported/wpt/web-animations/testcommon.js"></script>
On 2016/07/12 at 05:30:33, nainar wrote:
> should I change the ones that are comparing times to assert_times_equal or
remove this?

Oh, good point. Yeah, I guess convert to use assert_times_equal.

[nainar, 2016-07-12 05:47:31]

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/web-animations-api/timed-item.html (right):

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:3: <link
rel="https://w3c.github.io/web-animations/#the-animation-interface">
Done.

https://codereview.chromium.org/2141863002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/web-animations-api/timed-item.html:6: <script
src="../imported/wpt/web-animations/testcommon.js"></script>
Done.

[suzyh_UTC10 (ex-contributor), 2016-07-12 05:49:50]

lgtm

[nainar, 2016-07-12 05:50:50]

The CQ bit was checked by nainar@chromium.org

[commit-bot: I haz the power, 2016-07-12 05:51:51]

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-07-12 07:24:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-12 07:24:07]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-12 07:25:36]

Description was changed from

==========
Rename TimedItem => ComputedTimingProperties and Timing => AnimationEffectTiming
in test

BUG=623437
==========

to

==========
Rename TimedItem => ComputedTimingProperties and Timing => AnimationEffectTiming
in test

BUG=623437

Committed: https://crrev.com/1121153c958ea45a0bdadcdbb127a0b1e51daa5f
Cr-Commit-Position: refs/heads/master@{#404791}
==========

[commit-bot: I haz the power, 2016-07-12 07:25:36]

Patchset 3 (id:??) landed as
https://crrev.com/1121153c958ea45a0bdadcdbb127a0b1e51daa5f
Cr-Commit-Position: refs/heads/master@{#404791}