Test for the "first paint" case of bug 1509575

RESOLVED FIXED in Firefox 66

Status

()

enhancement
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: JanH, Assigned: JanH)

Tracking

unspecified
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment)

No description provided.
Assignee

Updated

5 months ago
Blocks: 1509575
We need to setup a case where the visual viewport is smaller than the layout
viewport. Faking a pinch zoom through touch events is imprecise, whereas calling
windowUtils.setResolutionAndScaleTo(value > 1) to simulate the equivalent
effect of a pinch-zoom on the page led to the subsequent scroll attempt failing
quite often even after calling waitForApzFlushedRepaints().

So instead, we set up a very wide layout viewport through the meta viewport tag,
which on Desktop means additionally enabling the dom.meta-viewport.enabled pref.
(In reply to Jan Henning [:JanH] from comment #1)
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=e3d1d3f851c0bcdb8122b6b84a290c1d42d79b37

Great, the original push is restricted only to that particular test paths, but the retriggers totally ignore that and run something completely different...
So the test isn't fundamentally broken anywhere, but that Try push is totally useless with regards to possible intermittent failures.

This depends on "Part 7: Generally enable Visual Viewport for Mochitests. r?botond" from bug 1498812.

Depends on: 1498812

Comment 5

5 months ago
Pushed by mozilla@buttercookie.de:
https://hg.mozilla.org/integration/autoland/rev/0355f47af6ce
Test that resetting WindowUtils.isFirstPaint doesn't clobber APZ scroll position. r=botond
Flags: needinfo?(jh+bugzilla)

helper_bug1509575.html | visual viewport did scroll - got 18, expected 100

I wonder if perhaps the test is running in a window that sufficiently wide that 2000px of horizontal content is not enough to give it a 100px scroll range.

Depends on: 1519594

Comment 8

5 months ago
Pushed by mozilla@buttercookie.de:
https://hg.mozilla.org/integration/autoland/rev/81378c45a359
Test that resetting WindowUtils.isFirstPaint doesn't clobber APZ scroll position. r=botond
Assignee

Updated

5 months ago
Flags: needinfo?(jh+bugzilla)

Comment 9

5 months ago
bugherder
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.