Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-03 12:40:06 +01:00
Code Issues Projects Releases Packages Wiki Activity

layout: Fix De Morgan's Law error in incremental reflow, allowing

Browse source 
float/absolute layout layout to be idempotent again.

Fixes the maze solver.
This commit is contained in:
Patrick Walton 2014-11-21 20:53:59 -08:00
parent e6e73b8da7
commit 95b57f55cd
1 changed files with 5 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

8
components/layout/block.rs
View file

@ -1264,8 +1264,10 @@ impl BlockFlow {
content_inline_size: Au, content_inline_size: Au,
optional_column_inline_sizes: Option<&[ColumnInlineSize]>) { optional_column_inline_sizes: Option<&[ColumnInlineSize]>) {
// Keep track of whether floats could impact each child. // Keep track of whether floats could impact each child.
let mut inline_start_floats_impact_child = self.base.flags.contains(IMPACTED_BY_LEFT_FLOATS); let mut inline_start_floats_impact_child =
let mut inline_end_floats_impact_child = self.base.flags.contains(IMPACTED_BY_RIGHT_FLOATS); self.base.flags.contains(IMPACTED_BY_LEFT_FLOATS);
let mut inline_end_floats_impact_child =
self.base.flags.contains(IMPACTED_BY_RIGHT_FLOATS);
let absolute_static_i_offset = if self.is_positioned() { let absolute_static_i_offset = if self.is_positioned() {
// This flow is the containing block. The static inline offset will be the inline-start // This flow is the containing block. The static inline offset will be the inline-start
@ -1332,7 +1334,7 @@ impl BlockFlow {
// and its inline-size is our content inline-size. // and its inline-size is our content inline-size.
{ {
let kid_base = flow::mut_base(kid); let kid_base = flow::mut_base(kid);
if !kid_base.flags.contains(IS_ABSOLUTELY_POSITIONED) || if !kid_base.flags.contains(IS_ABSOLUTELY_POSITIONED) &&
!kid_base.flags.is_float() { !kid_base.flags.is_float() {
kid_base.position.start.i = inline_start_content_edge kid_base.position.start.i = inline_start_content_edge
} }