This is a series of already-reviewed changes by @yodalee from #5025, rebased onto current servo master, with some fixups applied:
* Fixed build errors/warnings from the latest rust upgrade.
* Marked `tests/wpt/web-platform-tests/encoding/single-byte-decoder.html` as expecting CRASH.
I could not verify locally that the new test annotation is correct, since the test appears to hang rather than crash on my Linux box. (Or maybe I just didn't wait long enough.) If this crash isn't consistent, or if it takes a long time, maybe we should skip this test instead?
r? @jdm
Source-Repo: https://github.com/servo/servo
Source-Revision: 4846607e189a65baf8d9ac6566594b7893d07f12
Running `./mach run --release` without a release profile built suggests running `./mach build--release`. This PR adds a space before `--release`.
Source-Repo: https://github.com/servo/servo
Source-Revision: b20cc69a7a30f62fcdfee5c7f5ca61454199d425
I think this should have been changed in #3618 but was missed. r? @pcwalton
I wasn't able to come up with a good test case for this, partly because of other bugs related to floats and formatting contexts.
Source-Repo: https://github.com/servo/servo
Source-Revision: b79d06c8d2363ebd6bdb171a943d1c932a6cea68
In my mistake, I close PR for this commit.
I'm sorry for inconvenience.
@jdm 's comment
The test expectations for /tests/wpt/web-platform-tests/html/dom/interfaces.html and /tests/wpt/web-platform-tests/workers/interfaces.workers.js will need to be updated - http://mxr.mozilla.org/servo/search?string=appversion
According to your guide, I revised commit. Is this your intention?
r? @jdm
cc @yichoi
Source-Repo: https://github.com/servo/servo
Source-Revision: 225d7d2ece418d5ecefd7ce9047c4754ffb68c73
Transition events are not yet supported, and the only animatable
properties are `top`, `right`, `bottom`, and `left`. However, all other
features of transitions are supported. There are no automated tests at
present because I'm not sure how best to test it, but three manual tests
are included.
r? @glennw
Source-Repo: https://github.com/servo/servo
Source-Revision: ebdf1d494b6c986e6dfcb7d8fd3f0ffa126523ed
Otherwise they queue up if the event handler isn't 60FPS.
r? @jdm
Source-Repo: https://github.com/servo/servo
Source-Revision: c1cc31b9d66f3c61dd0aa7e6a1a43991187d09a5
I decided to use the old Netscape broken image link icon (later we may
replace the image asset for something more trendier).
Source-Repo: https://github.com/servo/servo
Source-Revision: 1e282d55d7e36f0e488a395b90e961e9200c3817
This allows us to do layout off the main thread, which is necessary for
transitions.
r? @glennw
Source-Repo: https://github.com/servo/servo
Source-Revision: 46d4ef81674d1de4b9066185c7e63c32b04c167e
* Group imports
* Convert seralization helper function to use `Iterator::fold`
* Remove basically pointless `serialize_value` helper function
* Wrap lines longer than 100 chars
* Add URLs to spec
Source-Repo: https://github.com/servo/servo
Source-Revision: 018e1f726d0ec0bf3d0dee68594ef1151dc8049d
I accidentally left it in after removing the calls to stdout/stderr in
d926b8342b492cfa442a72b4d4da01e7e23d9cba.
Source-Repo: https://github.com/servo/servo
Source-Revision: 5ead929fea34ca6d05d6bc4f0a07eeab4dfae207
* Previously, the function returned an owned String, which is not
necessary, so now it returns a slice
* Steps have now been documented/labeled
* The last step of the algorithm was incorrect; it would only slice the
path if the "/" was the last character, which is not what the spec
says. The spec says to slice up until (but not including) the last
"/". Also added a regression test for this.
Source-Repo: https://github.com/servo/servo
Source-Revision: 0d00e37c23dce23cad5b98f82ae711675f324810
Changed Key::N to KeyN, also changed Equals to Equal, and NumpadEquals to NumpadEqual.
Source-Repo: https://github.com/servo/servo
Source-Revision: 629edc5c46a869b5bb6bc51b8d39b0115702a00c
"Negative values for radius must cause the implementation to throw an IndexSizeError exception."
Source-Repo: https://github.com/servo/servo
Source-Revision: 1bd8e18d92c291418ce6a5712f7b4503d9b5104c
We should ensure the parameter is lowercased. Right now, the assert will
always return true.
Discussed in #5445
Introduced in ee2ccc4f872ba33a86057d87a99d1015b3c41cf1
Source-Repo: https://github.com/servo/servo
Source-Revision: 350a35428ab547e45e86826d2a818cb15d004bf5
For the majority of these cases, `as_slice` can be removed due to
`Deref`. In particular, `Deref` for:
* `String` -> `str`
* `Atom` -> `str`
The latter of those two requires, a bump of the locked `string-cache`
library
Source-Repo: https://github.com/servo/servo
Source-Revision: 0fd41847a39be387c03bd5d8b6f2aec2b1bb6f66
These became unused in f45db7714c8c3244fe20d1b0a104cb845d0b79f5; I don't know
why I didn't notice that.
Source-Repo: https://github.com/servo/servo
Source-Revision: 2b9bfb9af387fba1a080bed070cf6134eb5d10e1
This is for #4704. I'm not sure if this is the best approach, so I'm open to suggestions.
Source-Repo: https://github.com/servo/servo
Source-Revision: 439e3150d74453b86abbcc7934b7e5152f102940
Also adding some newlines for visual separation between
functions/methods
Source-Repo: https://github.com/servo/servo
Source-Revision: 699ace844c939ab97180acad67a984dc71108a12
Was introduced in 7aee1cae84704b885988a5985a7604747125ec1e
I noticed it isn't actively in use, so unless there's some bigger reason
why it exists, it could probably be removed
Source-Repo: https://github.com/servo/servo
Source-Revision: 96fb1dac3d346cf2e46e5d5ec5abf79134743dad
Adenilson Cavalcanti
Keith Yeung
Florian Eula
yodalee
Kyle Cesare
snf
Nicholas Nethercote
Matt Brubeck
Prabhjyot Singh Sodhi
Deokjin Kim
Patrick Walton
Guro Bokum
Corey Farwell
Ms2ger
Mikko Vanhatalo
Mátyás Mustoha
Brandon DeRosier
Michael Wu
Mukilan Thiyagarajan
Thiago Pontes
Manish Goregaokar
Ryan Leavengood
Fabrice Desré