aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Auto merge of #21593 - chansuke:format_webvr_traits, r=jdmbors-servo2018-09-032-3/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format components webvr_traits #21373 <!-- Please describe your changes on the following line: --> Format `components/webvr_traits` with `rstfmt` command. --- <!-- 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 part of #21373. - [x] 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/21593) <!-- Reviewable:end -->
| * Format components webvr_traits #21373chansuke2018-09-032-3/+13
| |
* | Auto merge of #21588 - emilio:gecko-sync, r=emiliobors-servo2018-09-0368-1391/+2701
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Sync changes from mozilla-central. See each individual commit for details. https://bugzilla.mozilla.org/show_bug.cgi?id=1488172 <!-- 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/21588) <!-- Reviewable:end -->
| * | style: Update style build script for bindgen changes.Emilio Cobos Álvarez2018-09-031-9/+2
| | |
| * | style: Fix unit tests build.Emilio Cobos Álvarez2018-09-033-9/+27
| | |
| * | Fix other parts of the Servo build.Emilio Cobos Álvarez2018-09-039-8/+27
| | |
| * | style: Fix servo build.Emilio Cobos Álvarez2018-09-034-4/+9
| | |
| * | Add rtl and ltr as static atoms.Emilio Cobos Álvarez2018-09-031-0/+2
| | |
| * | style: Appease tidy.Emilio Cobos Álvarez2018-09-032-2/+2
| | |
| * | style: Simplify the Lang pseudo-class stuff a bit.Emilio Cobos Álvarez2018-09-035-70/+45
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D4754
| * | style: Use proper escaping for pseudo-class strings.Emilio Cobos Álvarez2018-09-031-2/+5
| | | | | | | | | | | | | | | | | | | | | We always serialize as an atom, which is the previous behavior (though previous code was using string escaping which I think was not totally sound either...). Differential Revision: https://phabricator.services.mozilla.com/D4753
| * | style: Shrink CascadeData by turning the bloom filters into hash sets.Emilio Cobos Álvarez2018-09-031-45/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're wasting 1kb there, which is kind of stupid. The only advantage of using a bloom filter is that memory usage doesn't increase even if there's a gazillion attribute selectors and such. But: * For IDs we already have a bunch of data structures for invalidation and such which key on the id, so the bloom filter would be a very minor thing. * For attribute selectors we don't have such a data structure, but if people used a gazillion attribute selectors we should! Differential Revision: https://phabricator.services.mozilla.com/D4668
| * | style: Use an Atom to store pseudo-class string arguments.Cameron McCormack2018-09-032-22/+4
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D4740
| * | style: Use an Atom to represent Direction values in pseudo-classes.Cameron McCormack2018-09-035-58/+54
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D4730
| * | style: Make the allocation of AuthorStyles for ShadowRoot lazy.Emilio Cobos Álvarez2018-09-033-28/+41
| | | | | | | | | | | | | | | | | | | | | | | | So that we don't waste a bunch of memory with stuff like <svg:use>. I plan to shrink AuthorStyles further, but this should help regardless, and isn't very complex. Differential Revision: https://phabricator.services.mozilla.com/D4618
| * | style: Parse byte slice in PathParser.Boris Chiou2018-09-031-48/+46
| | | | | | | | | | | | | | | | | | | | | We only care about ascii char for svg path, so we could parse the string as byte slice. Differential Revision: https://phabricator.services.mozilla.com/D4168
| * | style: Define path() for clip-path.Boris Chiou2018-09-034-29/+143
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For now, |clip-path: path()| is chrome-only, and not for shape-outside, so we only implement the parser for clip-path. Besides, I didn't put path() in BasicShape because path() doesn't use the reference box to resolve the percentage or keywords (i.e. SVG path only accept floating point or integer number as the css pixel value). Therefore, I add it into ShapeSource, instead of BasicShape. Differential Revision: https://phabricator.services.mozilla.com/D3633
| * | style: Move SVGPathData and its parser into svg_path.rs.Boris Chiou2018-09-034-500/+522
| | | | | | | | | | | | | | | | | | | | | | | | SVGPathData will be used by clip-path and offset-path (and/or more on the properties which support <basic-shape>). Therefore, let's move SVGPathData out of motion.rs. Differential Revision: https://phabricator.services.mozilla.com/D3631
| * | style: Accept unknown webkit pseudo-element.Xidorn Quan2018-09-033-4/+35
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D4186
| * | style: Adjust an assertion to account for the changes from bug 1485930.Emilio Cobos Álvarez2018-09-031-1/+1
| | |
| * | style: Use cbindgen for basic_shape::FillRule.Boris Chiou2018-09-033-8/+12
| | | | | | | | | | | | | | | | | | | | | Just a minor fix to use cbindgen to avoid the conversion between basic_shape::FillRule and mozilla::StyleFillRule. Differential Revision: https://phabricator.services.mozilla.com/D4171
| * | style: Drop the manually implementation of ToCSS for BasicShape::Polygon.Boris Chiou2018-09-033-37/+27
| | | | | | | | | | | | | | | | | | | | | | | | The implementation of ToCSS for Polygon has some rule, and we could use skip_if to handle and serialization of fill-rule. However, we should derive ToCSS for the pair of LengthOrPercentages, so define a new type for it. Differential Revision: https://phabricator.services.mozilla.com/D4153
| * | style: Make tree pseudo-element prefix not case-sensitive.Xidorn Quan2018-09-033-5/+4
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D4185
| * | style: Remove an assertion that doesn't hold in some cases.Emilio Cobos Álvarez2018-09-031-5/+10
| | |
| * | style: Make the counters non-atomic counters and merge afterwards.Emilio Cobos Álvarez2018-09-033-11/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | This was consistently faster in the benchmark (even when counters were disabled, which was slightly suspicious, but...). Anyway, it's not really much code, most of it is FFI copy-pasta. Differential Revision: https://phabricator.services.mozilla.com/D3874
| * | style: Add a test for the use counters.Emilio Cobos Álvarez2018-09-032-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mostly testing that they work, and that they record what we expect them to record, that is, the actual property that was parsed, and not the properties that it'd resolve or expand to. That may be another tricky part for CSSOM, I think style setters would fail an alias test if implemented with the current setup (we do the property lookup in C++). Differential Revision: https://phabricator.services.mozilla.com/D3829
| * | style: Hook the use counters into StyleSheet parsing.Emilio Cobos Álvarez2018-09-038-1/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Still not hooked into telemetry, I talked with :janerik and :gfritzsche about that, but test incoming! This intentionally doesn't handle CSSOM and such for now, will file followups for those, though should be trivial. I want to unify / clean up how we do the use counters and the error reporting stuff for CSSOM, since the current function call still shows up in profiles, but that should be a follow-up. Differential Revision: https://phabricator.services.mozilla.com/D3828
| * | style: Add a very simple use counter implementation.Emilio Cobos Álvarez2018-09-034-45/+105
| | | | | | | | | | | | | | | | | | As simple as I could make it, for now. We can improve on this. Differential Revision: https://phabricator.services.mozilla.com/D3827
| * | style: Remove useless StyleDisplay conversion.Emilio Cobos Álvarez2018-09-032-20/+2
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3896
| * | style: Use a consistent style for longhands.Emilio Cobos Álvarez2018-09-0319-894/+1293
| | | | | | | | | | | | | | | | | | It's a bit of a mess. Differential Revision: https://phabricator.services.mozilla.com/D3892
| * | style: Make text-orientation, unicode-bidi, contain and will-change ↵Emilio Cobos Álvarez2018-09-033-18/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | non-animatable. Per recent CSSWG resolutions: https://github.com/w3c/csswg-drafts/issues/2737 https://github.com/w3c/csswg-drafts/issues/2751 Differential Revision: https://phabricator.services.mozilla.com/D3888
| * | style: Serialize clip-path and shape-outside using Servo.Emilio Cobos Álvarez2018-09-032-43/+85
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3653
| * | style: Add MallocSizeOf impls for 128-bit integers.Cameron McCormack2018-09-031-2/+2
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3947
| * | style: Use macro for path parser.Boris Chiou2018-09-031-163/+78
| | | | | | | | | | | | | | | | | | | | | | | | There are a lot of duplicates, so we use macro to refine them. Depends on D2963 Differential Revision: https://phabricator.services.mozilla.com/D2966
| * | style: Define offset-path and implement it in style system.Boris Chiou2018-09-038-3/+742
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Define OffsetPath & SVGPathData on the servo-side, and StyleMotion & StyleSVGPath on the gecko-side. We parse the SVG Path string into a vector of PathCommand. To build the gfx::Path, we will convert it into gfx::Path later in a different patch. The basic flow is: - Parse SVG Path String into SVGPathData (in Rust). - Use cbindgen to make sure the layout of PathCommand and StylePathCommand, and then set the Box[PathCommand] into nsTArray<StylePathCommand>. - Try to convert nsTArray<StylePathCommand> into gfx::Path. (This part will be implemented in a different patch.) Finally, we use the gfx::Path to create a motion path transform. The layout implementation is in the later patch. Depends on D2962 Differential Revision: https://phabricator.services.mozilla.com/D2963
| * | style: Simplify PropertyDeclarationBlock::get a little.Cameron McCormack2018-09-031-8/+2
| | | | | | | | | | | | | | | | | | Depends On D3747 Differential Revision: https://phabricator.services.mozilla.com/D3748
| * | style: Replace NormalDeclarationIterator return type with `impl Trait`.Cameron McCormack2018-09-031-52/+13
|/ / | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3747
* | Auto merge of #21585 - servo-wpt-sync:wpt_update_02-09-2018, r=jdmbors-servo2018-09-0355-152/+966
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sync WPT with upstream (02-09-2018) Automated downstream sync of changes from upstream as of 02-09-2018. [no-wpt-sync] <!-- 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/21585) <!-- Reviewable:end -->
| * | Update web-platform-tests to revision 0313d9f383d954ef401e79f3b669a5781aa3441aWPT Sync Bot2018-09-0255-152/+966
| | |
* | | Auto merge of #21584 - kingdido999:master, r=jdmbors-servo2018-09-024-19/+47
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format components dom_struct, domobject_derive and embedder_traits #21373 ```bash rustfmt $(find components/dom_struct -type f -name "*.rs") rustfmt $(find components/domobject_derive -type f -name "*.rs") rustfmt $(find components/embedder_traits -type f -name "*.rs") ``` <!-- 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/21584) <!-- Reviewable:end -->
| * | Format components dom_struct, domobject_derive and embedder_traits #21373kingdido9992018-09-034-19/+47
|/ /
* | Auto merge of #21569 - servo-wpt-sync:wpt_update_01-09-2018, r=jdmbors-servo2018-09-02844-3050/+19759
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sync WPT with upstream (01-09-2018) Automated downstream sync of changes from upstream as of 01-09-2018. [no-wpt-sync] <!-- 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/21569) <!-- Reviewable:end -->
| * | Update web-platform-tests to revision 2b7dace05fc1869398ee24f84fda4c0e4c0455aeWPT Sync Bot2018-09-02844-3050/+19759
| | |
* | | Auto merge of #21582 - kingdido999:master, r=jdmbors-servo2018-09-0217-462/+670
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format components devtools and devtools_traits #21373 <!-- Please describe your changes on the following line: --> ```bash rustfmt $(find components/devtools -type f -name "*.rs") rustfmt $(find components/devtools_traits -type f -name "*.rs") ``` <!-- 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/21582) <!-- Reviewable:end -->
| * | Format components devtools and devtools_traits #21373kingdido9992018-09-0217-462/+670
| | |
* | | Auto merge of #21580 - servo:what-did-you-expect, r=jdmbors-servo2018-09-0231-204/+47
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | Clean up more test expectations and add bug links to them <!-- 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/21580) <!-- Reviewable:end -->
| * | Clean up more test expectations and add bug links to themAnthony Ramine2018-09-0228-43/+28
| | |
| * | Mark /css/css-transitions/transitions-animatable-properties-01.html as ↵Anthony Ramine2018-09-023-161/+19
|/ / | | | | | | timeout=long
* | Auto merge of #21578 - kingdido999:master, r=jdmbors-servo2018-09-016-227/+446
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format components config #21373 <!-- Please describe your changes on the following line: --> ```bash rustfmt $(find components/config/ -type f -name "*.rs") ``` <!-- 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/21578) <!-- Reviewable:end -->
| * | Format components config #21373kingdido9992018-09-026-227/+446
|/ /