Emilio Cobos Álvarez
Emilio Cobos Álvarez
Hah, I mid-aired with Anne of course :)
Not sure, the default value in Gecko is defined here: https://searchfox.org/mozilla-central/rev/a78233c11a6baf2c308fbed17eb16c6e57b6a2ac/browser/app/profile/firefox.js#431 But not sure why other libraries would change it.
I sent https://github.com/mozilla/web-ext/pull/1766 right before opening this issue for web-ext. I don't know about other users of this library.
In any case, `current_position` and `position` shouldn't really be subtly in different units. Let's document that if we need it.
Related: https://github.com/servo/servo/issues/11153
I think we should parse it, and clamp it to some appropriate range. We should probably just not serialize `inf`. Gecko doesn't serialize `inf`, maybe this bug was fixed?
It's `unchecked` in the sense that it doesn't check for EOF. But it does a bounds-check / panics if you misuse it.
Posted https://github.com/servo/rust-cssparser/pull/381 for posterity which uses a slice iterator. It seems that's a regression at least on the big-data-url test, so it'd need more work.
Hey, sure thing, go ahead!
I don't think we necessarily need a custom workflow, just something that can be benchmarked easily locally.