aboutsummaryrefslogtreecommitdiffstats
path: root/components
Commit message (Collapse)AuthorAgeFilesLines
* Auto merge of #10830 - cjkenn:10743, r=Ms2gerbors-servo2016-04-281-4/+11
|\ | | | | | | | | | | | | | | | | | | | | | | | | DOMImplementation::createDocument should set content type based on namespace Set document content type based on the namespace. Standard: https://dom.spec.whatwg.org/#dom-domimplementation-createdocument Fixes #10743. <!-- 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/10830) <!-- Reviewable:end -->
| * [10743] Add content type to XmlDocument constructor based on namespacecjkenned2016-04-281-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [10743] Fix namespace in createDocument test [10743] Remove test ini file, match returns static strings instead of DOMString. [10743] Fix arguments to XMLDocument::new Update failing test [10743] Add content type to XmlDocument constructor based on namespace [10743] Fix namespace in createDocument test [10743] Remove test ini file, match returns static strings instead of DOMString. [10743] Fix arguments to XMLDocument::new Update failing test
* | Auto merge of #10895 - mbrubeck:byteindex, r=pcwaltonbors-servo2016-04-2818-462/+269
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use byte indices instead of char indices for text runs Replace character indices with UTF-8 byte offsets throughout all code dealing with text runs. This eliminates a lot of complexity when converting from one to the other, and interoperates better with the rest of the Rust ecosystem. For most code this is just a simple replacement of char indices with byte indices. In a few places like glyph storage and text fragment scanning, it also lets us get rid of code that existed only to map between bytes and chars. Also includes some related fixes to text shaping, discovered while working on this conversion. See the commit messages for details. r? @pcwalton <!-- 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/10895) <!-- Reviewable:end -->
| * | Replace range::Range with std::ops::Range in scriptMatt Brubeck2016-04-287-21/+14
| | |
| * | Use byte indices instead of char indices for text runsMatt Brubeck2016-04-2814-424/+258
| | | | | | | | | | | | | | | | | | | | | Replace character indices with UTF-8 byte offsets throughout the code dealing with text shaping and breaking. This eliminates a lot of complexity when converting from one to the other, and interoperates better with the rest of the Rust ecosystem.
| * | Remove unneeded add_nonglyph_for_char_indexMatt Brubeck2016-04-282-20/+1
| | | | | | | | | | | | | | | | | | This is a no-op, since a "non-glyph" is simply `GlyphEntry(0)`. This is the same as `GlyphEntry::initial()`, which all the entries are already initialized to.
| * | Don't assume the first glyph is part of the first charMatt Brubeck2016-04-281-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Shaper::save_glyph_results incorrectly starts its loop by setting glyph_span to a length of 1. This means that the `if glyph_span.len() == 0` test in the inner loop will never succeed. Instead the glyph span should start out empty, and a glyph should be added only as the corresponding char is found. For comparison, see the Gecko code this was ported from: https://hg.mozilla.org/mozilla-central/file/ab0044bf/gfx/thebes/gfxHarfBuzzShaper.cpp#l1682
* | | Auto merge of #10891 - heycam:text-style, r=bholleybors-servo2016-04-282-5/+15
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | Specialize text node style resolution so geckolib can avoid inheriting non-inherited structs. r? @bholley <!-- 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/10891) <!-- Reviewable:end -->
| * | Specialize text node style resolution so geckolib can avoid inheriting ↵Cameron McCormack2016-04-292-5/+15
| | | | | | | | | | | | non-inherited structs.
* | | Auto merge of #10887 - askeing:fix_10374, r=KiChjangbors-servo2016-04-281-1/+14
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dispatch error event when link elements get invalid response fix #10374 <!-- 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/10887) <!-- Reviewable:end -->
| * | | modified the code, following the suggestions from revieweraskeing2016-04-281-12/+8
| | | |
| * | | Dispatch error event when link elements get invalid responseaskeing2016-04-281-0/+17
| | | |
* | | | Auto merge of #10867 - danlrobertson:sandbox, r=KiChjangbors-servo2016-04-283-212/+60
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix logic for cors cache match The current logic for a cors cache match does not consider "credentials is false and request's credentials mode is not "include" or credentials is true." I could have missed something, but `CacheRequestDetails::credentials` is set to true if credentials mode is "include", and false otherwise. So `(!cors_cache.credentials && !cors_req.credentials) || cors_cache.credentials` would be directly following the spec, but unless I'm mistaken `cors_cache.credentials || !cors_req.credentials` is logically the same. Fixes: #10525 <!-- 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/10867) <!-- Reviewable:end -->
| * | | | Cleanup CORSCacheDaniel Robertson2016-04-272-193/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the CORSCache trait, CORSCacheSender, CORSCacheThreadMsg, and CORSCacheThread. Rename BasicCORSCache to CORSCache and keep its old implementation of CORSCache.
| * | | | Fix logic for cors cache matchDaniel Robertson2016-04-273-26/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current logic for a cors cache match does not consider "credentials is false and request's credentials mode is not "include" or credentials is true."
* | | | | Auto merge of #10159 - pcwalton:iframes-know-their-pipelines, r=jdmbors-servo2016-04-284-9/+23
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | script: Make iframes know their pipeline IDs at all times, even after navigation. Since WebRender uses the pipeline ID stored in the iframe element to determine which pipeline to display, it had better be kept up to date! Closes #9919. r? @jdm <!-- 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/10159) <!-- Reviewable:end -->
| * | | | | script: Make iframes know their pipeline IDs at all times, even afterPatrick Walton2016-04-284-9/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | navigation. Since WebRender uses the pipeline ID stored in the iframe element to determine which pipeline to display, it had better be kept up to date! Closes #9919.
* | | | | | layout: Translate floats flowing out of blocks to compensate for inlinePatrick Walton2016-04-281-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | direction margins. Improves Reddit /r/rust.
* | | | | | layout: Reset the speculated inline size of floats out to compensate forPatrick Walton2016-04-281-9/+14
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | margins for all blocks, not just block formatting contexts. Partial fix for Reddit /r/rust.
* | | | | Auto merge of #10855 - servo:unit, r=noxbors-servo2016-04-281-0/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make css-properties.json checking a proper unit test. <!-- 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/10855) <!-- Reviewable:end -->
| * | | | | Make css-properties.json checking a proper unit test.Simon Sapin2016-04-281-0/+1
| | | | | |
* | | | | | Auto merge of #10880 - mbrubeck:char-at, r=noxbors-servo2016-04-283-64/+48
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code cleanup in gfx::text * Fix deprecation warnings by replacing `str::char_at` and related functions with iterators. * Replace some uses of `range::Range` with `std::ops::Range`. <!-- 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/10880) <!-- Reviewable:end -->
| * | | | | Get rid of unnecessary clamping in text shapingMatt Brubeck2016-04-271-3/+0
| | | | | |
| * | | | | Stop using deprecated `str::char_*` methodsMatt Brubeck2016-04-273-23/+15
| | | | | | | | | | | | | | | | | | | | | | | | Fixes deprecation warnings in the gfx crate.
| * | | | | Replace range::Range with std::ops::Range in text shaping codeMatt Brubeck2016-04-272-49/+44
| | | | | |
* | | | | | Auto merge of #10694 - fitzgen:profile-traces-to-file, r=SimonSapinbors-servo2016-04-2814-10/+751
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a method for dumping profiles as self-contained HTML w/ timeline visualization This commit adds the `--profiler-trace-path` flag. When combined with `-p` to enable profiling, it dumps a profile as a self-contained HTML file to the given path. The profile visualizes the traced operations as a Gantt-chart style timeline. Example output HTML file: http://media.fitzgeraldnick.com/dumping-grounds/trace-reddit.html Mostly I made this because I wanted to see what kind of data the profiler has, and thought that this might be useful for others as well. I'm happy to add tests if we can figure out how to integrate them into CI, but I'm not sure how that would work. Thoughts? <!-- 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/10694) <!-- Reviewable:end -->
| * | | | | | Add a method for dumping self-contained HTML timeline profilesNick Fitzgerald2016-04-2714-10/+751
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds the `--profiler-trace-path` flag. When combined with `-p` to enable profiling, it dumps a profile as a self-contained HTML file to the given path. The profile visualizes the traced operations as a gant-chart style timeline.
* | | | | | | Remove extraneous script_chan parameter from Trusted::newKeith Yeung2016-04-2815-62/+38
| |_|_|_|_|/ |/| | | | |
* | | | | | Auto merge of #10871 - daoshengmu:canvasWithoutContexts, r=jdmbors-servo2016-04-271-9/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issue #10755 Canvas without contexts While canvas without contexts, we just need to return the function without drawing any pixel. Fixes #10755. <!-- 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/10871) <!-- Reviewable:end -->
| * | | | | Avoid drawing while canvases without contexts.Daosheng Mu2016-04-281-9/+3
| | | | | |
* | | | | | Auto merge of #10816 - SunyDays:errorevent_argument, r=frewsxcvbors-servo2016-04-271-0/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the errorevent argument to Document::createEvent new_uninitialized function on ErrorEvent has already been in components/script/dom/errorevent.rs close #10738 <!-- 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/10816) <!-- Reviewable:end -->
| * | | | | | Implement the errorevent argument to Document::createEventSunyDays2016-04-271-0/+3
| | |/ / / / | |/| | | |
* | | | | | Auto merge of #10647 - ConnorGBrewster:parse_xml, r=jdmbors-servo2016-04-275-29/+56
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Finish hooking up XML parser This is a work in progress PR for #10581. I just want to make sure I am headed in the right direction. cc @jdm <!-- 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/10647) <!-- Reviewable:end -->
| * | | | | Finish hooking up XML parserConnor Brewster2016-04-275-29/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | added script integration with xml5ever Updated test expectations Removed timeout test expectation Refactors application/xhtml+xml is treated as HTML Updated xml5ever Updated Text Expectations
* | | | | | Auto merge of #10869 - frewsxcv:dry, r=jdmbors-servo2016-04-271-9/+11
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DRY out `StreamedResponse::from_http_response`. <!-- 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/10869) <!-- Reviewable:end -->
| * | | | | | DRY out `StreamedResponse::from_http_response`.Corey Farwell2016-04-271-9/+11
| | | | | | |
* | | | | | | Auto merge of #10866 - broesamle:issue10856-a, r=Ms2gerbors-servo2016-04-275-170/+178
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move abstract parser infrastructure from servohtmlparser.rs to parse Fixes #10856 <!-- 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/10866) <!-- Reviewable:end -->
| * | | | | | | Move abstract parser infrastructure from servohtmlparser.rs to parseMartin Broesamle2016-04-275-170/+178
| | | | | | | |
* | | | | | | | Auto merge of #10848 - heycam:struct-accessor, r=bholleybors-servo2016-04-2725-23/+49
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Servo_GetStyleFoo functions to get style structs from GeckoComputedValues r?@bholley <!-- 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/10848) <!-- Reviewable:end -->
| * | | | | | | | Add Servo_GetStyleFoo functions to get style structs from GeckoComputedValues.Cameron McCormack2016-04-2725-23/+49
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While we're here we also: * remove any code conditional on style_struct.gecko_ffi_name, since all style structs now do have a corresponding Geckos struct * add new UIReset and XUL style structs, so that all Gecko structs are now present (apart from Variables, which is special)
* | | | | | | | Auto merge of #10868 - frewsxcv:loadcancelled, r=Ms2gerbors-servo2016-04-274-6/+6
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add `NetworkError::LoadCancelled` variant. <!-- 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/10868) <!-- Reviewable:end -->
| * | | | | | | | Remove superfluous end-of-file whitespace.Corey Farwell2016-04-271-1/+0
| | | | | | | | |
| * | | | | | | | Add `NetworkError::LoadCancelled` variant.Corey Farwell2016-04-273-5/+6
| |/ / / / / / /
* | | | | | | | Auto merge of #10838 - bd339:iss9551, r=KiChjangbors-servo2016-04-271-1/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement step 3 according to spec Goes towards fixing #9551 <!-- 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/10838) <!-- Reviewable:end -->
| * | | | | | | | Implement step 3 according to specbd3392016-04-271-1/+2
| | | | | | | | |
* | | | | | | | | Return a Result<i64, ()> from do_parse_integerAkshit Khurana2016-04-261-7/+7
| |_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | Fixes #10860
* | | | | | | | Auto merge of #10862 - asajeffrey:only-the-chrome-survives, r=SimonSapinbors-servo2016-04-261-0/+2
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't kill the root pipeline when randomly killing pipelines. Useful for stress-testing the browser chrome, e.g. testing #10334. <!-- 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/10862) <!-- Reviewable:end -->
| * | | | | | | Don't kill the root pipeline when randomly killing pipelines.Alan Jeffrey2016-04-261-0/+2
| |/ / / / / /
* | | | | | | Auto merge of #10820 - SunyDays:focusevent_argument, r=Ms2gerbors-servo2016-04-262-0/+8
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the "focusevent" argument to Document::createEvent close #10739 <!-- 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/10820) <!-- Reviewable:end -->
| * | | | | | Implement the focusevent argument to Document::createEventSunyDays2016-04-272-0/+8
| | | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-06-03 08:35:19 +0000