{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/web-platform-tests/issues/10700","repository_url":"https://api.github.com/repos/w3c/web-platform-tests","labels_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/10700/labels{/name}","comments_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/10700/comments","events_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/10700/events","html_url":"https://github.com/w3c/web-platform-tests/pull/10700","id":318592344,"number":10700,"title":"Parsing tests for CSS UI Level 3","user":{"login":"ewilligers","id":929206,"avatar_url":"https://avatars1.githubusercontent.com/u/929206?v=4","gravatar_id":"","url":"https://api.github.com/users/ewilligers","html_url":"https://github.com/ewilligers","followers_url":"https://api.github.com/users/ewilligers/followers","following_url":"https://api.github.com/users/ewilligers/following{/other_user}","gists_url":"https://api.github.com/users/ewilligers/gists{/gist_id}","starred_url":"https://api.github.com/users/ewilligers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ewilligers/subscriptions","organizations_url":"https://api.github.com/users/ewilligers/orgs","repos_url":"https://api.github.com/users/ewilligers/repos","events_url":"https://api.github.com/users/ewilligers/events{/privacy}","received_events_url":"https://api.github.com/users/ewilligers/received_events","type":"User","site_admin":false},"labels":[{"id":573648720,"url":"https://api.github.com/repos/w3c/web-platform-tests/labels/css-ui","name":"css-ui","color":"ededed","default":false},{"id":93361070,"url":"https://api.github.com/repos/w3c/web-platform-tests/labels/wg-css","name":"wg-css","color":"0052cc","default":false}],"state":"open","locked":false,"assignee":{"login":"FremyCompany","id":364405,"avatar_url":"https://avatars3.githubusercontent.com/u/364405?v=4","gravatar_id":"","url":"https://api.github.com/users/FremyCompany","html_url":"https://github.com/FremyCompany","followers_url":"https://api.github.com/users/FremyCompany/followers","following_url":"https://api.github.com/users/FremyCompany/following{/other_user}","gists_url":"https://api.github.com/users/FremyCompany/gists{/gist_id}","starred_url":"https://api.github.com/users/FremyCompany/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FremyCompany/subscriptions","organizations_url":"https://api.github.com/users/FremyCompany/orgs","repos_url":"https://api.github.com/users/FremyCompany/repos","events_url":"https://api.github.com/users/FremyCompany/events{/privacy}","received_events_url":"https://api.github.com/users/FremyCompany/received_events","type":"User","site_admin":false},"assignees":[{"login":"FremyCompany","id":364405,"avatar_url":"https://avatars3.githubusercontent.com/u/364405?v=4","gravatar_id":"","url":"https://api.github.com/users/FremyCompany","html_url":"https://github.com/FremyCompany","followers_url":"https://api.github.com/users/FremyCompany/followers","following_url":"https://api.github.com/users/FremyCompany/following{/other_user}","gists_url":"https://api.github.com/users/FremyCompany/gists{/gist_id}","starred_url":"https://api.github.com/users/FremyCompany/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FremyCompany/subscriptions","organizations_url":"https://api.github.com/users/FremyCompany/orgs","repos_url":"https://api.github.com/users/FremyCompany/repos","events_url":"https://api.github.com/users/FremyCompany/events{/privacy}","received_events_url":"https://api.github.com/users/FremyCompany/received_events","type":"User","site_admin":false},{"login":"emilio","id":1323194,"avatar_url":"https://avatars1.githubusercontent.com/u/1323194?v=4","gravatar_id":"","url":"https://api.github.com/users/emilio","html_url":"https://github.com/emilio","followers_url":"https://api.github.com/users/emilio/followers","following_url":"https://api.github.com/users/emilio/following{/other_user}","gists_url":"https://api.github.com/users/emilio/gists{/gist_id}","starred_url":"https://api.github.com/users/emilio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/emilio/subscriptions","organizations_url":"https://api.github.com/users/emilio/orgs","repos_url":"https://api.github.com/users/emilio/repos","events_url":"https://api.github.com/users/emilio/events{/privacy}","received_events_url":"https://api.github.com/users/emilio/received_events","type":"User","site_admin":false}],"milestone":null,"comments":4,"created_at":"2018-04-28T02:29:53Z","updated_at":"2018-04-30T19:28:07Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/w3c/web-platform-tests/pulls/10700","html_url":"https://github.com/w3c/web-platform-tests/pull/10700","diff_url":"https://github.com/w3c/web-platform-tests/pull/10700.diff","patch_url":"https://github.com/w3c/web-platform-tests/pull/10700.patch"},"body":"The tests expose a number of bugs:\r\n\r\nhttps://developer.microsoft.com/en-us/microsoft-edge/platform/issues/16854772/\r\nhttps://developer.microsoft.com/en-us/microsoft-edge/platform/issues/16900961/\r\n\r\nhttps://bugzilla.mozilla.org/show_bug.cgi?id=1453258 [fixed]\r\nhttps://bugzilla.mozilla.org/show_bug.cgi?id=1457652\r\n\r\nhttps://bugs.chromium.org/p/chromium/issues/detail?id=708197 [fixed]\r\nhttps://bugs.chromium.org/p/chromium/issues/detail?id=831550\n\n<!-- Reviewable:start -->\n\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/w3c/web-platform-tests/issues/comments/385503254","html_url":"https://github.com/w3c/web-platform-tests/pull/10700#issuecomment-385503254","issue_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/10700","id":385503254,"user":{"login":"FremyCompany","id":364405,"avatar_url":"https://avatars3.githubusercontent.com/u/364405?v=4","gravatar_id":"","url":"https://api.github.com/users/FremyCompany","html_url":"https://github.com/FremyCompany","followers_url":"https://api.github.com/users/FremyCompany/followers","following_url":"https://api.github.com/users/FremyCompany/following{/other_user}","gists_url":"https://api.github.com/users/FremyCompany/gists{/gist_id}","starred_url":"https://api.github.com/users/FremyCompany/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FremyCompany/subscriptions","organizations_url":"https://api.github.com/users/FremyCompany/orgs","repos_url":"https://api.github.com/users/FremyCompany/repos","events_url":"https://api.github.com/users/FremyCompany/events{/privacy}","received_events_url":"https://api.github.com/users/FremyCompany/received_events","type":"User","site_admin":false},"created_at":"2018-04-30T19:28:07Z","updated_at":"2018-04-30T19:28:07Z","author_association":"CONTRIBUTOR","body":"> how do we avoid turning this into a temptation to support syntax without behavior?\r\nI think this is a valid concern in theory, but not really an issue in practice. Speaking for Edge here, tests aren't ever going to be the reason we would do this. There is nothing worse than gaining a few points on a test suite with a fake implementation that later causes you to have a bug on a real website. No user is ever looking at test suites, users look at real sites. \r\n\r\nNow, it's totally possible that Edge would ship syntax even if it doesn't support its behavior, for instance if we face real websites being broken (we have shipped `getMatchedCssRules` as a function returning null because some website called it and crashed if the function didn't exist but never actually used the results for anything, and from that day on we have been working with Chrome to have them remove this feature, and they finally did, so we will be able to remove this shim from our code too, yeah!). But as horrible as these kind of stories are, I don't think anyone here would consider passing a wpt test important enough to actually ship a broken feature. Broken behaviors can break real sites, we don't want to take that risk.\r\n\r\nFWIW though, I was doing exactly this right now in a prototype pull request because Edge does not parse `cursor: url(...) <number> <number>, fallback\" because we don't support cursors that are not of the `*.cur` format which doesn't need these numbers; however since we don't parse it, we don't even get the fallback value in Edge, so it's worth parsing these numbers even if we don't use them; these tests made me realize I hadn't thought about parsing `calc()` for a number so I'd say they were actually useful."}}
{"actor":{"display_login":"FremyCompany"}}