[wpt-sync] Sync PR 17054 - [css-ui] Correct property name
Categories
(Core :: CSS Parsing and Computation, defect, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync upstream])
Sync web-platform-tests PR 17054 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/17054
Details from upstream follow.
Mike Pennisi <mike@mikepennisi.com> wrote:
[css-ui] Correct property name
The test for the
appearance
property mistakenly referred to the vendor
prefixed version,-webkit-appearance
. In addition to contradicting the
test name and metadata, this caused the test intended for
-webkit-appearance
to reference the non-existent property
-webkit--webkit-appearance
.Correct both tests, using the suite's build process to generate the
-webkit-appearance
test from the corrected version of theappearance
test.
Recent research has demonstrated that the
menulist-button
value is necessary for web compatibility.This seems to be the only instance of this particular mistake:
$ git rev-parse HEAD 48af14aaebdbcaf90bc21d07965f7c16661f8d90 $ git grep -E 'moz-+moz' $ git grep -E 'webkit-+webkit' css/css-ui/webkit-appearance-menulist-button-002.html: #container > #drop-down-select { -webkit--webkit-appearance: none; -webkit--webkit-appearance: menulist-button; } $
Assignee | ||
Comment 1•5 years ago
|
||
PR 17054 applied with additional changes from upstream: 31e7edc073c29fec0e708a2e883c774b55452421
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=29e22ed5de29e541e4a90895d40947ea0cc864ba
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=bdcbea46827b4b37ba13b8e6ab7cf7f60427589f
Assignee | ||
Comment 4•5 years ago
|
||
Failed to get results from try push
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/82dcaaad1470 [wpt PR 17054] - [css-ui] Correct property name, a=testonly
Comment 6•5 years ago
|
||
bugherder |
Assignee | ||
Updated•5 years ago
|
Description
•