Skip to content

Conversation

@ragingwind
Copy link

i've tested this project on chrome browser on google tv(v11, logitech revue). unfortunately it had not work. so i've changed code about define property. it's just reporting for issue below.

add setter to avoid exception [1] and using defineProperty instead of defineGetter for inheritance error [2]

[1]: TypeError: Cannot set property interationTIme of par 0-NaN(0@0) [par 1-undefined (0 @-0.994)[] ] which has only a getter
[2]: can't found super object (ParGroup) if it's using defineGetter

add setter to avoid exception [1] and using defineProperty instead of __defineGetter__ to inheritance error [2]

[1]: TypeError: Cannot set property interationTIme of par 0-NaN(0@0) [par 1-undefined (0 @-0.994)[] ] which has only a getter
[2]: can't found super object (ParGroup) if it's using __defineGetter__
@ragingwind
Copy link
Author

I am sorry that I confused you with previous commit. please refer this comment.
i've removed useless and confues code. i'm trying to solve a type error [1] on chrome of google tv. the chrome raised exception [1]. i just changed declaration code of DEFAULT_GROUP, defineGetter to Object.defineProperty.

[1]: TypeError: Cannot set property interationTIme of par 0-NaN(0@0) [par
1-undefined (0 @-0.994)[] ] which has only a getter

@steveblock
Copy link
Contributor

I'm afraid I'm still not sure what the problem is. It sounds like you're having problems assigning to document.animationTimeline.iterationTime? When I try in Chrome on Linux, this does not cause an error.

Can you attach a test case? Have you repro'd this in regular Chrome, or just in Chrome for Google TV?

@ragingwind
Copy link
Author

i also don't know exactly what is problem. i've tested chrome on linux and
mac, i never got any error, it was works very well. i have a exception only
on chrome for google tv. i just would like to report to you this exception.

@steveblock
Copy link
Contributor

OK. Can you give me a repro case for the error, or at least explain what you were doing when you observed it?

@ragingwind
Copy link
Author

i tested with 'test-to-animation.html' in demo and i just logged the exception [1]

[1]:  TypeError: Cannot set property interationTIme of par 0-NaN(0@0) [par 1-undefined (0 @-0.994)[] ] which has only a getter

@dstoc
Copy link
Contributor

dstoc commented Mar 12, 2013

Things have changed significantly since this issue was logged. I tested that particular test on logitech revue and it seems to work now.

@dstoc dstoc closed this Mar 12, 2013
@ragingwind
Copy link
Author

great! thank 👍

alancutter pushed a commit to alancutter/web-animations-js-legacy that referenced this pull request Apr 6, 2016
Ensure 0 duration players get sampled at least once
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants