1594241 - CSSOM cssText property is missing styles from stylesheet

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, defect, P3)

Description

[Evan Jacobs]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.70 Safari/537.36

Steps to reproduce:

1. Go to a page and create a style element in head with this content:

<style id="style-test">
.a{border:none;font-size:36px;border-image:linear-gradient(315deg,#fff8c5,#fdb3e4);border-width:3px;border-style:solid;border-image-slice:1;background-color:white;}
</style>

2. Go to your console and inspect it:

document.getElementById("style-test").sheet.cssRules[0]

3. Notice that .cssText is missing the border-image declaration, but if you access the style property it's actually there:

document.getElementById("style-test").sheet.cssRules[0].style['border-image']

Actual results:

.cssText returns this:

.a { border: 3px solid; font-size: 36px; background-color: white; }

Expected results:

.cssText should return:

.a { border: none; font-size: 36px; border-image: linear-gradient(315deg, #fff8c5, #fdb3e4); border-width: 3px; border-style: solid; border-image-slice: 1; background-color: white; }

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Yeah, this is clearly a bug.

The border shorthand always resets border-image, etc, to its initial value: https://searchfox.org/mozilla-central/rev/8b7aa8af652f87d39349067a5bc9c0256bf6dedc/servo/components/style/properties/shorthands/border.mako.rs#147

But it doesn't check whether those properties are initial when serializing back: https://searchfox.org/mozilla-central/rev/8b7aa8af652f87d39349067a5bc9c0256bf6dedc/servo/components/style/properties/shorthands/border.mako.rs#182

Should be a pretty easy bug to fix, we effectively need to add a check like:

% for name in "outset repeat slice source width".split():
if self.border_image_${name} != border_image_${name}::get_initial_specified_value() {
    return Ok(());
}
% endfor

Somewhere around here.

And add a regression test if there's none of course.

Comment 3

[Erik Nordin [:nordzilla]]

Attachment: Bug 1594241 - Check for border-image-* initial specified values when serializing border shorthand r=emilio,boris

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e31cda624e1b
Check for border-image-* initial specified values when serializing border shorthand r=emilio

Comment 5

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/20590 for changes under testing/web-platform/tests

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/e31cda624e1b

Comment 7

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Can't merge web-platform-tests PR due to failing upstream checks:
Github PR https://github.com/web-platform-tests/wpt/pull/20590
* Community-TC (pull_request) (https://community-tc.services.mozilla.com/tasks/groups/CqRSQ6FfQmeaUHCTqbO0IQ)

Comment 8

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 9

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot