Closed Bug 1516322 Opened 6 years ago Closed 5 years ago

MobileViewportManager::ShrinkToDisplaySizeIfNeeded() is broken in RDM on Windows with non WebRender

Categories

(Core :: Layout, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox66 --- wontfix
firefox70 --- fixed

People

(Reporter: hiro, Assigned: hiro)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

STR 1) Enable apz.allow_zooming 2) Open https://hg.mozilla.org/mozilla-central/file/d2bc993e14ea/layout/reftests/meta-viewport/initial-scale-0_5-ref.html in RDM 3) Make sure touch simulator is enabled 4) Reload the html A green box should be positioned at the top left of the viewport, but actually it's not. Interestingly this issue doesn't happen on WebRender at all. Anyway, I am going to skip some reftests that will be added in bug 1423013.
Note that apz.allow_zooming is not yet a supported configuration on desktop, so it is expected that some things will go wrong. I am adding this bug as a dependency of bug 1461360, which tracks the various issues that need to be fixed before we can enable apz.allow_zooming on desktop.
Summary: MobileViewportmanager::ShrinkToDisplaySizeIfNeeded() is broken in RDM on Windows with non WebRender → MobileViewportManager::ShrinkToDisplaySizeIfNeeded() is broken in RDM on Windows with non WebRender

Now all reftests in layout/reftests/meta-viewport pass on Windows too.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f09b90aa39322f7a82c84f9a0eb1b2d6ba7d3d2&selectedJob=254888756

I am going to enabled them on Windows.

Pushed by hikezoe.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/59d854235354 Enable all reftests in layout/reftests/meta-viewport on Windows. r=botond
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Assignee: nobody → hikezoe.birchill
QA Whiteboard: [qa-70b-p2]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: