mirror of
https://github.com/servo/servo.git
synced 2025-08-02 20:20:14 +01:00
Mix stacking contexts into the positioned content list
Sometimes positioned content needs to be layered on top of stacking contexts. The layer synthesis code can do this, but the current design prevents it because stacking contexts are stored in a separate struct member. In order to preserve tree order, mix stacking contexts into the positioned content list, by adding a new StackingContextClass DisplayItem. Such items do not have a base DisplayItem. In some ways this simplifies the code, because we no longer have to have a separate code path in the StackingContextLayerCreator. Fixes #7779. Fixes #7983. Fixes #8122. Fixes #8310.
This commit is contained in:
parent
5c11c88e92
commit
c1a38e240a
13 changed files with 330 additions and 295 deletions
|
@ -1103,19 +1103,21 @@ impl BaseFlow {
|
|||
let all_items = match self.display_list_building_result {
|
||||
DisplayListBuildingResult::None => Vec::new(),
|
||||
DisplayListBuildingResult::StackingContext(ref stacking_context) => {
|
||||
stacking_context.display_list.all_display_items()
|
||||
stacking_context.display_list.flatten()
|
||||
}
|
||||
DisplayListBuildingResult::Normal(ref display_list) => display_list.all_display_items(),
|
||||
DisplayListBuildingResult::Normal(ref display_list) => display_list.flatten(),
|
||||
};
|
||||
|
||||
for item in &all_items {
|
||||
let paint_bounds = item.base().clip.clone().intersect_rect(&item.base().bounds);
|
||||
if !paint_bounds.might_be_nonempty() {
|
||||
continue;
|
||||
}
|
||||
if let Some(base_item) = item.base() {
|
||||
let paint_bounds = base_item.clip.clone().intersect_rect(&base_item.bounds);
|
||||
if !paint_bounds.might_be_nonempty() {
|
||||
continue;
|
||||
}
|
||||
|
||||
if bounds.union(&paint_bounds.bounding_rect()) != bounds {
|
||||
error!("DisplayList item {:?} outside of Flow overflow ({:?})", item, paint_bounds);
|
||||
if bounds.union(&paint_bounds.bounding_rect()) != bounds {
|
||||
error!("DisplayList item {:?} outside of Flow overflow ({:?})", item, paint_bounds);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue