gecko-dev/layout
Emilio Cobos Álvarez 9882821e73 Bug 1692684 - Don't create transitions for invalid ::marker properties. r=hiro
The test-case in the bug does something interesting, where it causes a
transition on the parent by removing a CSS rule, and that causes us to
transition text-underline-offset on our ::marker, via the magic of
font-size-relative properties.

text-underline-offset, while it gets inherited from ::marker, is not a
valid CSS property to specify on marker per spec, so we trim it here:

  https://searchfox.org/mozilla-central/rev/899bbd9e5a0d6de9bb9f068c48b1445c7905d9cf/servo/ports/geckolib/glue.rs#5709-5712

And that causes us to create a transition with an empty effect and
everything goes downhill from here.

For now, just bail out in a nicer way than we were doing. I still need
to look into whether we should handle inherited transitions differently
from non-inherited one in this case...

I think our behavior after this patch would be correct for the test-case
(because text-underline-offset would transition on the parent and
::marker would inherit it). If you specify transition only on the marker
we'd refuse to transition (which I guess it is somewhat of a sensible
behavior).

Differential Revision: https://phabricator.services.mozilla.com/D105124
2021-02-24 16:35:17 +00:00
..
base Bug 1590884 - Extra small cleanups. DONTBUILD 2021-02-24 15:21:38 +00:00
build Backed out 3 changesets (bug 1672431) for causing xpc failures in IOUtils. CLOSED TREE 2021-02-22 20:56:05 +02:00
docs Bug 1682703: [css-grid-3] Remove in-tree masonry draft spec, now that it lives in csswg repo. r=mats DONTBUILD 2020-12-16 05:17:36 +00:00
forms Bug 1694059 - Use WebRender to render most non-native-theme widgets when possible. r=mstange 2021-02-23 10:49:37 +00:00
generic Bug 1590884 - Move XUL accesskey handling to DOM and remove nsXULLabelFrame. r=emilio 2021-02-24 14:46:28 +00:00
inspector Bug 1684909 - Fix a mochitest that didn't account for image-set. 2021-01-12 00:12:21 +01:00
ipc Bug 1645528 - Connect nsRefreshDrivers in content processes with a widget-local vsync source r=mattwoodrow,emilio 2020-12-02 09:47:53 +00:00
mathml Bug 1693001 - Remove redundant nullptr checks of return value of infallible new in layout/. r=emilio 2021-02-16 15:52:09 +00:00
media Bug 1654103: Standardize on Black for Python code in mozilla-central. 2020-10-26 18:34:53 +00:00
painting Bug 1694059 - Use WebRender to render most non-native-theme widgets when possible. r=mstange 2021-02-23 10:49:37 +00:00
printing Bug 1691894 - Simplify uses of WithEntryHandle that only use OrInsert(With) by using GetOrInsertWith. r=xpcom-reviewers,necko-reviewers,jgilbert,nika 2021-02-22 12:07:46 +00:00
reftests Bug 1560242: Increase fuzzy tolerance for reftest flexbox-resizeviewport-1.xhtml to account for observed fuzziness levels. (no review; just a reftest annotation adjustment) DONTBUILD 2021-02-23 18:56:06 +00:00
style Bug 1692684 - Don't create transitions for invalid ::marker properties. r=hiro 2021-02-24 16:35:17 +00:00
svg Bug 1693032 - SVG text reflow can flow through multiple outer svg elements r=emilio 2021-02-17 14:51:47 +00:00
tables Bug 1693409 Part 2 - Don't enforce table's min-content inline-size when resolving flex base size. r=dholbert 2021-02-19 22:12:19 +00:00
tools Bug 1642709 - Make a relative path from topsrcdir work in jstestbrowser. r=jmaher 2021-02-22 20:32:26 +00:00
xul Bug 1590884 - Extra small cleanups. DONTBUILD 2021-02-24 15:21:38 +00:00
moz.build Bug 1654103: Standardize on Black for Python code in mozilla-central. 2020-10-26 18:34:53 +00:00