1577258 - Simplify PresShell::ResizeReflowIgnoreOverride.

Status: RESOLVED FIXED

(Core :: Layout, task)

Description

[Emilio Cobos Álvarez [:emilio]]

I want this to:

• Thread all the resizes via that (this includes allowing it to not flush).
• Making it less scary to touch in general.

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1577258 - early-return for noop resizes.

This avoids doing wasted work and sending spurious resize
events if this case would be hit.

In practice, it cannot be hit yet, I think, because
callers do check for this and bail out earlier. But
there's no assertion to that respect so this shouldn't
hurt.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1577258 - Add a simpler resize reflow function for when we're not shrink-wrapping.

This is much easier than the existing ResizeReflowIgnoreOverride function, and
this will allow me to avoid flushing if needed (we already kinda do that
already with the "suppressResizeReflow" thing), which in turn allows me to
consolidate a bunch of the logic for resizes.

The function should be much easier to follow:

• Set the new viewport (async, doesn't do any work).
• Invalidate layout as needed due to the viewport change (that is, resize hint
  for the root frame, and invalidate isizes if needed). Also async and doesn't
  do any reflowing itself.
• Flush layout / do the reflowing all at once. I think we can stop doing this
  much more often now, but that's follow-up work.

Comment 3

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1577258 - Streamline the shrink-wrapping resize reflow code.

Now that this code path is on its own, we can write more straight-forward code.

Comment 4

[Emilio Cobos Álvarez [:emilio]]

Going to try landing these as incrementally as possible since this code is non-trivial.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/907ee4a0aa61
early-return for noop resizes. r=botond

Comment 6

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/907ee4a0aa61

Comment 7

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1577258 - Explicitly flush layout in an a11y test.

We have an optimization to avoid an expensive reflow from SetFullZoom, see
mSuppressResizeReflow1.

That was done because we used to do a full synchronous reflow right after. We no
longer do that, but due to that member we also don't invalidate!

My second patch in this bug changes the behavior of that flag so that we don't
synchronously reflow, but we do invalidate. So in turn this test before the
change wasn't really testing the zoomed code-path since it was using the clean
layout from before the zoom operation.

a11y getBounds and co. don't flush layout (they probably should), but since with
my patch we dirty the frame tree, and dirty frames return bogus offsets, the
test starts failing.

Flush layout explicitly to ensure we're testing the zoomed code path.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/49ff97acd5ce
Explicitly flush layout in an a11y test. r=eeejay

Comment 9

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/49ff97acd5ce

Comment 10

[Chris Peterson [:cpeterson]]

Adding this bug to GV's September sprint because it blocks Fenix image bug 1551659.

Comment 11

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/releases/mozilla-beta/rev/49ff97acd5ce

Comment 12

[Boris Zbarsky [:bzbarsky]]

Waiting for a response to https://phabricator.services.mozilla.com/D43799#inline-273207

Comment 13

[Emilio Cobos Álvarez [:emilio]]

Yeah, sorry, I'll be on PTO until Thursday. I replied, but I'll update the patch following your suggestions before re-requesting review.

Comment 14

[Boris Zbarsky [:bzbarsky]]

Yep, no problem. Thank you for following up!

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/37fd8d379530
Add a simpler resize reflow function for when we're not shrink-wrapping. r=bzbarsky

Comment 16

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/37fd8d379530

Comment 17

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8eeddedeedb8
Streamline the shrink-wrapping resize reflow code. r=botond

Comment 18

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/8eeddedeedb8