106632 2013-01-10 21:42:05 -0800 getComputedStyle on animation from pixels to percent returns null or asserts 2022-08-30 02:23:06 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED CONFIGURATION CHANGED HasReduction P2 Normal --- 29084 1 esprehn mikelawther ahmad.saleem792 ap dino eric esprehn johnlarsene menard mikelawther mithro payoroj929 simon.fraser tabatkins oldest_to_newest 804953 0 182263 esprehn 2013-01-10 21:42:05 -0800 Created attachment 182263 Reduction If you're transitioning on a length property from a pixel value to a percent and you call getComputedStyle to check the current value we return null because you end up in CSSPrimitiveValue::CSSPrimitiveValue(const Length& length) with the type Calculated which hits an ASSERT_NOT_REACHED(); In a release build we return null because of error handling in CSSComputedStyleDeclaration. 805557 1 tabatkins 2013-01-11 13:17:41 -0800 +ccing Mike Lawther, as he's the calc() author. 830654 2 mithro 2013-02-12 00:38:01 -0800 This appears to actually be related to Bug 29084, top/bottom/left/right all seem to work differently and hence behave as this bug reports. Width works as expected by this test. 839187 3 mithro 2013-02-21 20:48:55 -0800 This issue is fixed by the patch found in Bug 29084. 1894753 6 ahmad.saleem792 2022-08-30 02:23:06 -0700 I am not able to reproduce this bug in Safari 15.6.1 and it shows as intended all values (though it shows animation is 60 fps compared to other browser, which show 120 fps) but since all other browsers (Chrome Canary 107 and Firefox Nightly 106) also work. I am going to mark this as "RESOLVED CONFIGURATION CHANGED". Thanks! 182263 2013-01-10 21:42:05 -0800 2013-01-10 21:42:05 -0800 Reduction Reduction.html text/html 1831 esprehn PCFET0NUWVBFIGh0bWw+CgkKPHN0eWxlPgogICAgZGl2IHsKICAgICAgICB3aWR0aDogMTUwcHg7 CiAgICAgICAgaGVpZ2h0OiAxNTBweDsKICAgICAgICBwb3NpdGlvbjogcmVsYXRpdmU7CiAgICAg ICAgYm9yZGVyOiAxcHggc29saWQgcmVkOwogICAgfQoKICAgIHNwYW4gewogICAgICAgIHBvc2l0 aW9uOiBhYnNvbHV0ZTsKICAgICAgICAvKiBNdXN0IHN0YXJ0IGZyb20gbm9uLXplcm8uICovCiAg ICAgICAgdG9wOiAxMHB4OwogICAgICAgIGxlZnQ6IDA7CiAgICAgICAgd2lkdGg6IDIwcHg7CiAg ICAgICAgaGVpZ2h0OiAyMHB4OwogICAgICAgIGJhY2tncm91bmQ6IHJlZDsKICAgIH0KCiAgICAu bW92ZSBzcGFuIHsKICAgICAgICAvKiBNdXN0IHRyYW5zaXRpb24gdG8gYSBwZXJjZW50LiAqLwog ICAgICAgIHRvcDogMTAwJTsKICAgICAgICAtd2Via2l0LXRyYW5zaXRpb246IHRvcCAxcyBsaW5l YXIgMC4wMXM7CiAgICAgICAgLW1vei10cmFuc2l0aW9uOiB0b3AgMXMgbGluZWFyIDAuMDFzOwog ICAgICAgIHRyYW5zaXRpb246IHRvcCAxcyBsaW5lYXIgMC4wMXM7CiAgICB9Cjwvc3R5bGU+Cgo8 c2NyaXB0PgogICAgb25sb2FkID0gZnVuY3Rpb24oKSB7CiAgICAgICAgdmFyIGRpdiA9IGRvY3Vt ZW50LnF1ZXJ5U2VsZWN0b3IoJ2RpdicpOwogICAgICAgIHZhciBvbCA9IGRvY3VtZW50LnF1ZXJ5 U2VsZWN0b3IoJ29sJyk7CiAgICAgICAgZGl2LmNsYXNzTmFtZSA9ICdtb3ZlJzsKCiAgICAgICAg dmFyIG9sZFZhbHVlOwoKICAgICAgICB2YXIgcmVxdWVzdEFuaW1hdGlvbkZyYW1lID0gd2luZG93 LndlYmtpdFJlcXVlc3RBbmltYXRpb25GcmFtZSB8fCAKICAgICAgICAgICAgd2luZG93Lm1velJl cXVlc3RBbmltYXRpb25GcmFtZSB8fCAKICAgICAgICAgICAgd2luZG93LnJlcXVlc3RBbmltYXRp b25GcmFtZTsKCiAgICAgICAgZnVuY3Rpb24gbG9nKCkgewogICAgICAgICAgICAvKgogICAgICAg ICAgICAgICAgVGhpcyBzaG91bGQgcmV0dXJuIHRoZSB0b3AgdmFsdWVzIGFzIHdlIHRyYW5zaXRp b24gYnV0CiAgICAgICAgICAgICAgICBpbnN0ZWFkIGl0IGdldHMgbnVsbCBvciBhc3NlcnRzIGlu IGEgZGVidWcgYnVpbGQgaW4KICAgICAgICAgICAgICAgIENTU1ByaW1pdGl2ZVZhbHVlOjpDU1NQ cmltaXRpdmVWYWx1ZShjb25zdCBMZW5ndGgmIGxlbmd0aCkKICAgICAgICAgICAgICAgIGJlY2F1 c2UgbGVuZ3RoLnR5cGUoKSBpcyBDYWxjdWxhdGVkLgogICAgICAgICAgICAqLwogICAgICAgICAg ICB2YXIgdG9wID0gZ2V0Q29tcHV0ZWRTdHlsZShkaXYuZmlyc3RFbGVtZW50Q2hpbGQpLmdldFBy b3BlcnR5VmFsdWUoJ3RvcCcpOwogICAgICAgICAgICBpZiAob2xkVmFsdWUgIT0gdG9wKSB7CiAg ICAgICAgICAgICAgICBvbGRWYWx1ZSA9IHRvcDsKICAgICAgICAgICAgICAgIG9sLmFwcGVuZENo aWxkKGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoJ2xpJykpLnRleHRDb250ZW50ID0gU3RyaW5nKHRv cCk7CiAgICAgICAgICAgIH0KICAgICAgICAgICAgcmVxdWVzdEFuaW1hdGlvbkZyYW1lKGxvZyk7 CiAgICAgICAgfQoKICAgICAgICByZXF1ZXN0QW5pbWF0aW9uRnJhbWUobG9nKTsKICAgIH0KPC9z Y3JpcHQ+Cgo8cD5nZXRDb21wdXRlZFN0eWxlIG9uIGEgbGVuZ3RoIHByb3BlcnR5IGR1cmluZyBh IHRyYW5zaXRpb24gb3IgYW5pbWF0aW9uIGZyb20gYSBwaXhlbCB0byBhIHBlcmNlbnQgYXNzZXJ0 cyAob3IgcmV0dXJucyBudWxsIGluIHJlbGVhc2UgYnVpbGRzKTwvcD4KICAgIAo8ZGl2PjxzcGFu Pjwvc3Bhbj48L2Rpdj4KCjxoMT5Db21wdXRlZCB0b3Agd2hpbGUgYW5pbWF0aW5nPC9oMT4KPG9s Pjwvb2w+Cg==