|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
This is revealed by test layout/style/test_computed_style.html that there is a difference between Stylo and Gecko that Gecko fails to serialize the computed value of mask shorthand there. It is because Gecko checks whether all subproperties of mask except mask-image are initial for serializing the computed value . This check fails for "mask: url(#mask)" because in Stylo, mask-repeat is set to "no-repeat" for initial, while Gecko checks it against "repeat". The spec seems to agree with Servo that initial value of mask-repeat is "no-repeat", and it specially notes that its initial value is different from background-repeat (which is "repeat"). However, although Blink doesn't have mask-repeat, its initial value of -webkit-mask-repeat seems to be "repeat", which matches Gecko. It is unclear to me whether we should change Gecko to match the spec (and Servo) or we should change Servo (and spec) to match Gecko and Blink.  https://dxr.mozilla.org/mozilla-central/rev/3b96f277325747fe668ca8cd896d2f581238e4ee/layout/style/nsComputedDOMStyle.cpp#6361-6377
CJ, what do you think about this?
Per bug 1346193 the WG agreed to change the initial value to "repeat", but the spec change still hasn't been made yet.
Ok, so we should change Stylo to match the resolution.
Comment on attachment 8867611 [details] Bug 1364273 - Change mask-repeat initial value from no-repeat to repeat. https://reviewboard.mozilla.org/r/139176/#review142402 ::: commit-message-e66de:1 (Diff revision 1) > +Bug 1364273 - Change mask-repeat initial value from no-preat to repeat. no-repeat
Comment on attachment 8867976 [details] Bug 1364273 - (followup) update stylo-failures.md to reduce failure count of mask. https://reviewboard.mozilla.org/r/139514/#review142858
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/6028e667129f (followup) update stylo-failures.md to reduce failure count of mask. r=cjku