aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Auto merge of #22116 - jdm:gleamup, r=noxbors-servo2018-11-051-13/+13
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update gleam. This allows running the three.js examples on Pixel 1 devices. - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #22082 - [x] These changes do not require tests because no real android test coverage. <!-- 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/22116) <!-- Reviewable:end -->
| * | | | | Update gleam.Josh Matthews2018-11-051-13/+13
| | | | | |
* | | | | | Auto merge of #22099 - pyfisch:net-fmt, r=jdmbors-servo2018-11-0540-3048/+5217
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format net and net_traits crates Manually formatted the fetch_scheme file portion to limit right drift. Part of #21373 <!-- 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/22099) <!-- Reviewable:end -->
| * | | | | | Rustfmt net_traits cratePyfisch2018-11-0310-131/+229
| | | | | | |
| * | | | | | Reduce right drift in scheme_fetchPyfisch2018-11-031-124/+108
| | | | | | |
| * | | | | | Rustfmt net cratePyfisch2018-11-0330-2881/+4968
| | | | | | |
* | | | | | | Auto merge of #22051 - mandreyel:track-focused-bc-of-each-tlbc, r=cbrewsterbors-servo2018-11-051-76/+116
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update Constellation to track each browser's focused browsing context Since there may be multiple browsers (top-level browsing contexts), each has a focused browsing context. However, we were not keeping track of each browser's focused browsing context, so e.g when switching tabs the `Constellation::focused_browsing_context_id` would not be set to the switched-to browser's focused browsing context. This PR introduces a `browser_ids` `HashMap` in constellation, that maps each of the top-level browsing context's ids to their currently focused browsing context's id, so that when the active browser is changed with the `SelectBrowser` message, we can look up and restore the selected browser's focused browsing context. This is a wip. For one, I'm not a fan of adding another hash map to constellation, and since there already is a hash map for keeping track of a browser's joint session history, we could introduce some `Browser` struct to hold data for a browser like its session history and focused browsing context (and possibly more later). But wanted to implement the bare-bones logic to first ensure correctness and will refactor later. Also, we may need new tests but I'm not sure. --- <!-- 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 #17401 (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/22051) <!-- Reviewable:end -->
| * | | | | | Remove Constellation::focused_browsing_context_idmandreyel2018-11-021-69/+39
| | | | | | |
| * | | | | | Add browsers map to Constellation for per browser bookkeepingmandreyel2018-11-021-56/+85
| | | | | | |
| * | | | | | Update Constellation to track each browser's focused browsing contextmandreyel2018-11-021-21/+62
| | | | | | |
* | | | | | | Auto merge of #22097 - fabrizio8:unused_scroll, r=emiliobors-servo2018-11-051-38/+5
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unused scroll Combined on_scroll_window_event with on_scroll_start_window_event and on_scroll_end_window_event --- <!-- 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 #22075 (github issue number if applicable). <!-- Either: --> - [X] There are tests for these changes: - Ran unit-test, test-tidy with no errors, and tested scrolling on webpages - [ ] These changes do not require tests <!-- 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/22097) <!-- Reviewable:end -->
| * | | | | | Removed unused in_scroll_transaction and removed import of std::time:Instant ↵fabrizio82018-11-041-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | since it was only used by in_scroll_transaction.
| * | | | | | removed an unused variablefabrizio82018-11-031-1/+0
| | | | | | |
| * | | | | | fix #22075fabrizio82018-11-021-33/+2
| | |/ / / / | |/| | | | | | | | | | | | | | | | Combined on_scroll_window_event with on_scroll_start_window_event and on_scroll_end_window_event
* | | | | | Auto merge of #21864 - ferjm:android.gst.binaries.update, r=SimonSapinbors-servo2018-11-051-3/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update GStreamer binaries for Android. Include required codecs - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors This should finally work for both armv7 and x86. I moved the binaries from Github to S3. <!-- 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/21864) <!-- Reviewable:end -->
| * | | | | | Update GStreamer binaries for Android. Include required codecsFernando Jiménez Moreno2018-11-051-3/+5
| | |_|/ / / | |/| | | |
* | | | | | Auto merge of #22111 - emilio:gecko-sync, r=emiliobors-servo2018-11-0547-1087/+1326
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Sync changes from mozilla-central. See each individual commit for details. https://bugzilla.mozilla.org/show_bug.cgi?id=1504644 <!-- 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/22111) <!-- Reviewable:end -->
| * | | | | style: Fix servo build.Emilio Cobos Álvarez2018-11-054-3/+17
| | | | | |
| * | | | | style: Add the safe area constant names as atoms.Emilio Cobos Álvarez2018-11-051-0/+4
| | | | | |
| * | | | | Fix tidy issues.Emilio Cobos Álvarez2018-11-054-6/+4
| | | | | |
| * | | | | Update remaining references to cssparser 0.24.Emilio Cobos Álvarez2018-11-055-15/+15
| | | | | |
| * | | | | style: Simplify invalid custom property handling.Emilio Cobos Álvarez2018-11-051-61/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's a bit useless to keep a set of invalid properties if we're going to use them just to reject lookups into another key. This makes it more consistent with the cascade / no-references code, and should not change behavior. Differential Revision: https://phabricator.services.mozilla.com/D9632
| * | | | | style: Implement the env() function with hardcoded zeros for safe-area-inset.Emilio Cobos Álvarez2018-11-0511-137/+347
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Intent to Implement and Ship: https://groups.google.com/d/msg/mozilla.dev.platform/EVKyR1B87T0/_l-_qK8SAAAJ Differential Revision: https://phabricator.services.mozilla.com/D9609
| * | | | | style: Simplify the SVG animation code.Emilio Cobos Álvarez2018-11-055-156/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's overly generic for no good reason. Differential Revision: https://phabricator.services.mozilla.com/D10844
| * | | | | style: Move animation of svg-related bits outside of animated_properties.Emilio Cobos Álvarez2018-11-053-205/+215
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Being in mako is unnecessary, and makes it harder to debug and such. Differential Revision: https://phabricator.services.mozilla.com/D10843
| * | | | | style: Remove nscsspropertyid_is_{animatable,transitionable}.Emilio Cobos Álvarez2018-11-053-27/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are better ways, plus the existing code didn't handle aliases at all (not that it needed to, but it's better if it does). Differential Revision: https://phabricator.services.mozilla.com/D10838
| * | | | | style: Move various font-related animation code to its own file.Emilio Cobos Álvarez2018-11-053-147/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Similarly, the code is not trivial and doesn't really need to be in mako. Differential Revision: https://phabricator.services.mozilla.com/D10842
| * | | | | style: Move various length animation implementations to its own file.Emilio Cobos Álvarez2018-11-053-112/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's nicer, I think. Differential Revision: https://phabricator.services.mozilla.com/D10841
| * | | | | style: Move the keyframes property priority stuff outside of ↵Emilio Cobos Álvarez2018-11-052-64/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | animated_properties. Similarly, no need to be here. Differential Revision: https://phabricator.services.mozilla.com/D10840
| * | | | | style: Move shorthand IDL order stuff out of animated_properties.Emilio Cobos Álvarez2018-11-052-28/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Doesn't really need to be in a mako file. Differential Revision: https://phabricator.services.mozilla.com/D10839
| * | | | | style: Don't keep a separate list of ignored-when-colors-disabled longhands.Emilio Cobos Álvarez2018-11-052-36/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most of the change is moving sets around to be static functions on LonghandIdSet. I think I like that pattern, but I can also make the new set a global static and add mako code to be `pub` or something. Though I think the LonghandIdSet::foo().contains(..) pattern is nice to read :) Differential Revision: https://phabricator.services.mozilla.com/D10653
| * | | | | style: Don't match document author rules if not needed for revalidation.Emilio Cobos Álvarez2018-11-051-23/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you're in a ShadowRoot and can share style with a sibling, the sharing code is smart enough to skip document author rules. But then it could get confused if you also include document rules, since revalidation selectors are matched against these. This is not a correctness issue, because we're matching more than what we need, and avoid sharing if we failed. Also fix the detection for user rules in any_applicable_rule_data. Differential Revision: https://phabricator.services.mozilla.com/D10117
| * | | | | style: Use references in the shapes code.Emilio Cobos Álvarez2018-11-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It doesn't make much sense to return const UniquePtr<Foo>& for something that can't be null, it's just confusing. Also make more stuff actually const. Differential Revision: https://phabricator.services.mozilla.com/D10647
| * | | | | style: Fix inconsistent CRISPEDGES constant name.Cameron McCormack2018-11-051-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D10452
| * | | | | style: Support unprefixed image-rendering: crisp-edges.Cameron McCormack2018-11-051-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For now, we keep supporting the prefixed version, since there are examples/instructions on the Web that don't include an unprefixed value. Differential Revision: https://phabricator.services.mozilla.com/D10451
| * | | | | style: Handle reversed ranges in @font-face descriptors.Cameron McCormack2018-11-051-4/+19
| | | | | | | | | | | | | | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D10327
| * | | | | style: Drop "mozilla" prefix in cbindgen_types in ServoBindings.toml.Boris Chiou2018-11-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A minor update to drop the redundant "mozilla" namespace prefix in `cbindgen_types` array. Depends on D10305 Differential Revision: https://phabricator.services.mozilla.com/D10325
| * | | | | style: Use alias for StylePathCommand.Boris Chiou2018-11-052-13/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So we could drop transmute in svg_path related functions. Differential Revision: https://phabricator.services.mozilla.com/D10140
| * | | | | style: Use alias for StyleFillRule.Boris Chiou2018-11-058-25/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This needs to update the "fill-rule" and "clip-rule" to use predefined_type to avoid some compilation errors. Differential Revision: https://phabricator.services.mozilla.com/D10142
| * | | | | style: Use alias for StyleDisplay and StyleDisplayMode.Boris Chiou2018-11-052-18/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Map these two types to their original rust type in rust-bindgen. Differential Revision: https://phabricator.services.mozilla.com/D10141
| * | | | | style: Use alias for StyleAppearance.Boris Chiou2018-11-051-29/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So we could avoid generating it in rust-bindgen and drop transmute. Differential Revision: https://phabricator.services.mozilla.com/D10304
| * | | | | style: Add a special list for cbindgen types to avoid generating redundant ↵Boris Chiou2018-11-051-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rust types. We will blacklist this type and add a module raw line to map the gecko type to its rust type (as an alias). Differential Revision: https://phabricator.services.mozilla.com/D10303
| * | | | | style: Interpolate the angle between mis-matched rotate() functions when the ↵Brian Birtles2018-11-051-2/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | angle of one is zero. Bug: 1501176 Reviewed-by: hiro
| * | | | | style: Update cbindgen due to breaking change.Emilio Cobos Álvarez2018-11-051-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/eqrion/cbindgen/pull/233 changed the way one of the options we use work. I think the new behavior is better, but we should do this sooner rather than later, and fix broken builds. Differential Revision: https://phabricator.services.mozilla.com/D10301
| * | | | | style: Support ::before / ::after on ::slotted pseudos.Emilio Cobos Álvarez2018-11-052-57/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See https://github.com/w3c/csswg-drafts/issues/3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994
| * | | | | style: Ignore border-image-source when overriding document colors.Sean Voisen2018-11-052-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D10017
| * | | | | style: Add a fast path for querySelector{,All} when we have classes or tags ↵Emilio Cobos Álvarez2018-11-051-13/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in the rightmost compound. Before this patch we were only optimizing the case of a single selector, which is fine, but not enough to catch ones like .foo .bar or so. This patch allows us to optimize classes and tags in the rightmost compound, while keeping the current optimization for #id selectors. Need to profile this, but code-wise should be ready for review. Differential Revision: https://phabricator.services.mozilla.com/D9351
| * | | | | style: Don't allow auto in grid line names.Emilio Cobos Álvarez2018-11-052-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See https://github.com/w3c/csswg-drafts/issues/2856. Differential Revision: https://phabricator.services.mozilla.com/D9882
| * | | | | style: Remove unused style constant.Emilio Cobos Álvarez2018-11-051-6/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also remove specified-value-only keywords, since those are handled only in Rust code and C++ doesn't need to know about them. Differential Revision: https://phabricator.services.mozilla.com/D9634
* | | | | Auto merge of #22109 - servo-wpt-sync:wpt_update_04-11-2018, r=jdmbors-servo2018-11-0444-789/+769
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sync WPT with upstream (04-11-2018) Automated downstream sync of changes from upstream as of 04-11-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/22109) <!-- Reviewable:end -->