New issue
Advanced search Search tips

Issue 900119 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[WPT] css/css-transforms/inheritance.html - transform-style has initial value fails

Project Member Reported by monorail...@ecosystem-infra.iam.gserviceaccount.com, Oct 30

Issue description

WPT import https://crrev.com/c/1307069 introduced new failures in external/wpt/css/css-transforms:

List of new failures:
external/wpt/css/css-transforms/inheritance.html new failing tests: https://crrev.com/c/1307069/4/third_party/WebKit/LayoutTests/external/wpt/css/css-transforms/inheritance-expected.txt

This import contains upstream changes from b622fea47d516f82e303c1954d0aee2652466c80 to 7b078f0d6ff82157c57f343838f61b5200686e53:
[filter-effects] Inheritance and initial values (#13105): https://github.com/web-platform-tests/wpt/commit/7b078f0d6ff82157c57f343838f61b5200686e53 [affecting this directory]
[css-properties-values-api] Compute <angle> values to 'deg'.: https://github.com/web-platform-tests/wpt/commit/0cc29e423fd0785b5f906a903d9010519595a4ac [affecting this directory]
[css-properties-values-api] Proper computed value for <color>.: https://github.com/web-platform-tests/wpt/commit/38bb114c431508bf122a9699869c7a72e4feb5b3 [affecting this directory]
Use `t.step_func` and expected path checking: https://github.com/web-platform-tests/wpt/commit/76b8a916dcabe7f1160b1690d1c2721118955693
Assign a name to iframes when loading the initial about:blank: https://github.com/web-platform-tests/wpt/commit/4939eaea72260f3ded7c1d935c929b220cfaa9ab
Python 3: Use six.create_bound_method() in test_response.py (#13624): https://github.com/web-platform-tests/wpt/commit/669ee6464677befda509ca4056ab8269db7e94a8
[docs] Embed a video with intro to wpt: https://github.com/web-platform-tests/wpt/commit/2ea1f768b5844be4fe7fb5edac11a1b0da5ae16a
CSS: Move :where test to WPT: https://github.com/web-platform-tests/wpt/commit/efcea0501c0c702af64fc3a464efff00a549b99f [affecting this directory]
Use the correct out-of-flow containing block in continuations.: https://github.com/web-platform-tests/wpt/commit/38470be9afc947a30b9b1a066d8419cec4d0a3a8 [affecting this directory]
WPT: Adjust expected timing resolution: https://github.com/web-platform-tests/wpt/commit/14b8cd38f01ee9127919c12a7d29e372643c2097
Ignore includers of untested mixins (#13611): https://github.com/web-platform-tests/wpt/commit/e9e06cd3ede66193be47ff0d436c62a606d5fd50
Merge pull request #13734 from web-platform-tests/beaufortfrancois-patch-1: https://github.com/web-platform-tests/wpt/commit/1d5dbb331ad61eec01d55bc160b99772de3254d7
Remove SVG use attribute change handling code from nsSVGUseFrame to SVGUseElement.: https://github.com/web-platform-tests/wpt/commit/d535ea6d2380fcb88b42c3c5a9abc41b272396cc
Force line-height:normal for themed comboboxes for compat with other UAs.: https://github.com/web-platform-tests/wpt/commit/13b44870fb7d6b5b70ed14c1c5da735fe3cb8465
Swapping @MSFTkihans with @zouhir (#13733): https://github.com/web-platform-tests/wpt/commit/2e7a549d07c54cbdb9002634bf5a4cc6ce8f9acb
[filter-effects] computed filter (#13751): https://github.com/web-platform-tests/wpt/commit/2b4768cec91fc083a1247adadd542291f2558ac9 [affecting this directory]
[filter-effects] filter arguments optional (#13750): https://github.com/web-platform-tests/wpt/commit/8d90bc4a082eb30f8843e38068f29a9e81bf2fca [affecting this directory]
Update comments and test mix-blend-mode on pattern: https://github.com/web-platform-tests/wpt/commit/66aabf66f58aee669c26df6e67f85fbc6d3aaf9f
[svg] circle radius 0 disables rendering (#13749): https://github.com/web-platform-tests/wpt/commit/6051b8b55390be840b2eaa179fdfe62a1941c945
[css-transforms] Transforms initial values (#13023): https://github.com/web-platform-tests/wpt/commit/71c05e71317fec5f71d5575cd636b0a8e3835f9c [affecting this directory]
Fix pip aspect ratio test: https://github.com/web-platform-tests/wpt/commit/39322dfcc55488ee0b2dd8e4495d20cf103c6e15

 
Components: Blink>CSS
Components: -Blink>CSS Blink>Animation
Cc: smcgruer@chromium.org
Labels: Hotlist-Interop Pri-3 Type-Bug
Status: Available (was: Untriaged)
Summary: [WPT] css/css-transforms/inheritance.html - transform-style has initial value fails (was: [WPT] New failures introduced in external/wpt/css/css-transforms by import https://crrev.com/c/1307069)
Looks like we have 'flat' as the initial value for transform-style, but the test expects 'auto'.

Default value set here: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/css/css_properties.json5?l=3254&rcl=ed34d0a004be05146e49cb1486961988c0ce349c

Strangely, the MDN page lists 'flat' as the default value: https://developer.mozilla.org/en-US/docs/Web/CSS/transform-style , but the spec has listed 'auto' for over 2 years (https://drafts.csswg.org/css-transforms-2/#transform-style-property).

It is noted in the original PR that (all?) browsers incorrectly report 'flat'; https://github.com/web-platform-tests/wpt/pull/13023

We may need to do some web-compat work here to figure out if it will be safe to change our initial value to 'auto'.

Hotlist-Interop due to difference between blink + spec.

Sign in to add a comment