diff options
author | bors-servo <metajack+bors@gmail.com> | 2015-08-11 12:57:02 -0600 |
---|---|---|
committer | bors-servo <metajack+bors@gmail.com> | 2015-08-11 12:57:02 -0600 |
commit | 7dc83e7820df43b1b617ae8dcf661398b0bd0842 (patch) | |
tree | e5ed747a201eb0486c718c571597d59000ebf1e5 /components/script/dom/htmlheadingelement.rs | |
parent | 7ce47266acc7c23de537905746b274b4a75424fa (diff) | |
parent | ae378a8c3e1b77fe4e1ab3b8602900d32844756c (diff) | |
download | servo-7dc83e7820df43b1b617ae8dcf661398b0bd0842.tar.gz servo-7dc83e7820df43b1b617ae8dcf661398b0bd0842.zip |
Auto merge of #7024 - pcwalton:whitespace-stripping, r=mbrubeck
layout: Rewrite whitespace stripping.
This patch makes Servo unconditionally strip whitespace before text run
scanning (assuming that the `white-space` property allows it). Whitespace
stripping during reflow is now only used for handling whitespace at the ends of
lines; reflow now never attempts to handle ignorable whitespace.
Many CSS tests pass now. There are some new failures, however.
The following reference tests now fail due to a pre-existing bug whereby
whitespace is used to calculate the position of inline hypothetical boxes for
elements with `display: inline; position: absolute`:
* `absolute-replaced-height-036.htm`
* `vertical-align-sub-001.htm`
* `vertical-align-super-001.htm`
The following reference tests fail due to a pre-existing bug whereby we don't
handle `font-size: 0` properly in inline reflow:
* `font-size-zero-1.htm`
* `font-size-zero-2.htm`
The following reference test fails due to the fact that it relied on our
incorrect insertion of whitespace to make room for the black background:
* `inline-formatting-context-007.htm`
r? @mbrubeck
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7024)
<!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom/htmlheadingelement.rs')
0 files changed, 0 insertions, 0 deletions