Open Bug 1552046 Opened 5 years ago Updated 2 years ago

Get viewport-resize-event-on-load-overflowing-page.html to pass (at least on Android)

Categories

(Core :: Layout, defect, P3)

defect

Tracking

()

Tracking Status
firefox68 --- affected

People

(Reporter: botond, Unassigned)

References

(Blocks 2 open bugs)

Details

After working around the test harness issue affecting viewport-resize-event-on-load-overflowing-page.html in bug 1550813, the test now performs the expected assertion, but it still fails. This bug tracks investigating and fixing the cause.

I did look at this failure a bit. We get two resize events on the test case, but the test assumes no resize event receives if the scrollbar thickness is zero (it's zero on our Android). The two resize events come from;

  1. a PresShell::SetVisualViewportSize call triggered by adding a viewport meta tag
  2. a PresShell::SetResolutionAndScaleTo call triggered by a before-first-paint event (the event calls MobileViewportManager::SetInitialViewport and the function calls RefreshViewportSize and it calls UpdateResolution)

As for 1), it's the first time we set the visual viewport size, so I think we don't need to fire a resize event for the case.
As for 2), it's the first paint for us, so I think we don't need to fire a resize event either.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.