Closed Bug 1695402 Opened 4 years ago Closed 4 years ago

Support image-set() on the cursor property.

Categories

(Core :: CSS Parsing and Computation, task, P3)

task

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: emilio, Assigned: mtigley)

References

(Blocks 2 open bugs, )

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

There are two properties left that should support image-set per spec and don't. One is cursor, the other is content.

In the long term, we should probably support all <image> syntaxes for both properties. However this is more difficult for two reasons:

  • For cursor, there's only one callsite that processes the images, but then the imgIContainer gets passed around to all sorts of widgety / platform-specific code.

  • For content, we don't have the setup to disguise a CSS image as an image request, so we need to rewrite a lot of code. Also not great.

But we can both do an incremental step towards that and allow image-set for those if we do something like storing an Image, but not parsing non-url values.

That way image-set will be automatically supported, the code would need minimal changes, and as I said it'd be a step in the right direction.

In order to do this, we'd need to:

  • Add an extra parameter to image parsing, much like allow_none, to control whether we should parse only url-like values.

  • Change CursorImage type definitions to use Image instead of ImageUrl, and the parsing code to use the new parameter.

  • Change this line, which should stop compiling, by something like image.image.GetImageRequest().

  • That's it, we need to add some tests to property_database.js to ensure we parse image-set(url(..)) but not image-set(linear-gradient(...))

Blocks: 1425694

For reference, there's a discussion in the CSS Working Group about this.

Emilio, shouldn't we try to get a resolution for this from the CSSWG, first?

Sebastian

Flags: needinfo?(emilio)
Keywords: dev-doc-needed

Idk, the discussion in the issue seems pretty uncontroversial and it is supported by other browsers so...

Flags: needinfo?(emilio)
Assignee: nobody → mtigley
Severity: -- → S3
Status: NEW → ASSIGNED
Priority: -- → P3
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/78567f53575e Support image-set() on the cursor property. r=emilio

Backed out for failures on test_css-properties-db.js

backout: https://hg.mozilla.org/integration/autoland/rev/6c5bc506c6b829d8847f0eaa2a1373b52aa7042b

push: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&revision=78567f53575e99a903b7c8bb18fed2f8b345604b&searchStr=xpcshell

failure log: https://treeherder.mozilla.org/logviewer?job_id=331929239&repo=autoland&lineNumber=3432

[task 2021-03-03T17:22:20.565Z] 17:22:20 INFO - TEST-START | devtools/shared/tests/xpcshell/test_css-properties-db.js
[task 2021-03-03T17:22:20.745Z] 17:22:20 WARNING - TEST-UNEXPECTED-FAIL | devtools/shared/tests/xpcshell/test_css-properties-db.js | xpcshell return code: 0
[task 2021-03-03T17:22:20.746Z] 17:22:20 INFO - TEST-INFO took 176ms
[task 2021-03-03T17:22:20.747Z] 17:22:20 INFO - >>>>>>>
[task 2021-03-03T17:22:20.747Z] 17:22:20 INFO - (xpcshell/head.js) | test MAIN run_test pending (1)
[task 2021-03-03T17:22:20.748Z] 17:22:20 INFO - TEST-PASS | devtools/shared/tests/xpcshell/test_css-properties-db.js | run_test - [run_test : 42] The pseudo elements match on the client and platform. If this assertion fails, then the client side CSS properties list in devtools is out of sync with the CSS properties on the platform. To fix this assertion run mach devtools-css-db to re-generate the client side properties. - [":after",":before",":marker",":backdrop",":cue",":first-letter",":first-line",":selection",":-moz-focus-inner",":-moz-progress-bar",":-moz-range-track",":-moz-range-progress",":-moz-range-thumb",":-moz-meter-bar",":placeholder",":-moz-color-swatch",":file-selector-button"] deepEqual [":after",":before",":marker",":backdrop",":cue",":first-letter",":first-line",":selection",":-moz-focus-inner",":-moz-progress-bar",":-moz-range-track",":-moz-range-progress",":-moz-range-thumb",":-moz-meter-bar",":placeholder",":-moz-color-swatch",":file-selector-button"]
[task 2021-03-03T17:22:20.756Z] 17:22:20 INFO - TEST-PASS | devtools/shared/tests/xpcshell/test_css-properties-db.js | run_test - [run_test : 50] The preferences match on the client and platform. If this assertion fails, then the client side CSS properties list in devtools is out of sync with the CSS properties on the platform. To fix this assertion run mach devtools-css-db to re-generate the client side properties. - [["aspect-ratio","layout.css.aspect-ratio.enabled"],["font-optical-sizing","layout.css.font-variations.enabled"],["initial-letter","layout.css.initial-letter.enabled"],["masonry-auto-flow","layout.css.grid-template-masonry-value.enabled"],["math-depth","layout.css.math-depth.enabled"],["math-style","layout.css.math-style.enabled"],["-moz-osx-font-smoothing","layout.css.osx-font-smoothing.enabled"],["offset-rotate","layout.css.motion-path.enabled"],["overflow-anchor","layout.css.scroll-anchoring.enabled"],["touch-action","layout.css.touch_action.enabled"],["-webkit-line-clamp","layout.css.webkit-line-clamp.enabled"],["overflow-clip-box-block","layout.css.overflow-clip-box.enabled"],["overflow-clip-box-inline","layout.css.overflow-clip-box.enabled"],["overflow-block","layout.css.overflow-logical.enabled"],["overflow-inline","layout.css.overflow-logical.enabled"],["overscroll-behavior-block","layout.css.overscroll-behavior.enabled"],["overscroll-behavior-inline","layout.css.overscroll-behavior.enabled"],["overscroll-behavior-x","layout.css.overscroll-behavior.enabled"],["overscroll-behavior-y","layout.css.overscroll-behavior.enabled"],["align-tracks","layout.css.grid-template-masonry-value.enabled"],["backdrop-filter","layout.css.backdrop-filter.enabled"],["font-variation-settings","layout.css.font-variations.enabled"],["justify-tracks","layout.css.grid-template-masonry-value.enabled"],["offset-anchor","layout.css.motion-path.enabled"],["offset-path","layout.css.motion-path.enabled"],["rotate","layout.css.individual-transform.enabled"],["scale","layout.css.individual-transform.enabled"],["translate","layout.css.individual-transform.enabled"],["offset-distance","layout.css.motion-path.enabled"],["overflow-clip-box","layout.css.overflow-clip-box.enabled"],["overscroll-behavior","layout.css.overscroll-behavior.enabled"],["offset","layout.css.motion-path.enabled"],["zoom","layout.css.zoom-transform-hack.enabled"],["-moz-transition-duration","layout.css.prefixes.transitions"],["-moz-transition-timing-function","layout.css.prefixes.transitions"],["-moz-transition-property","layout.css.prefixes.transitions"],["-moz-transition-delay","layout.css.prefixes.transitions"],["-moz-animation-name","layout.css.prefixes.animations"],["-moz-animation-duration","layout.css.prefixes.animations"],["-moz-animation-timing-function","layout.css.prefixes.animations"],["-moz-animation-iteration-count","layout.css.prefixes.animations"],["-moz-animation-direction","layout.css.prefixes.animations"],["-moz-animation-play-state","layout.css.prefixes.animations"],["-moz-animation-fill-mode","layout.css.prefixes.animations"],["-moz-animation-delay","layout.css.prefixes.animations"],["-moz-transform","layout.css.prefixes.transforms"],["-moz-perspective","layout.css.prefixes.transforms"],["-moz-perspective-origin","layout.css.prefixes.transforms"],["-moz-backface-visibility","layout.css.prefixes.transforms"],["-moz-transform-style","layout.css.prefixes.transforms"],["-moz-transform-origin","layout.css.prefixes.transforms"],["-moz-column-width","layout.css.prefixes.columns"],["-moz-column-count","layout.css.prefixes.columns"],["-moz-column-fill","layout.css.prefixes.columns"],["-moz-column-rule-width","layout.css.prefixes.columns"],["-moz-column-rule-color","layout.css.prefixes.columns"],["-moz-column-span","layout.css.prefixes.columns"],["-moz-column-rule-style","layout.css.prefixes.columns"],["-moz-font-feature-settings","layout.css.prefixes.font-features"],["-moz-font-language-override","layout.css.prefixes.font-features"],["-moz-box-sizing","layout.css.prefixes.box-sizing"],["-moz-column-gap","layout.css.prefixes.columns"],["-moz-border-image","layout.css.prefixes.border-image"],["-moz-transition","layout.css.prefixes.transitions"],["-moz-animation","layout.css.prefixes.animations"],["-moz-columns","layout.css.prefixes.columns"],["-moz-column-rule","layout.css.prefixes.columns"]] deepEqual [["aspect-ratio","layout.css.aspect-ratio.enabled"],["font-optical-sizing","layout.css.font-variations.enabled"],["initial-letter","layout.css.initial-letter.enabled"],["masonry-auto-flow","layout.css.grid-template-masonry-value.enabled"],["math-depth","layout.css.math-depth.enabled"],["math-style","layout.css.math-style.enabled"],["-moz-osx-font-smoothing","layout.css.osx-font-smoothing.enabled"],["offset-rotate","layout.css.motion-path.enabled"],["overflow-anchor","layout.css.scroll-anchoring.enabled"],["touch-action","layout.css.touch_action.enabled"],["-webkit-line-clamp","layout.css.webkit-line-clamp.enabled"],["overflow-clip-box-block","layout.css.overflow-clip-box.enabled"],["overflow-clip-box-inline","layout.css.overflow-clip-box.enabled"],["overflow-block","layout.css.overflow-logical.enabled"],["overflow-inline","layout.css.overflow-logical.enabled"],["overscroll-behavior-block","layout.css.overscroll-behavior.enabled"],["overscroll-behavior-inline","layout.css.overscroll-behavior.enabled"],["overscroll-behavior-x","layout.css.overscroll-behavior.enabled"],["overscroll-behavior-y","layout.css.overscroll-behavior.enabled"],["align-tracks","layout.css.grid-template-masonry-value.enabled"],["backdrop-filter","layout.css.backdrop-filter.enabled"],["font-variation-settings","layout.css.font-variations.enabled"],["justify-tracks","layout.css.grid-template-masonry-value.enabled"],["offset-anchor","layout.css.motion-path.enabled"],["offset-path","layout.css.motion-path.enabled"],["rotate","layout.css.individual-transform.enabled"],["scale","layout.css.individual-transform.enabled"],["translate","layout.css.individual-transform.enabled"],["offset-distance","layout.css.motion-path.enabled"],["overflow-clip-box","layout.css.overflow-clip-box.enabled"],["overscroll-behavior","layout.css.overscroll-behavior.enabled"],["offset","layout.css.motion-path.enabled"],["zoom","layout.css.zoom-transform-hack.enabled"],["-moz-transition-duration","layout.css.prefixes.transitions"],["-moz-transition-timing-function","layout.css.prefixes.transitions"],["-moz-transition-property","layout.css.prefixes.transitions"],["-moz-transition-delay","layout.css.prefixes.transitions"],["-moz-animation-name","layout.css.prefixes.animations"],["-moz-animation-duration","layout.css.prefixes.animations"],["-moz-animation-timing-function","layout.css.prefixes.animations"],["-moz-animation-iteration-count","layout.css.prefixes.animations"],["-moz-animation-direction","layout.css.prefixes.animations"],["-moz-animation-play-state","layout.css.prefixes.animations"],["-moz-animation-fill-mode","layout.css.prefixes.animations"],["-moz-animation-delay","layout.css.prefixes.animations"],["-moz-transform","layout.css.prefixes.transforms"],["-moz-perspective","layout.css.prefixes.transforms"],["-moz-perspective-origin","layout.css.prefixes.transforms"],["-moz-backface-visibility","layout.css.prefixes.transforms"],["-moz-transform-style","layout.css.prefixes.transforms"],["-moz-transform-origin","layout.css.prefixes.transforms"],["-moz-column-width","layout.css.prefixes.columns"],["-moz-column-count","layout.css.prefixes.columns"],["-moz-column-fill","layout.css.prefixes.columns"],["-moz-column-rule-width","layout.css.prefixes.columns"],["-moz-column-rule-color","layout.css.prefixes.columns"],["-moz-column-span","layout.css.prefixes.columns"],["-moz-column-rule-style","layout.css.prefixes.columns"],["-moz-font-feature-settings","layout.css.prefixes.font-features"],["-moz-font-language-override","layout.css.prefixes.font-features"],["-moz-box-sizing","layout.css.prefixes.box-sizing"],["-moz-column-gap","layout.css.prefixes.columns"],["-moz-border-image","layout.css.prefixes.border-image"],["-moz-transition","layout.css.prefixes.transitions"],["-moz-animation","layout.css.prefixes.animations"],["-moz-columns","layout.css.prefixes.columns"],["-moz-column-rule","layout.css.prefixes.columns"]]
[task 2021-03-03T17:22:20.758Z] 17:22:20 INFO - TEST-PASS | devtools/shared/tests/xpcshell/test_css-properties-db.js | run_test - [run_test : 71] The static database and platform match for "-moz-animation". - true == true
[task 2021-03-03T17:22:20.758Z] 17:22:20 INFO - TEST-PASS | devtools/shared/tests/xpcshell/test_css-properties-db.js | run_test - [run_test : 71] The static database and platform match for "-moz-animation-delay". - true == true

Flags: needinfo?(mtigley)
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6644ba088cce Support image-set() on the cursor property. r=emilio
Blocks: 1696314
Flags: needinfo?(mtigley)
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
Regressions: 1696677

Added to release notes

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: