Closed Bug 1818329 Opened 2 years ago Closed 2 years ago

Allow negatives when parsing media feature values.

Categories

(Core :: CSS Parsing and Computation, task)

task

Tracking

()

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

No description provided.

Clean-up some other test expectations while at it.

No longer blocks: interop-2023
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fa64d4d68a0c Don't reject negative values at parse time in media features. r=dshin
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/89c458212f3c Don't reject negative resolutions either. r=layout-reviewers,jfkthame
Backout by csabou@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/beff44676d63 Backed out changeset 89c458212f3c for causing wpt reftest failures on image-set-invalid-resolution-rendering-2.html
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Backout by csabou@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c915de76eab3 Backed out changeset fa64d4d68a0c for causing wpt reftest failures on mq-negative-range-001.html. CLOSED TREE
Attachment #9319476 - Attachment description: Bug 1818329 - Don't reject negative resolutions either. r=#style,#layout → Bug 1818329 - Don't reject negative resolutions either. r=jfkthame
Flags: needinfo?(emilio)
Attachment #9319319 - Attachment description: Bug 1818329 - Don't reject negative values at parse time in media features. r=#layout,#style → Bug 1818329 - Don't reject negative values at parse time in media features. r=dshin
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8c7eb3fca5a8 Don't reject negative values at parse time in media features. r=dshin https://hg.mozilla.org/integration/autoland/rev/af4cdf10aafd Don't reject negative resolutions either. r=layout-reviewers,jfkthame
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38704 for changes under testing/web-platform/tests
Upstream PR was closed without merging
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6c3a6a328d47 Don't reject negative values at parse time in media features. r=dshin https://hg.mozilla.org/integration/autoland/rev/01c1a2d4f4fd Don't reject negative resolutions either. r=layout-reviewers,jfkthame

Backed out for causing wpt failures in image-set-parsing.html.

  • Backout link
  • Push with failures
  • Failure Log
  • Failure line: TEST-UNEXPECTED-FAIL | /css/css-images/image-set/image-set-parsing.html | e.style['background-image'] = "image-set(url(example.png) 0x)" should not set the property value - assert_equals: expected "" but got "image-set(url("example.png") 0x)"
Flags: needinfo?(emilio)
Flags: needinfo?(emilio)
Upstream PR was closed without merging
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d9f50896a6e9 Don't reject negative values at parse time in media features. r=dshin https://hg.mozilla.org/integration/autoland/rev/50b2f3e731ef Don't reject negative resolutions either. r=layout-reviewers,jfkthame
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: