aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Disable incremental compilation on CISimon Sapin2018-01-103-0/+3
| | | | | We don’t keep the `target` directory between builds, and incremental cache takes a lot of space.
* We use Nigthly Cargo with Stable RustSimon Sapin2018-01-101-3/+2
| | | | | Nightly now has incremental compilation as a stable feature, but that’s not in the stable channel yet.
* Don’t set codegen-units to 4.Simon Sapin2018-01-101-3/+0
| | | | | The default is now 16: https://github.com/rust-lang/rust/blob/b5361d0d41/src/librustc/session/mod.rs#L738-L788
* Bump gl_generator to 0.8Bastien Orivel2018-01-102-40/+60
| | | | Fixes #19735
* Auto merge of #19395 - servo:rustup.rs, r=jdmbors-servo2018-01-1013-375/+189
|\ | | | | | | | | | | | | | | | | | | | | Use rustup.rs instead of custom bootstrap Fixes #11361, closes #18874, fixes #19365. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19395) <!-- Reviewable:end -->
| * rustup.rs: Use MSVC rather than GNU toolchains on WindowsSimon Sapin2018-01-101-1/+5
| |
| * Revert "Use rustup "proxies" instead of `rustup run`"Simon Sapin2018-01-101-12/+13
| | | | | | | | This reverts commit ad22368646290510149d33a0a41fc436ac4a0e96.
| * Use rustup "proxies" instead of `rustup run`Simon Sapin2018-01-102-13/+13
| | | | | | | | | | | | To make sure we’re indeed running rustup’s proxy rather than some other `cargo` for example, run the `rustup` executable with a different `argv[0]`.
| * Fix "rustup is not installed" messageSimon Sapin2018-01-101-9/+8
| |
| * Cache rustup toolchains on AppVeyor and Travis-CISimon Sapin2018-01-102-2/+4
| |
| * Print a better error message when rustup is too oldSimon Sapin2018-01-101-0/+7
| |
| * Print a link to instructions if rustup is not found.Simon Sapin2018-01-101-1/+11
| |
| * Add rustup.rs instructions in READMESimon Sapin2018-01-101-5/+29
| |
| * Use rustup.rs instead of custom bootstrapSimon Sapin2018-01-1011-362/+125
| | | | | | | | Fixes #11361, closes #18874
| * Revert "Use rustdoc --document-private-item instead of deprecated flags"Simon Sapin2018-01-103-6/+10
| | | | | | | | This reverts commit d96f0ff6a77e309682898cc7e1905a7bd0feb3cf.
* | Auto merge of #19727 - nox:cr-placeholder, r=emiliobors-servo2018-01-103-10/+10
|\ \ | | | | | | | | | | | | | | | | | | | | | Properly handle CR in textarea placeholders (fixes #19717) <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19727) <!-- Reviewable:end -->
| * | Properly handle CR in textarea placeholders (fixes #19717)Anthony Ramine2018-01-103-10/+10
| |/
* | Auto merge of #19737 - CYBAI:move-paint-order-out-of-mako, r=emiliobors-servo2018-01-105-148/+148
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Move paint-order outside of mako This is a sub-PR of #19015 r? emilio --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #19736 - [x] These changes do not require tests <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19737) <!-- Reviewable:end -->
| * style: Move paint-order outside of makoCYBAI2018-01-105-148/+148
|/
* Auto merge of #19734 - emilio:less-indirection, r=mbrubeckbors-servo2018-01-095-40/+37
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Remove some unneeded indirection. All TElement's implement Copy, and are just pointers, so the double indirection is stupid. I'm going to try to see if removing this double-indirection fixes some selector-matching performance, and this is a trivial pre-requisite while I wait for Talos results. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19734) <!-- Reviewable:end -->
| * style: Remove some unneeded indirection.Emilio Cobos Álvarez2018-01-105-40/+37
| | | | | | | | | | | | | | | | | | All TElement's implement Copy, and are just pointers, so the double indirection is stupid. I'm going to try to see if removing this double-indirection fixes some selector-matching performance, and this is a trivial pre-requisite while I wait for Talos results.
* | Auto merge of #19730 - NLincoln:19172-number-input-type-validations, r=KiChjangbors-servo2018-01-095-137/+16
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | number input type validations I used rust's builtin float parser to implement this. Rust's parser is more permissive than what browsers support (in this case), so I added some code to handle those edge cases. This passes all the prewritten test cases locally, but I fell asleep last night before updating the manifests :sweat_smile: --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes are part of #19172 <!-- Either: --> - [X] There are tests for these changes <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19730) <!-- Reviewable:end -->
| * | input type=number validationsNathan2018-01-095-137/+16
| | |
* | | Auto merge of #19733 - emilio:less-unwrap, r=glennwbors-servo2018-01-092-31/+27
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | style: Use Option::get_or_insert_with. Less unwraps is better. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19733) <!-- Reviewable:end -->
| * | style: Use Option::get_or_insert_with.Emilio Cobos Álvarez2018-01-102-31/+27
| |/ | | | | | | Less unwraps is better.
* | Auto merge of #19732 - gootorov:master, r=jdmbors-servo2018-01-091-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update Gentoo Linux setup to include clang <!-- Please describe your changes on the following line: -->While other Linux distributions include clang package as required in the setup section of README, Gentoo does not. This patch fixes it. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [ ] `./mach build -d` does not report any errors - [ ] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because it is a documentation fix <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19732) <!-- Reviewable:end -->
| * | Update Gentoo Linux setup to include clangIgor Gutorov2018-01-091-1/+1
| | |
* | | Auto merge of #19728 - servo:wptup, r=jdmbors-servo2018-01-093679-95381/+163071
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | Update web-platform-tests <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19728) <!-- Reviewable:end -->
| * | Avoid for(const ... of ...) usage in webidl parser.Josh Matthews2018-01-091-3/+5
| | | | | | | | | | | | Servo's JS engine doesn't support this construct yet.
| * | Update web-platform-tests to revision be5419e845d39089ba6dc338c1bd0fa279108317Josh Matthews2018-01-093679-95382/+163070
| | |
* | | Auto merge of #19720 - aneeshusa:support-env-vars-in-buildbot-steps, r=jdmbors-servo2018-01-096-35/+51
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update buildbot_steps lint to handle env variables https://github.com/servo/saltfs/pull/687 added support for specifying environment variables in `buildbot_steps.yml`. Update the servo-tidy buildbot_steps.yml linter to reflect this. Use the voluptuous Python library (BSD 3-clause license) for validation in lieu of a much larger hand-written implementation. Extracted out of #17171. Helps with servo/saltfs#770. <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [ ] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because they change the tests <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19720) <!-- Reviewable:end -->
| * | | Deny warnings in rust code globally in CIAneesh Agrawal2018-01-093-22/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the new environment variable support for `buildbot_steps.yml` to do this globally instead of having to patch every call, which is error prone. Set this variable globally in .travis.yml and appveyor.yml as well.
| * | | Update buildbot_steps lint to handle env variablesAneesh Agrawal2018-01-083-13/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/servo/saltfs/pull/687 added support for specifying environment variables in `buildbot_steps.yml`. Update the servo-tidy buildbot_steps.yml linter to reflect this. Use the voluptuous Python library (BSD 3-clause license) for validation in lieu of a much larger hand-written implementation. Update the tidy self tests to take into account the new error messages.
* | | | Auto merge of #19725 - glennw:update-wr-bs-line, r=mbrubeckbors-servo2018-01-091-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update WR (inset box shadow fix, line decoration optimizations) <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19725) <!-- Reviewable:end -->
| * | | | Update WR (inset box shadow fix, line decoration optimizations)Glenn Watson2018-01-091-2/+2
| | | | |
* | | | | Auto merge of #19721 - emilio:slotted, r=heycambors-servo2018-01-0921-392/+830
|\ \ \ \ \ | |_|_|/ / |/| | | / | | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Support ::slotted better. This allows to selector-match ::slotted, though we still don't parse it. Bug: 1425834, 1424607, 1425755 Reviewed-by: heycam MozReview-Commit-ID: ItELHkf2PMl <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19721) <!-- Reviewable:end -->
| * | | style: Update bindings.Emilio Cobos Álvarez2018-01-093-89/+67
| | | |
| * | | style: rename normal_style_rule_data to normal_rule_data for consistency.Emilio Cobos Álvarez2018-01-091-16/+16
| | | | | | | | | | | | | | | | MozReview-Commit-ID: HyWAsI7VP2p
| * | | style: Parse ::slotted() if the webcomponents pref is on.Emilio Cobos Álvarez2018-01-091-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | Bug: 1425834 Reviewed-by: heycam MozReview-Commit-ID: IOFneIgu7Rm
| * | | style: Match slots in reverse tree order.Emilio Cobos Álvarez2018-01-091-25/+35
| | | | | | | | | | | | | | | | MozReview-Commit-ID: 7OfHuDDn2Aw
| * | | style: Add a necessary check for style sharing between ::slotted() nodes.Emilio Cobos Álvarez2018-01-091-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If two elements are assigned to different slots, even within the same shadow root, they could match different rules, due to the slot being assigned to yet another slot in another shadow root. MozReview-Commit-ID: CfZJCVxIQzM
| * | | style: Make invalidation state also be with the ::slotted rules.Emilio Cobos Álvarez2018-01-096-188/+311
| | | | | | | | | | | | | | | | MozReview-Commit-ID: GYmsXYvL9vj
| * | | style: Store ::slotted rules separately on the cascade data, since they may ↵Emilio Cobos Álvarez2018-01-094-51/+161
| | | | | | | | | | | | | | | | | | | | | | | | cross the shadow boundary. MozReview-Commit-ID: EY9nK3169vv
| * | | style: Add invalidation support for ::slotted().Emilio Cobos Álvarez2018-01-097-98/+282
| | | | | | | | | | | | | | | | | | | | | | | | Bug: 1424607 Reviewed-by: heycam MozReview-Commit-ID: 8pIVUx27o7x
| * | | style: Implement GeckoElement::assigned_slot.Emilio Cobos Álvarez2018-01-082-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows to selector-match ::slotted, though we still don't parse it. Bug: 1425755 Reviewed-by: heycam MozReview-Commit-ID: ItELHkf2PMl
| * | | Update markup5ever.Emilio Cobos Álvarez2018-01-081-4/+4
| | | |
* | | | Auto merge of #19724 - hindenbug:19723-remove-fixme-comments, r=metajackbors-servo2018-01-081-2/+2
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove out of date FIXME comments from domimplementation <!-- Please describe your changes on the following line: --> This removes out of date FIXME comments from `domimplementation.rs` --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [ ] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #19723 <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because this is a comment cleanup <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19724) <!-- Reviewable:end -->
| * | | Remove out of date comments from domimplementationManoj2018-01-081-2/+2
|/ / /
* | | Auto merge of #19647 - tigercosmos:x3, r=jdmbors-servo2018-01-083-2/+25
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | replace DocumentSource::NotFromParser with DocumentSource::FromParser <!-- Please describe your changes on the following line: --> per https://github.com/servo/servo/pull/4297/files#r158729975 --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19647) <!-- Reviewable:end -->
| * | | replace DocumentSource::NotFromParser with DocumentSource::FromParsertigercosmos2018-01-063-2/+25
| | | |