aboutsummaryrefslogtreecommitdiffstats
path: root/components
Commit message (Collapse)AuthorAgeFilesLines
* Auto merge of #6979 - mbrubeck:counter, r=pcwaltonbors-servo2015-08-041-9/+29
|\ | | | | | | | | | | | | | | | | | | | | | | Fix crash caused by negative numbers in list counters. Fixes #6818. The test still fails because of #6978. r? @pcwalton <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6979) <!-- Reviewable:end -->
| * Fix handling of negative numbers in list counters.Matt Brubeck2015-08-041-9/+29
| | | | | | | | Fixes #6818.
* | Auto merge of #6954 - dzbarsky:stroke_rect, r=glennwbors-servo2015-08-043-2/+17
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement strokeRect with zero-width or zero-height rects <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6954) <!-- Reviewable:end -->
| * | Implement strokeRect with zero-width or zero-height rectsDavid Zbarsky2015-08-043-2/+17
| | |
* | | Auto merge of #6945 - Manishearth:fixedme, r=jdmbors-servo2015-08-041-4/+6
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If let is not an issue for lints r? @Ms2ger <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6945) <!-- Reviewable:end -->
| * | | Remove fixme for if let in lints and replace with unreachable!()Manish Goregaokar2015-08-041-4/+6
| | | |
* | | | Auto merge of #6940 - pcwalton:inline-pseudo-repair-jumpiness, r=mbrubeckbors-servo2015-08-045-13/+54
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | layout: When repairing styles for incremental reflow, only repair styles of nodes that represent the dirty node, *including its pseudo-element*. r? @mbrubeck <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6940) <!-- Reviewable:end -->
| * | | | layout: When repairing styles for incremental reflow, only repairPatrick Walton2015-08-045-13/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | styles of nodes that represent the dirty node, *including its pseudo-element*. Fixes lots more jumpiness. A manual test, `inline-pseudo-repair-jumpiness.html`, has been added. I was unable to automate it, so I will file a followup issue on that.
* | | | | layout: Don't use the block container inline size as the initialPatrick Walton2015-08-041-4/+17
|/ / / / | | | | | | | | | | | | | | | | | | | | computed table inline size. Makes google.com fully centered.
* | | | Auto merge of #6950 - servo:struct-pattern, r=glennwbors-servo2015-08-041-0/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a comment to explain the struct pattern trick. https://github.com/servo/servo/issues/6912#issuecomment-127429643 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6950) <!-- Reviewable:end -->
| * | | | Add a comment to explain the struct pattern trick.Simon Sapin2015-08-041-0/+4
| | | | | | | | | | | | | | | | | | | | https://github.com/servo/servo/issues/6912#issuecomment-127429643
* | | | | Auto merge of #6946 - pcwalton:block-formatting-context-margins, r=mbrubeckbors-servo2015-08-041-3/+4
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | layout: If the container of a block formatting context has margins in the inline direction, subtract those from the inline size of preceding floats. Makes the content area on http://reddit.com/r/rust visible. r? @mbrubeck <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6946) <!-- Reviewable:end -->
| * | | | layout: If the container of a block formatting context has margins inPatrick Walton2015-08-031-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the inline direction, subtract those from the inline size of preceding floats. Makes the content area on http://reddit.com/r/rust visible.
* | | | | Auto merge of #6943 - dzbarsky:context, r=jdmbors-servo2015-08-042-90/+29
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clean up some methods in CanvasRendeingContext2D <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6943) <!-- Reviewable:end -->
| * | | | | Clean up some methods in CanvasRendeingContext2DDavid Zbarsky2015-08-032-90/+29
| | | | | |
* | | | | | Auto merge of #6938 - pcwalton:inline-repair-jumpiness, r=mbrubeckbors-servo2015-08-031-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | construct: When repairing styles for incremental reflow, only repair styles of nodes that represent the dirty node. Fixes jumpiness on many pages; e.g. the WPT results pages. For some reason, this would not reproduce with an automated test. r? @mbrubeck <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6938) <!-- Reviewable:end -->
| * | | | | | construct: When repairing styles for incremental reflow, only repairPatrick Walton2015-08-031-0/+5
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | styles of nodes that represent the dirty node. Fixes jumpiness on many pages; e.g. the WPT results pages. For some reason, this would not reproduce with an automated test.
* | | | | | Auto merge of #6937 - mbrubeck:hb_funcs, r=glennwbors-servo2015-08-034-28/+35
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Store Harfbuzz callback functions in a static var r? gw <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6937) <!-- Reviewable:end -->
| * | | | | Make Harfbuzz font_funcs staticMatt Brubeck2015-08-034-13/+20
| | | | | |
| * | | | | Sort gfx crates.io dependencies and move to topMatt Brubeck2015-08-031-15/+15
| |/ / / /
* | | | | Start reporting memory usage for Window and all nodes in all DOM trees for ↵Josh Matthews2015-08-0331-107/+309
| | | | | | | | | | | | | | | | | | | | frame treese in script tasks.
* | | | | Auto merge of #6932 - boghison:memrs, r=SimonSapinbors-servo2015-08-031-0/+42
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement HeapSizeOf for Url. Fixes #6912 Fixes #6912 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6932) <!-- Reviewable:end -->
| * | | | Implement HeapSizeOf for Url. Fixes #6912Bogdan Cuza2015-08-031-0/+42
| | |/ / | |/| |
* | | | Auto merge of #6784 - glennw:offset-ext, r=pcwaltonbors-servo2015-08-0319-24/+268
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement offsetParent/Top/Left/Width/Height. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6784) <!-- Reviewable:end -->
| * | | | Fix panic when called on root node, and update test expectations.Glenn Watson2015-08-041-2/+2
| | | | | | | | | | | | | | | | | | | | The three tests that are marked as failures are because they use offsetWidth as a dummy to force a layout flush. Now that these exist, they expose an unrelated bug with reflow.
| * | | | Implement offsetParent/Top/Left/Width/Height.Glenn Watson2015-08-0319-24/+268
| | |_|/ | |/| |
* | | | Auto merge of #6926 - Ms2ger:dwgs-args, r=jdmbors-servo2015-08-034-62/+54
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement a WorkerGlobalScopeInit struct to pass arguments to WorkerGlobalScope::new_inherited more easily. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6926) <!-- Reviewable:end -->
| * | | | Pass WorkerGlobalScopeInit to DedicatedWorkerGlobalScope::run_worker_scope.Ms2ger2015-08-032-28/+21
| | | | |
| * | | | Implement a WorkerGlobalScopeInit struct to pass arguments to ↵Ms2ger2015-08-032-39/+38
| | | | | | | | | | | | | | | | | | | | WorkerGlobalScope::new_inherited more easily.
| * | | | Take WorkerGlobalScope's worker_id out of its Option.Ms2ger2015-08-034-8/+8
| |/ / / | | | | | | | | | | | | | | | | Unsurprisingly, every worker has an id, so there is no need to wrap it in an Option.
* | | | Auto merge of #6920 - Wafflespeanut:NIT, r=jdmbors-servo2015-08-032-21/+19
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed an invalid FIXME and fixed some doc comments... There are some bad module-level doc comments in `flow.rs` which has directly affected [Servo's docs](http://doc.servo.org/layout/flow/index.html) and so, this fixes that. Oh, and #6728 is having a hard time getting closed and so I've also removed the `FIXME` comment related to it. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6920) <!-- Reviewable:end -->
| * | | | Removed an invalid FIXME and fixed a doc commentRavi Shankar2015-08-032-21/+19
| |/ / /
* | | | Auto merge of #6721 - mbrubeck:unicode-bidi, r=pcwaltonbors-servo2015-08-038-39/+124
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the unicode-bidi CSS property r? @pcwalton <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6721) <!-- Reviewable:end -->
| * | | Implement the unicode-bidi propertyMatt Brubeck2015-08-038-39/+124
| | |/ | |/|
* | | Auto merge of #6917 - Ms2ger:lock-alert, r=metajackbors-servo2015-08-031-1/+5
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lock and flush stdout in Window#alert. We use alert() to communicate test results to wptrunner. Unfortunately, sometimes the alert output is interleaved with other output on stdout, causing wptrunner to classify the test result as a timeout. I hope this will avoid that scenario. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6917) <!-- Reviewable:end -->
| * | Lock and flush stdout in Window#alert.Ms2ger2015-08-031-1/+5
| |/ | | | | | | | | | | | | We use alert() to communicate test results to wptrunner. Unfortunately, sometimes the alert output is interleaved with other output on stdout, causing wptrunner to classify the test result as a timeout. I hope this will avoid that scenario.
* | Auto merge of #6903 - glennw:fix-height-again, r=pcwaltonbors-servo2015-08-039-116/+151
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix percentage height calculation, absolute containing block height calculations. It's not possible to correctly determine during the css cascade whether the container height is explicitly specified. Additionally, the spec https://drafts.csswg.org/css2/visudet.html#the-height-property says this should affect the *used* height, rather than the computed height. This significantly improves the layout in #6643. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6903) <!-- Reviewable:end -->
| * | Fix percentage height calculation, absolute containing block height ↵Glenn Watson2015-08-039-116/+151
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | calculations. It's not possible to correctly determine during the css cascade whether the container height is explicitly specified. Additionally, the spec https://drafts.csswg.org/css2/visudet.html#the-height-property says this should affect the *used* height, rather than the computed height. This significantly improves the layout in #6643.
* | | Auto merge of #6902 - khflab:master, r=jdmbors-servo2015-08-031-3/+2
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | [FIX] Bug fix #6756 fixes #6756 waiting for review. Signed-off-by: Kayo Hamid <kayo@bvcupons.com.br> <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6902) <!-- Reviewable:end -->
| * | [FIX] Bug fix #6756Kayo Hamid2015-08-021-3/+2
| | | | | | | | | | | | Signed-off-by: Kayo Hamid <kayo@bvcupons.com.br>
* | | Auto merge of #6893 - HarryLovesCode:master, r=jdmbors-servo2015-08-028-40/+67
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | Fixes Issue #6866 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6893) <!-- Reviewable:end -->
| * | Fixes issue #6866Harrison G2015-08-028-40/+67
| |/
* | Auto merge of #6895 - pcwalton:better-transitions, r=glennwbors-servo2015-08-025-50/+91
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | layout: Tie transitions to the DOM node and finish them instantly when new styles are set. Tying transitions to the DOM node avoids quadratic complexity when updating them. Finishing transitions instantly when styles are updated makes our behavior more correct. r? @glennw <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6895) <!-- Reviewable:end -->
| * layout: Tie transitions to the DOM node and finish them instantly whenPatrick Walton2015-08-015-50/+91
| | | | | | | | | | | | | | | | | | | | new styles are set. Tying transitions to the DOM node avoids quadratic complexity when updating them. Finishing transitions instantly when styles are updated makes our behavior more correct.
* | Auto merge of #6722 - samfoo:sts-headers, r=jdmbors-servo2015-08-023-12/+35
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Obey Strict-Transport-Security header Resolves #6703. Done: * [x] When STS headers received, add the host to the HSTS list Todo: * [ ] Persist the in-memory list so that it's reloaded on the next browser boot * [ ] Add tests to `http_loader::load` - it's pretty well completely untested right now, but it's a bit gnarly to untangle and without mocking, it's hard to deal with the dependency on making a real network request. Writing a mock request object should be doable for testing, but there's a lot going on in the function right now. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6722) <!-- Reviewable:end -->
| * | Ensure STS headers are only honoured when they come from an HTTPS hostSam Gibson2015-08-021-18/+19
| | |
| * | Obey Strict-Transport-Security headerSam Gibson2015-08-023-10/+32
| |/
* / script: Use `Arc::make_unique` instead of `Arc::get_mut` when updatingPatrick Walton2015-08-022-12/+11
|/ | | | | | | inline styles. Transitions make the reasoning in the comment in the relevant sections not true.
* Auto merge of #6894 - pcwalton:double-precision-timestamps, r=metajackbors-servo2015-08-014-57/+60
|\ | | | | | | | | | | | | | | | | | | | | | | | | style: Switch animation timestamps to be doubles instead of floats. 32-bit floats are not enough to hold timestamps since the epoch and result in jank. r? @metajack <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6894) <!-- Reviewable:end -->
| * style: Switch animation timestamps to be doubles instead of floats.Patrick Walton2015-08-014-57/+60
| | | | | | | | | | 32-bit floats are not enough to hold timestamps since the epoch and result in jank.