aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Remove old rendering backend.Glenn Watson2016-10-1855-3615/+426
| | | | | | | | | | | | | | This removes paint threads, rust-layers dependency, and changes optional webrender types to be required. The use_webrender option has been removed, however I've left the "-w" command line option in place so that wpt runner can continue to pass that. Once it's removed from there we can also remove the -w option. Once this stage is complete, it should be fine to change the display list building code to generate webrender display lists directly and avoid the conversion step.
* Auto merge of #13787 - lucasloisp:fix-12415/htmll-htmlo, r=emiliobors-servo2016-10-1612-22/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed incorrectly exposed webidl interfaces <!-- Please describe your changes on the following line: --> I removed exposure in Window and Worker to some interfaces that was not specified in the specification. Specifically: - HTMLLinkElement.webidl - HTMLMapElement.webidl - HTMLMediaElement.webidl - HTMLMetaElement.webidl - HTMLMeterElement.webidl - HTMLModElement.webidl - HTMLObjectElement.webidl - HTMLOListElement.webidl - HTMLOptGroupElement.webidl - HTMLOptionElement.webidl - HTMLOutputElement.webidl --- <!-- 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 - [X] These changes partially fix #12415 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because they do not add any feature <!-- 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/13787) <!-- Reviewable:end -->
| * Removed incorrectly exposed webidl interfacesLucas Lois2016-10-1612-22/+1
| | | | | | | | | | I removed exposure in Window and Worker to some interfaces that was not specified in the specification.
* | Auto merge of #11624 - g-k:wpt-cookie-path-tests, r=jdmbors-servo2016-10-164-0/+172
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | wpt cookie path tests Adds wpt tests for the cookie path attribute. Notes: * I included polyfills for fetch and promise to get tests running. Assuming these changes are OK, I'll remove them and add failure expectations so they don't get synced to w3c. * I didn't work out a way to test sending cookies when the request and cookie paths exactly match (e.g. request path `/cookies/path/match-exact-page.sub.html` and cookie path `/cookies/path/match-exact-page.sub.html` will not send cookies to `/cookies/resources/echo-json.py`) * test names could be clearer * everything can be squashed to one commit --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #8756 (github issue number if applicable). - [x] These changes do not require tests because they are tests (and if we test our tests do we need to test our test tests? ...) <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11624) <!-- Reviewable:end -->
| * test cookie path matchingGreg Guthe2016-10-144-0/+172
| |
* | Auto merge of #13785 - gterzian:use_veqdequeu_in_servoparser, r=noxbors-servo2016-10-162-5/+15
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | switch to using DOMRefCell<VeqDeque<String>> for ServoParser::pending… <!-- Please describe your changes on the following line: --> The field ServoParser::pending_input should be a DOMRefCell<VeqDeque<String>>, this lets us use VecDeque::pop_front instead of Vec::remove in ServoParser::take_next_input_chunk. --- <!-- 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 - [x] These changes fix #13690 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> …_input <!-- 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/13785) <!-- Reviewable:end -->
| * | switch to using DOMRefCell<VeqDeque<String>> for ServoParser::pending_inputGregory2016-10-162-5/+15
| | |
* | | Auto merge of #13747 - scotttrinh:canvas-webidl, r=Ms2gerbors-servo2016-10-166-36/+15
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Partial fix for #12415: expose Canvas interfaces <!-- Please describe your changes on the following line: --> Fix interface exposure per [the specs](https://html.spec.whatwg.org/multipage/scripting.html#2dcontext). One lingering question I have is if [`CanvasPath`](https://html.spec.whatwg.org/multipage/scripting.html#canvaspath) needs to be added to `interfaces.worker.js` since it *is* exposed to the `Worker` environment. I tried adding it to the `interfaces.worker.js` list but it failed tests at https://github.com/servo/servo/blob/master/tests/wpt/mozilla/tests/mozilla/interfaces.js#L82 --- <!-- 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 - [x] These changes contain a partial fix for #12415 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because _____ <!-- 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/13747) <!-- Reviewable:end -->
| * | | Partial fix for #12415: expose Canvas interfacesScott Trinh2016-10-136-36/+15
| | | |
* | | | Auto merge of #13715 - heycam:url-or-none, r=Manishearthbors-servo2016-10-157-156/+69
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Factor out a UrlOrNone type for -moz-binding and list-style-image to use <!-- Please describe your changes on the following line: --> r? @Manishearth --- <!-- 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 _____ <!-- 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/13715) <!-- Reviewable:end -->
| * | | | Make list-style-image use UrlOrNone.Cameron McCormack2016-10-162-81/+7
| | | | | | | | | | | | | | | | | | | | MozReview-Commit-ID: 6E82Tq5nrfC
| * | | | Factor out a UrlOrNone value type and make -moz-binding use it.Cameron McCormack2016-10-165-66/+62
| | | | | | | | | | | | | | | | | | | | MozReview-Commit-ID: L57QEf40e2m
| * | | | Use UrlExtraData::make_from to avoid manually cloning the ↵Cameron McCormack2016-10-161-11/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | ParserContextExtraData contents. MozReview-Commit-ID: FoCmI6swp5Q
* | | | Auto merge of #13756 - pcwalton:relative-position-clip, r=notriddlebors-servo2016-10-159-162/+251
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | layout: Rewrite clipping to be a two-phase process that takes physical border box positions and transforms into account. Clipping region computation now follows a simple process: (1) in the parent's coordinate system, parents store appropriate clipping regions into children; (2) each child moves its clipping region to its own coordinate system if necessary. Because clipping region computation is now based on stacking-relative border box positions and the `transform_rect` method, it can handle `position: relative` offsets and more types of transforms, such as scaling. Improves etsy.com. Closes #13753. <!-- 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/13756) <!-- Reviewable:end -->
| * | | layout: Rewrite clipping to be a two-phase process that takes physicalPatrick Walton2016-10-159-162/+251
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | border box positions and transforms into account. Clipping region computation now follows a simple process: (1) in the parent's coordinate system, parents store appropriate clipping regions into children; (2) each child moves its clipping region to its own coordinate system if necessary. Because clipping region computation is now based on stacking-relative border box positions and the `transform_rect` method, it can handle `position: relative` offsets and more types of transforms, such as scaling. Improves etsy.com. Closes #13753.
* | | Auto merge of #13765 - pcwalton:relative-position-vertical-percentage, r=emiliobors-servo2016-10-1414-40/+81
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | layout: Make percentages in `top` and `bottom` for relative positions relative to height, not width. Improves etsy.com. Closes #13760. r? @mbrubeck <!-- 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/13765) <!-- Reviewable:end -->
| * | layout: Make percentages in `top` and `bottom` for relative positionsPatrick Walton2016-10-1414-40/+81
| | | | | | | | | | | | | | | | | | | | | | | | relative to height, not width. Improves etsy.com. Closes #13760.
* | | Auto merge of #13703 - Rafagd:my_changes, r=Ms2gerbors-servo2016-10-145-73/+6
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replacing Node.rootNode for Node.getRootNode. - Changed the specification for the Node element to reflect the new standard. - Changed the implementation of the NodeMethod. I'm still waiting for some answers for the questions I've posted on #13699, but I'm already doing the pull request so I can get some feedback as well. --- <!-- 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 - [x] These changes fix #13699 (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- 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/13703) <!-- Reviewable:end -->
| * | | Replacing Node.prototype.rootNode for Node.prototype.getRootNode.Rafael Gomes Dantas2016-10-135-73/+6
| | | |
* | | | Auto merge of #13772 - bholley:version_bump, r=SimonSapinbors-servo2016-10-141-1/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update rust-stable-version to 1.12 This matches what Gecko CI will be using shortly: https://bugzilla.mozilla.org/show_bug.cgi?id=1306438 <!-- 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/13772) <!-- Reviewable:end -->
| * | | Update rust-stable-version to 1.12.Bobby Holley2016-10-141-1/+1
|/ / /
* | | Auto merge of #13759 - bholley:owning_handle, r=SimonSapinbors-servo2016-10-149-9/+134
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement OwningHandle in style I've also PR-ed this against upstream [1], but I don't want to block on that in case it takes a while to be merged / published. [1] https://github.com/Kimundi/owning-ref-rs/pull/15 <!-- 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/13759) <!-- Reviewable:end -->
| * | | Implement OwningHandle in style.Bobby Holley2016-10-139-9/+134
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I've also PR-ed this against upstream [1], but I don't want to block on that in case it takes a while to be merged / published. [1] https://github.com/Kimundi/owning-ref-rs/pull/15
* | | | Auto merge of #13751 - joewalker:font-kerning, r=Manishearthbors-servo2016-10-141-2/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for 'font-kerning' in geckolib Add support for 'font-kerning' in geckolib These changes fix #13667 $ ./mach build -d It looks like you passed an unrecognized argument into mach. $ ./mach test-tidy It looks like you are trying to run an unknown mach command: test-tidy I'm clearly doing something wrong here, but I'm not sure what - [x] These changes do not require tests because "[For stylo it's okay to not have tests since many of the reftests fail due to unrelated reasons](https://github.com/servo/servo/pull/13570#issuecomment-251354116)" :) <!-- 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/13751) <!-- Reviewable:end -->
| * | | | Add support for 'font-kerning' in geckolibJoe Walker2016-10-141-2/+7
| | | | |
* | | | | Auto merge of #13750 - servo:fetch-blob, r=Manishearthbors-servo2016-10-1411-64/+237
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement blob url support in the fetch stack. <!-- 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/13750) <!-- Reviewable:end -->
| * | | | | Update the bug number for Blob-XHR-revoke.html.Ms2ger2016-10-142-2/+2
| | | | | |
| * | | | | Implement blob url support in the fetch stack.Ms2ger2016-10-146-4/+147
| | | | | |
| * | | | | Stop spawning threads unnecessarily in FileManager.Ms2ger2016-10-141-12/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The spawned threads remain for select_file and select_files, as those may need to wait indefinitely for the user's response.
| * | | | | Avoid an unnecessary clone when calling FileManagerStore::try_read_file.Ms2ger2016-10-141-4/+4
| | | | | |
| * | | | | Add a FileManager to FetchContext.Ms2ger2016-10-144-38/+66
| | | | | |
| * | | | | Stop wrapping FileManager in an Arc.Ms2ger2016-10-143-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | It already contains an Arc internally.
| * | | | | Add missing type to uniformOutOfBounds.html.ini.Ms2ger2016-10-141-0/+1
|/ / / / /
* | | | | Auto merge of #13754 - Manishearth:refptr, r=emilio,mystorbors-servo2016-10-1312-79/+287
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add sugar for RefPtr<T>; use for `quotes` property Carry-over from https://bugzilla.mozilla.org/show_bug.cgi?id=1309848 r=mystor,emilio <!-- 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/13754) <!-- Reviewable:end -->
| * | | | | implement |quotes| propertyManish Goregaokar2016-10-142-2/+23
| | | | | | | | | | | | | | | | | | | | | | | | MozReview-Commit-ID: 5eAaJGXSkob
| * | | | | Add bindings for constructing nsStyleQuoteValuesManish Goregaokar2016-10-142-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | MozReview-Commit-ID: EjuzgZKMpPp
| * | | | | Add safer bindings for RefPtrManish Goregaokar2016-10-149-76/+249
| | | | | |
* | | | | | Auto merge of #13738 - pcwalton:table-vertical-align-absolute, r=mbrubeckbors-servo2016-10-134-34/+111
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | layout: Don't try to vertically align absolutely positioned children of table rows. Improves http://reddit.com/r/aww. r? @mbrubeck <!-- 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/13738) <!-- Reviewable:end -->
| * | | | | | layout: Don't try to vertically align absolutely positioned children ofPatrick Walton2016-10-134-34/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | table rows. Improves http://reddit.com/r/aww.
* | | | | | | Auto merge of #13682 - mrobinson:push_pop, r=pcwaltonbors-servo2016-10-134-479/+435
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Integrate stacking contexts into the display list <!-- Please describe your changes on the following line: --> --- <!-- 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: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Integrate stacking contexts into the display list by adding two new entry types, PushStackingContext and PopStackingContext. This allows us to eliminate the ugly offsets map that DisplayList used to contain and seems to speed up display list construction. With this approach we are able to also completely prune pseudo-stacking contexts from the final display list and remove their (minimal) overhead from display list traversal Traversing the display list is also a bit simpler now. Additionally, this will allow easier editing of the DisplayList to properly support scrolling roots. The push/pop entries can be duplicated to clone complex StackingContext trees between layers. <!-- 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/13682) <!-- Reviewable:end -->
| * | | | | | | Integrate stacking contexts into the display listMartin Robinson2016-10-134-479/+435
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Integrate stacking contexts into the display list by adding two new entry types, PushStackingContext and PopStackingContext. This allows us to eliminate the ugly offsets map that DisplayList used to contain and seems to speed up display list construction. With this approach we are able to also completely prune pseudo-stacking contexts from the final display list and remove their (minimal) overhead from display list traversal Traversing the display list is also a bit simpler now. Additionally, this will allow easier editing of the DisplayList to properly support scrolling roots. The push/pop entries can be duplicated to clone complex StackingContext trees between layers.
* | | | | | | | Auto merge of #13755 - fhartwig:early-return, r=KiChjangbors-servo2016-10-131-41/+41
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Invert conditional and return early from report_pending_exception <!-- Please describe your changes on the following line: --> Previously, almost the entire body of `report_pending_exception` was in a conditional branch. We now return early if the function body does not need to be executed. --- <!-- 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 - [X] These changes fix #13746 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because they don't add any new functionality. <!-- 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/13755) <!-- Reviewable:end -->
| * | | | | | | Invert conditional and return early from report_pending_exceptionFlorian Hartwig2016-10-131-41/+41
| | |_|_|_|/ / | |/| | | | |
* | | | | | | Auto merge of #13727 - mbrubeck:warnings, r=Manishearthbors-servo2016-10-131-1/+0
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix unused feature warning in build-cef <!-- 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/13727) <!-- Reviewable:end -->
| * | | | | | Remove obsolete try_borrow feature from CEFMatt Brubeck2016-10-121-1/+0
| | | | | | |
* | | | | | | Auto merge of #13737 - pcwalton:expand-matches, r=emiliobors-servo2016-10-134-24/+93
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | resources: Avoid using `:matches` in `presentational-hints.css`, since we don't support it. Based on IRC discussions, this was determined to be an acceptable stopgap in lieu of implementing `:matches`, which has complicated precedence rules. Improves Google search results. r? @nox <!-- 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/13737) <!-- Reviewable:end -->
| * | | | | | | resources: Avoid using `:matches` in `presentational-hints.css`, sincePatrick Walton2016-10-134-24/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we don't support it. Based on IRC discussions, this was determined to be an acceptable stopgap in lieu of implementing `:matches`, which has complicated precedence rules. Improves Google search results.
* | | | | | | | Auto merge of #13732 - pcwalton:inline-block-vertical-align-overflow, ↵bors-servo2016-10-1313-57/+97
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | r=notriddle layout: Use the margin box for vertical positioning of `inline-block` fragments if `overflow` is not `visible` per CSS 2.1 § 10.8.1. Additionally, this patch reverts the change introduced in #12642 in favor of the spec-compliant behavior described above. This patch also removes the `inline_block_overflow.html` reftest introduced in #3725, as the behavior it expected contradicted CSS 2.1 (and in fact the test fails in Gecko). The changes that this patch makes to `input_selection_a.html` and `input_selection_incremental_a.html` are necessary workarounds to make the tests pass in light of the fact that Servo's UA stylesheet applies `overflow: hidden` to `<input>` elements. I believe that the changes are not necessary in other rendering engines because they hard-code `overflow: hidden`-like behavior for `<input>` elements, while Servo uses the actual CSS `overflow: hidden` behavior. As far as I can tell, Servo's behavior is arguably more spec-compliant, but it remains to be seen how Web compatible it is. Improves the Google results pages. Closes #13707. r? @notriddle <!-- 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/13732) <!-- Reviewable:end -->
| * | | | | | | | layout: Use the margin box for vertical positioning of `inline-block`Patrick Walton2016-10-1313-57/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fragments if `overflow` is not `visible` per CSS 2.1 § 10.8.1. Additionally, this patch reverts the change introduced in #12642 in favor of the spec-compliant behavior described above. This patch also removes the `inline_block_overflow.html` reftest introduced in #3725, as the behavior it expected contradicted CSS 2.1 (and in fact the test fails in Gecko). The changes that this patch makes to `input_selection_a.html` and `input_selection_incremental_a.html` are necessary workarounds to make the tests pass in light of the fact that Servo's UA stylesheet applies `overflow: hidden` to `<input>` elements. I believe that the changes are not necessary in other rendering engines because they hard-code `overflow: hidden`-like behavior for `<input>` elements, while Servo uses the actual CSS `overflow: hidden` behavior. As far as I can tell, Servo's behavior is arguably more spec-compliant, but it remains to be seen how Web compatible it is. Improves the Google results pages. Closes #13707.
* | | | | | | | | Auto merge of #13743 - emilio:wayland-not-yet, r=larsbergstrombors-servo2016-10-132-8/+8
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | servo: Update glutin to pick the wayland disabling. <!-- Please describe your changes on the following line: --> --- <!-- 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 <!-- Either: --> - [x] These changes do not require tests because dependency bump. <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> r? @larsbergstrom <!-- 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/13743) <!-- Reviewable:end -->
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-25 18:23:25 +0000