The default bug view has changed. See this FAQ.

stylo: need to support skipping parent-display-based display fixup for anonymous boxes

RESOLVED FIXED in Firefox 55

Status

()

Core
CSS Parsing and Computation
P1
normal
RESOLVED FIXED
12 days ago
8 days ago

People

(Reporter: bz, Assigned: bz)

Tracking

(Blocks: 1 bug)

Trunk
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

We need this at least for the combobox display frame.  Without this, layout/reftests/forms/select/997709-2.html renders totally wrong.
This should be easily doable, we should just need to pass the relevant flag to precomputed_style_for_pseudo, either from servo directly, or from gecko if it's conditional depending on the frame.
Comment hidden (mozreview-request)
This doesn't make us pass any tests, because unfortunately our rendering of <select> is still all broken.
https://github.com/servo/servo/pull/15910 for the servo side.

Comment 5

12 days ago
mozreview-review
Comment on attachment 8846244 [details]
Bug 1346481.  Add stylo support for skipping parent-display-based fixup for anonymous boxes.

https://reviewboard.mozilla.org/r/119328/#review121272

r=me, with or without the following nit that I also left in the servo PR.

::: servo/components/script_layout_interface/wrapper_traits.rs:411
(Diff revision 1)
>                              let new_style =
>                                  context.stylist.precomputed_values_for_pseudo(
>                                      &style_pseudo,
>                                      Some(data.styles().primary.values()),
>                                      &context.default_computed_values,
> +                                    false,

I think it'd be more elegant to pass CascadeFlags directly to precomputed_values_for_pseudo. We'd pass CascadeFlags::empty() here.

I don't like multiple boolean arguments in general, and should be straight-forward to do, wdyt?
Attachment #8846244 - Flags: review?(emilio+bugs) → review+
Priority: -- → P1
Comment on attachment 8846244 [details]
Bug 1346481.  Add stylo support for skipping parent-display-based fixup for anonymous boxes.

https://reviewboard.mozilla.org/r/119328/#review121272

> I think it'd be more elegant to pass CascadeFlags directly to precomputed_values_for_pseudo. We'd pass CascadeFlags::empty() here.
> 
> I don't like multiple boolean arguments in general, and should be straight-forward to do, wdyt?

> I think it'd be more elegant to pass CascadeFlags directly to precomputed_values_for_pseudo

I agree.  I thought there was some reason it wasn't done that way.

I'll fix it.
Comment hidden (mozreview-request)

Comment 8

9 days ago
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/8fd06045940a
Add stylo support for skipping parent-display-based fixup for anonymous boxes.  r=emilio

Comment 9

8 days ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8fd06045940a
Status: NEW → RESOLVED
Last Resolved: 8 days ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.