Fuguo
Fuguo
@PotatoCP Are you still doing this? Need to resolve conflict.
@mrobinson Can this PR be merged?
Chromium uses the `IsLayoutDependent` function to determine whether the style needs reflow. https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/css/css_computed_style_declaration.h;l=44;drc=69cb4a24bbe798ac9d4d7bb476a91fb1763df562;bpv=1;bpt=1 By default, reflow is not required. Reflow is triggered only when the style is `BlockSzie` `Bottom` `Height`...
`IsLayoutDependent` will be called here https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/css/css_computed_style_declaration.cc;l=326;drc=f0cb0ae3f9b142a11fdc5efc77e27a5d53b6b6cf
> I had the same issue today and found a workaround by using a custom visitor that implements `tracing::field::Visit` and passed that to `tracing::Event::record(&self, visitor: &mut dyn Visit)`. Maybe that's...
> You can see why this is problem if you run Servo on desktop, pinch zoom, and then scroll with the mouse wheel. Without the pan here, you cannot scroll...
> @kongbai1996 Is seems the PR contents and description have completely changed here? yes
> This one is quite flaky today. #40457 But I'm not sure if we can add a test for this, as we don't support touch dispatch in webdriver yet. I...
Can this be merged? Or do we need to rerun the WPT tests?
> What does this PR actually achieve? Can you show two videos, showing what's different after this PR? The size of `unscaled_viewport_size` is fixed during initialization and has no update...