1553227 - Remove old scroll-snap implementation

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Emilio Cobos Álvarez [:emilio]]

It (or bug 1552587) block other work.

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1553227 - Remove old CSS scroll snap implementation. r=hiro,botond

This will save us some time from figuring out what's the best thing to do in
bug 1552587, so that other patches I have in flight (mainly bug 1552708) can
land, since we cannot add a single byte to nsStyleDisplay right now otherwise.

The code removed here is well isolated and not that complicated, so it seems to
me that should be easy to bring back should we have an emergency (and I commit
to doing that while preserving the nsStyleDisplay size limit if we need to :)).

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6c2047bc2f3e
Remove old CSS scroll snap implementation. r=hiro

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/267ddc3595fe
followup: fix animation-type-longhand.js.

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/6c2047bc2f3e
https://hg.mozilla.org/mozilla-central/rev/267ddc3595fe