aboutsummaryrefslogtreecommitdiffstats
path: root/components
Commit message (Collapse)AuthorAgeFilesLines
* tmpJosh Matthews2018-08-231-2/+2
|
* Auto merge of #21341 - servo:webgl, r=avadacatavrabors-servo2018-08-233-141/+155
|\ | | | | | | | | | | | | | | | | | | Properly check limit in gl.activeTexture() Fixes #20531. <!-- 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/21341) <!-- Reviewable:end -->
| * Properly check limit in gl.activeTexture()Anthony Ramine2018-08-233-141/+155
| |
* | Auto merge of #21487 - cdisselkoen:remove-comment, r=jdmbors-servo2018-08-221-3/+0
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove outdated comment in stylist.rs <!-- Please describe your changes on the following line: --> This comment on `cascade_style_and_visited` is unclear because it refers to `is_link`, which is not a parameter of that function. It used to refer to a flag in `CascadeFlags` back when there was a `cascade_flags: CascadeFlags` parameter; that parameter was removed in commit cd04664. I don't believe it accurately reflects how the code works anymore, and is best just removed. --- <!-- 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 they only touch comments <!-- 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/21487) <!-- Reviewable:end -->
| * | Remove outdated comment in stylist.rsCraig Disselkoen2018-08-221-3/+0
| |/
* | Auto merge of #21430 - Eijebong:ws-origin, r=jdmbors-servo2018-08-225-6/+17
|\ \ | |/ |/| | | | | | | | | | | Properly set the origin on websocket messages <!-- 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/21430) <!-- Reviewable:end -->
| * Properly set the origin on websocket messagesBastien Orivel2018-08-165-6/+17
| |
* | Upgraded to SM 60Alan Jeffrey2018-08-2027-309/+328
| |
* | Auto merge of #21452 - emilio:gecko-sync, r=emiliobors-servo2018-08-1940-1338/+1928
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: Import changes from mozilla-central. See each individual commit for details. <!-- 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/21452) <!-- Reviewable:end -->
| * | Appease tidy.Emilio Cobos Álvarez2018-08-184-31/+26
| | |
| * | Port servo to the new media query system.Emilio Cobos Álvarez2018-08-183-134/+57
| | | | | | | | | | | | | | | Port `width`, and also add the `scan` media feature so I don't need to add ugliness just to workaround the unused keyword_evaluator macro.
| * | Add `scan` as a static atom.Emilio Cobos Álvarez2018-08-181-0/+1
| | |
| * | Add `width` as a static atom.Emilio Cobos Álvarez2018-08-181-0/+1
| | |
| * | style: Relax the version requirement of num-integer.Emilio Cobos Álvarez2018-08-181-1/+1
| | |
| * | style: Make webkit device-pixel-ratio media queries a proper alias to ↵Emilio Cobos Álvarez2018-08-182-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | resolution. According to the spec: https://compat.spec.whatwg.org/#css-media-queries-webkit-device-pixel-ratio And to the Chromium implementation: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/css/media_query_evaluator.cc?l=366&rcl=1d7328865bcf06a687aafc18ff95d55317030672 They're no different than resolution. In our implementation `resolution` does slightly different stuff. Given we still haven't shipped -webkit-device-pixel-ratio, making this match resolution looks better than the opposite. Differential Revision: https://phabricator.services.mozilla.com/D3588
| * | style: Use AspectRatio directly for RangeOrOperator::evaluate.Xidorn Quan2018-08-182-12/+29
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3587
| * | style: Remove use of `fnv` in bloom.rs.Nicholas Nethercote2018-08-183-43/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To support that, this patch also does the following. - Removes the insert(), remove() and might_contain() methods, because they are specialized versions of insert_hash(), remove_hash(), and might_contain_hash(), and they are only used by tests within this file. - Moves hash() from the top level into create_and_insert_some_stuff(). - Changes create_and_insert_some_stuff() so that instead of hashing consecutive integers, it instead hashes stringified consecutive integers, which matches real usage a little better. - Raises the false_positives limit a little to account for the above changes. Bug: 1484096 Reviewed-by: heycam
| * | style: Deduplicate system metric atoms.Emilio Cobos Álvarez2018-08-181-25/+23
| | | | | | | | | | | | | | | | | | | | | Now that :-moz-system-metric is gone, there's no real reason for the atoms to be separate. Differential Revision: https://phabricator.services.mozilla.com/D3497
| * | style: Add scrollbar-width property.Xidorn Quan2018-08-181-0/+11
| | | | | | | | | | | | | | | Bug: 1475033 Reviewed-by: heycam
| * | style: Remove an inaccurate and useless debug message.Emilio Cobos Álvarez2018-08-181-5/+0
| | |
| * | style: Manually inline class and ID getters.Emilio Cobos Álvarez2018-08-184-83/+137
| | | | | | | | | | | | | | | | | | | | | Somewhat ugly but hopefully not too much. Somehow it ends up removing more lines than adding. Differential Revision: https://phabricator.services.mozilla.com/D3536
| * | style: Add some spec links to media queries.Emilio Cobos Álvarez2018-08-181-2/+23
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3489
| * | style: Simplify some code in NoCalcLength::parse_dimension.Xidorn Quan2018-08-181-38/+26
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3473
| * | style: Rewrite media queries so that they work on an evaluator function.Emilio Cobos Álvarez2018-08-1810-782/+1374
| | | | | | | | | | | | | | | | | | | | | | | | This moves most of the code to be Rust, except potentially some evaluator functions, and allows to unblock the use case from any-hover / any-pointer and remove nsMediaFeatures. Differential Revision: https://phabricator.services.mozilla.com/D2976
| * | style: Simplify visited-related code in invalidation.Emilio Cobos Álvarez2018-08-181-78/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We match with AllLinksVisitedAndUnvisited for style invalidation, and we already do a subtree restyle because :visited matching doesn't depend on the actual element state. So all this stuff is just not needed. The comment points to the attribute tests in bug 1328509, but those still trivially pass with this change. I think this was unneeded since I introduced AllLinksVisitedAndUnvisited, or maybe since https://github.com/servo/servo/pull/19520. In any case it doesn't really matter, and I already had done this cleanup in my WIP patches for bug 1406622, but I guess this is a slightly more suitable place to land them :) Differential Revision: https://phabricator.services.mozilla.com/D3305
| * | style: no-op visited changes earlier if visited links are disabled.Emilio Cobos Álvarez2018-08-182-41/+14
| | | | | | | | | | | | | | | | | | | | | | | | We force a repaint from ContentStateChangedInternal if visited links are disabled, and that's observable. Let's cut it off as early as we can to avoid timing attacks even when :visited is disabled. Differential Revision: https://phabricator.services.mozilla.com/D3304
| * | style: Generate static atom hash in StaticAtoms.py.Cameron McCormack2018-08-181-3/+6
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3295
| * | style: Remove support for multiple static atom sources.Cameron McCormack2018-08-181-35/+24
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3285
| * | style: Move CSS anonymous box atoms to nsGkAtoms.Cameron McCormack2018-08-181-28/+13
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3284
| * | style: Move CSS pseudo-element atoms to nsGkAtoms.Cameron McCormack2018-08-182-30/+30
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3283
| * | style: Define atom type in nsGkAtomList.h.Cameron McCormack2018-08-181-9/+11
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3282
| * | style: remove unused kernel32-sys dependency from style.Nathan Froyd2018-08-181-3/+0
| | | | | | | | | | | | | | | Bug: 1483344 Reviewed-by: emilio
| * | Use encoding_rs for XPCOM string encoding conversions.Henri Sivonen2018-08-181-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correctness improvements: * UTF errors are handled safely per spec instead of dangerously truncating strings. * There are fewer converter implementations. Performance improvements: * The old code did exact buffer length math, which meant doing UTF math twice on each input string (once for length calculation and another time for conversion). Exact length math is more complicated when handling errors properly, which the old code didn't do. The new code does UTF math on the string content only once (when converting) but risks allocating more than once. There are heuristics in place to lower the probability of reallocation in cases where the double math avoidance isn't enough of a saving to absorb an allocation and memcpy. * Previously, in UTF-16 <-> UTF-8 conversions, an ASCII prefix was optimized but a single non-ASCII code point pessimized the rest of the string. The new code tries to get back on the fast ASCII path. * UTF-16 to Latin1 conversion guarantees less about handling of out-of-range input to eliminate an operation from the inner loop on x86/x86_64. * When assigning to a pre-existing string, the new code tries to reuse the old buffer instead of first releasing the old buffer and then allocating a new one. * When reallocating from the new code, the memcpy covers only the data that is part of the logical length of the old string instead of memcpying the whole capacity. (For old callers old excess memcpy behavior is preserved due to bogus callers. See bug 1472113.) * UTF-8 strings in XPConnect that are in the Latin1 range are passed to SpiderMonkey as Latin1. New features: * Conversion between UTF-8 and Latin1 is added in order to enable faster future interop between Rust code (or otherwise UTF-8-using code) and text node and SpiderMonkey code that uses Latin1. Bug: 1402247 Reviewed-by: Nika,erahm,froydnj
| * | style: Rename StyleUserInterface to StyleUI.Xidorn Quan2018-08-181-1/+1
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3276
| * | style: Remove unused BorrowedAtom.Cameron McCormack2018-08-181-4/+0
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3159
| * | style: Swap order of values in 'overflow' shorthand property.L. David Baron2018-08-181-6/+6
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D3069
| * | style: Use function pointer rather than Fn trait object for ↵Xidorn Quan2018-08-181-4/+5
| | | | | | | | | | | | | | | | | | collect_property_completion_keywords. Differential Revision: https://phabricator.services.mozilla.com/D2985
| * | style: Implement flow-relative values for resize property.Xidorn Quan2018-08-187-9/+87
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D2908
| * | style: Make several more specified values Copy.Xidorn Quan2018-08-183-4/+8
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D2932
| * | style: Add check for non-Copy specified value to ensure specified_is_copy ↵Xidorn Quan2018-08-183-1/+29
| | | | | | | | | | | | | | | | | | always returns the right result. Differential Revision: https://phabricator.services.mozilla.com/D2931
| * | style: Put overflow: -moz-scrollbar-* behind pref.Xidorn Quan2018-08-181-22/+28
| | | | | | | | | | | | Differential Revision: https://phabricator.services.mozilla.com/D2845
| * | style: Use union to wrap different shape-like types. r=xidornBoris Chiou2018-08-182-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | 1. We will add more shape-like types in the future, so it's better to use union to reduce the memory usage. 2. Those shape-like types are mutual exclusive, so we could use union to wrap them. Differential Revision: https://phabricator.services.mozilla.com/D2746
* | | Network: Preserve HEAD on 303 redirecttigercosmos2018-08-191-1/+1
|/ / | | | | | | fixed #21449. This is an update for spec.
* | Auto merge of #21442 - kingdido999:rustfmt/bluetooth, r=jdmbors-servo2018-08-175-343/+489
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format components bluetooth and bluetooth_traits <!-- Please describe your changes on the following line: --> ```bash rustfmt components/bluetooth/*.rs rustfmt components/bluetooth_traits/*.rs ``` --- <!-- 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 a part of #21373 . - [x] These changes do not require tests because it's code format change. <!-- 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/21442) <!-- Reviewable:end -->
| * | Format component bluetooth_traitskingdido9992018-08-173-37/+47
| | |
| * | Format component bluetoothkingdido9992018-08-172-306/+442
| | |
* | | Format components atoms #21373kingdido9992018-08-171-1/+2
|/ /
* | Auto merge of #21428 - kingdido999:rustfmt/allocator, r=jdmbors-servo2018-08-161-17/+25
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Format component allocator <!-- Please describe your changes on the following line: --> Format `components/allocator` with: ```bash rustfmt components/allocator/*.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. Waiting for #21423 to be resolved. - [x] `./mach test-tidy` does not report any errors - [ ] These changes fix a part of #21373. - [x] These changes do not require tests because they format the code only. <!-- 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/21428) <!-- Reviewable:end -->
| * | Format component allocator #21373kingdido9992018-08-161-17/+25
| |/
* / The WS protocols should be case sensitive, not insensitiveBastien Orivel2018-08-161-3/+3
|/
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-27 11:38:28 +0000