aboutsummaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Fix serialization of namespace and universal selectorsSimon Sapin2017-05-161-20/+5
| |_|_|/ |/| | | | | | | | | | | | | | | Fix #16017 Fix #16020
* | | | Add size_of tests for geckolib selectorsSimon Sapin2017-05-162-1/+18
| | | |
* | | | Add size_of_test crateSimon Sapin2017-05-163-30/+4
| |_|/ |/| |
* | | Auto merge of #16860 - glennw:preserve3d, r=mbrubeckbors-servo2017-05-158-20/+4
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update WR (preserve-3d, AA improvements) <!-- 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/16860) <!-- Reviewable:end -->
| * | | Update WR (ellipse border-radius clip, preserve-3d functionality).Glenn Watson2017-05-168-20/+4
| | |/ | |/|
* | | Bug 1364412: Convert pseudo-elements to an enum. r=hiro,xidornEmilio Cobos Álvarez2017-05-161-25/+0
| | |
* | | Bug 1364412: Properly handle state restyle hints for pseudo-elements. r=bholleyEmilio Cobos Álvarez2017-05-161-3/+0
| | |
* | | Bug 1364412: Store full selectors in the Rule object. r=bholleyEmilio Cobos Álvarez2017-05-161-9/+6
| | | | | | | | | | | | | | | MozReview-Commit-ID: EKLKxNCghLD Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
* | | Auto merge of #16862 - CJKu:bug-1310885-part-6, r=heycambors-servo2017-05-151-2/+8
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | stylo: Pass Cached ImageValue from stylo back to gecko <!-- Please describe your changes on the following line: --> This is part 6 ~ part 9 patch in bug 1310885 gecko bug link: https://bugzilla.mozilla.org/show_bug.cgi?id=1310885 --- <!-- 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/16862) <!-- Reviewable:end -->
| * | Stylo: Pass style list's image_value to gecko.cku2017-05-161-2/+8
| | |
* | | Auto merge of #16295 - jdm:transition-safety, r=noxbors-servo2017-05-152-0/+37
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Root nodes for the duration of their CSS transitions This ensures that we can pass a node address as part of the asynchronous transition end notification, making it safe to fire the corresponding DOM event on the node from the script thread. Without explicitly rooting this node when the transition starts, we risk the node being GCed before the transition is complete. --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #14972 - [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/16295) <!-- Reviewable:end -->
| * | | Root nodes for the duration of their CSS transitions.Josh Matthews2017-05-152-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ensures that we can pass a node address as part of the asynchronous transition end notification, making it safe to fire the corresponding DOM event on the node from the script thread. Without explicitly rooting this node when the transition starts, we risk the node being GCed before the transition is complete.
* | | | Auto merge of #16699 - jdm:make-cross-origin-image-tests-fail, ↵bors-servo2017-05-1535-103/+648
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | r=ajeffrey,jgraham Make cross-origin canvas drawing tests use a same-origin redirect. These tests either pass or maintain existing known failures in Firefox and Chrome, and expose the problem in our current implementation that #15887 is addressing. <!-- 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/16699) <!-- Reviewable:end -->
| * | | Add same-origin redirect variants to 2d canvas security tests.Josh Matthews2017-05-1035-103/+648
| | | |
* | | | Auto merge of #16869 - nox:mime, r=noxbors-servo2017-05-155-30/+68
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | Ignore mime type parameters in nosniff (fixes #16049) <!-- 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/16869) <!-- Reviewable:end -->
| * | | Ignore mime type parameters in nosniffFabrice Desré2017-05-155-30/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch implements the following changes: - Only check for the toplevel/sublevel part of the mime type when deciding if it's a js or css resource, ignoring the mime parameters. - Fix the wpt tests that did not escape url parameters properly and also used an invalid syntax for the mime parameter. - Update the wpt manifest.
* | | | Auto merge of #16859 - nox:gradients, r=emiliobors-servo2017-05-152-61/+32
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rewrite style images with a good dose of generics 💉 <!-- 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/16859) <!-- Reviewable:end -->
| * | | | Rename LayerImage to ImageLayer and make it a type aliasAnthony Ramine2017-05-151-6/+9
| | | | |
| * | | | Clean up methods on specified::ImageAnthony Ramine2017-05-151-0/+1
| | | | |
| * | | | Refactor GradientAnthony Ramine2017-05-151-54/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In a similar way to Position, now specified and computed gradients share a common Gradient type defined in style::values::generics::image. This allows us to reuse most code for many style traits like ToCss, HasViewportPercentage and ToComputedValue. The test changes are the fallout of the disappearance of AngleOrCorner::None, which align our code to the spec for serialisation, where components that can be omitted should be omitted.
| * | | | Introduce style::values::generics::imageAnthony Ramine2017-05-151-1/+1
| |/ / /
* | | | Auto merge of #16835 - ferjm:bug1350175.line.column.css.rules, ↵bors-servo2017-05-155-2/+30
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | r=upsuper,SimonSapin Stylo: Bug 1350175 - Support getting line / column number of CSS rules - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors Bugzilla bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1350175 <!-- 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/16835) <!-- Reviewable:end -->
| * | | | Stylo: Bug 1350175 - Support getting line / column number of CSS rulesFernando Jiménez Moreno2017-05-125-2/+30
| | | | | | | | | | | | | | | | | | | | Fix font_cache_thread test
* | | | | Add PARSING_MODE_ALLOW_ALL_NUMERIC_VALUES to force to parse negative values.Hiroyuki Ikezoe2017-05-141-1/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per SVG spec [1], we should also parse negative color components values for SMIL, but currently Gecko does not support it either. [1] https://www.w3.org/TR/SVG/implnote.html#RangeClamping
* | | | | Make ParsingMode bitflags.Hiroyuki Ikezoe2017-05-144-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | assert_parsing_mode_match() is mostly the same as assert_restyle_hints_match().
* | | | | Rename LengthParsingMode to ParsingMode and LengthParsingMode::SVG to ↵Hiroyuki Ikezoe2017-05-144-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PasingMode::AllowUnitlessLength. We need another flag that represents allow-negative-number for SMIL, so this enum will also comprise the another parsing mode that allows negative number.
* | | | | Auto merge of #16829 - hiikezoe:disallow-negative-duration, r=emiliobors-servo2017-05-134-16/+18
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disallow negative duration for animation and transition <!-- 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 - [X] These changes fix #15343 <!-- Either: --> - [X] There are tests for these changes written by @simon-whitehead . Thank you! <!-- 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/16829) <!-- Reviewable:end -->
| * | | | Add test cases for transition-duration.Simon Whitehead2017-05-132-0/+18
| | | | |
| * | | | Disallow negative duration for animation and transition.Hiroyuki Ikezoe2017-05-132-16/+0
| |/ / /
* | | | Make font-feature-settings a subprop of fontXidorn Quan2017-05-132-12/+14
| | | |
* | | | Auto merge of #16833 - MortimerGoro:layout_alignment, r=SimonSapinbors-servo2017-05-123-0/+31
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix unsafe AtomicRefCell<T> transmutes in Layout component <!-- Please describe your changes on the following line: --> Fixes unsafe transmute between `AtomicRefCell<PersistentLayoutData>` and `AtomicRefCell<PartialPersistentLayoutData>` which have different memory alignment in 32 bit archs leading to SEGV crashes. See https://github.com/servo/servo/issues/16817 and https://github.com/servo/servo/pull/16816 mem::align_of values in 32 bit archs (e.g. Android): ``` PersistentLayoutData 8 PersistentLayoutData 4 AtomicRefCell<PersistentLayoutData> 8 AtomicRefCell<PartialPersistentLayoutData> 4 ``` mem::align_of values in 64 bit archs ``` PersistentLayoutData 8 PersistentLayoutData 8 AtomicRefCell<PersistentLayoutData> 8 AtomicRefCell<PartialPersistentLayoutData> 8 ``` --- <!-- 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 #16817 (github issue number if applicable). <!-- Either: --> - [x] 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/16833) <!-- Reviewable:end -->
| * | | | Fix unsafe AtomicRefCell<PersistentLayoutData> transmute due to memory ↵Imanol Fernandez2017-05-123-0/+31
| |/ / / | | | | | | | | | | | | alignment differences
* | | | Auto merge of #16506 - cbrewster:about_chaos, r=asajeffreybors-servo2017-05-123-0/+32
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make non-initial about:blank loads async <!-- 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 - [X] These changes fix #14856 (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/16506) <!-- Reviewable:end -->
| * | | | Add test to make sure contentDocument is updated after load maturesConnor Brewster2017-05-122-0/+31
| | | | |
| * | | | Make non-initial about:blank loads asynchronousConnor Brewster2017-05-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't update iframe pipeline until load completes To preserve the previous functionality of delaying load events when a new navigation is triggered, pending pipeline id represents the current pending load. The load event is only fired if the load message's pipeline id matches the pending pipeline id. Track frame size on Frame instead of Pipeline Disabled matchMedia test Track creator pipeline id
* | | | | Auto merge of #16802 - emilio:slim-down-slc, r=bholleybors-servo2017-05-121-3/+3
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Slim down SharedStyleContext, and do various other cleanups around the style crate. This slims down SharedStyleContext, in preparation for a few things. First, I would like to eventually move the stylist to the document in Servo, in order for it to hold the StyleSheetSet. Also, this gets rid of a fair amount of overhead while creating it in stylo. Fixes bug 1363245. <!-- 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/16802) <!-- Reviewable:end -->
| * | | | style: Parameterize the update and rebuild methods to take an iterator.Emilio Cobos Álvarez2017-05-121-3/+3
| |/ / / | | | | | | | | | | | | In preparation to avoid cloning the stylesheets while rebuilding in Gecko.
* | | | Auto merge of #16815 - Manishearth:cssup, r=metajackbors-servo2017-05-113-5/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bump cssparser to 0.13.3 <!-- 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/16815) <!-- Reviewable:end -->
| * | | | Bump cssparser to 0.13.3Manish Goregaokar2017-05-113-5/+5
| |/ / /
* | | | Auto merge of #16807 - mrobinson:scroll-clamping, r=emiliobors-servo2017-05-112-0/+36
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix clamping of scroll position in window.scrollBy For rightward and downward overflow the spec says: Let x be max(0, min(x, viewport scrolling area width - viewport width)). Let y be max(0, min(y, viewport scrolling area height - viewport height)). Previously, those operations were reversed, which created negative overflow even when the overflow direction was downward. This change ensures that Servo matches spec behavior. <!-- 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 _____ <!-- 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/16807) <!-- Reviewable:end -->
| * | | Fix clamping of scroll position in window.scrollByMartin Robinson2017-05-112-0/+36
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | For rightward and downward overflow the spec says: Let x be max(0, min(x, viewport scrolling area width - viewport width)). Let y be max(0, min(y, viewport scrolling area height - viewport height)). Previously, those operations were reversed, which created negative overflow even when the overflow direction was downward. This change ensures that Servo matches spec behavior.
* | | Handle importance when inserting into the rule tree.Bobby Holley2017-05-111-5/+2
| | |
* | | Refactor PositionAnthony Ramine2017-05-108-68/+65
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A specified position is now a struct made of two values of different types, the first one being PositionComponent<X>, and the second one PositionComponent<Y>. A position component is represented by the new enum PositionComponent<Side>, with the three values Center, Length(LengthOrPercentage), and Side(Side, Option<LengthOrPercentage>). Side keywords are represented by the X and Y enums, which don't include a value for the center keyword anymore. They are accompanied by the Side trait, which allows us to determine whether a side keyword is "left" or "top". This refactor simplified the parsing and serialisation code and exposed bugs in it, where it would reject valid <position> values followed by arbitrary tokens, and where it would fail to prefer "left" to "right" when serialising positions in basic shapes.
* | Rename the Interpolate trait to AnimatableBrian Birtles2017-05-101-1/+1
| |
* | Stylo: Bug 1355408 - add support for @-moz-documentFernando Jiménez Moreno2017-05-091-5/+10
| |
* | Auto merge of #16303 - froydnj:rayon-version-bump, r=jdmbors-servo2017-05-081-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bump required rayon version to 0.7 ...and bring jpeg-decoder along for the ride. Minor tweaks were necessary because of rayon API changes. - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors <!-- 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/16303) <!-- Reviewable:end -->
| * | bump required rayon version to 0.7Nathan Froyd2017-05-051-1/+1
| | | | | | | | | | | | | | | ...and bring jpeg-decoder along for the ride. Minor tweaks were necessary because of rayon API changes.
* | | Auto merge of #16762 - upsuper:buildtime-pseudo, r=emiliobors-servo2017-05-082-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Generate atom files at build-time The commits here basically do the following things: 1. move all generated files for gecko into "gecko/generated" so that we can copy all of them around 2. make regen_atoms.py generate file to the out dir rather than in-tree 3. make the build script invoke regen_atoms.py when bindgen feature is enabled <!-- 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/16762) <!-- Reviewable:end -->
| * | | Rename gecko_ prefix of pseudo_element_helper file.Xidorn Quan2017-05-081-1/+1
| | | |
| * | | Move binding files into another directory.Xidorn Quan2017-05-081-1/+1
| | | | | | | | | | | | | | | | So that we can copy them when not using bindgen altogether.