1637540 - Wrong list of browser versions

Status: RESOLVED FIXED

(DevTools :: Inspector: Compatibility, defect, P2)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: image.png

STR:

1. Create an element with the following CSS style

.myStyle {
  -moz-binding: unset;
}

2. Load your test page, open DevTools and Select the Inspector & Compatibility panel
   You should see one compatibility issue in the list

3. Hover over the Firefox (desktop) icon associated with the compat issue.
   You should see a tooltip listing: 60, 74, 75 and 76
   See also the attached screenshot

4. But looking at the compat data-set for -moz-binding shows that this prop were removed in 67: version_removed: "67"

Link to the dataset: https://hg.mozilla.org/mozilla-central/raw-file/tip/devtools/client/inspector/compatibility/lib/dataset/css-properties.json

Is that a bug?

Honza

Comment 1

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: image.png

Quick screenshot of the dataset I am seeing.

Honza

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Daisuke, here is the bug I mentioned on Slack.

Honza

Comment 3

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1637540: Avoid comparing with perfix if the compat data is defined with prefix.

Comment 4

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1637540: Add test for a property defined with prefix.

Depends on D75256

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9eacf691cd0f
Avoid comparing with perfix if the compat data is defined with prefix. r=rcaliman
https://hg.mozilla.org/integration/autoland/rev/e4a152b7c578
Add test for a property defined with prefix. r=rcaliman

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/9eacf691cd0f
https://hg.mozilla.org/mozilla-central/rev/e4a152b7c578