1710059 - [fission] Hit testing scaled remote iframes is broken.

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[Emilio Cobos Álvarez (:emilio)]

Try to hit some of the elements in: http://crisal.io/tmp/scaled-frame-complex.html

Comment 1

[Emilio Cobos Álvarez (:emilio)]

I think this is more APZ-land, but some observations:

• APZ hit-testing itself seems to work fine (since I can drag the scrollbar thumb just fine), it's main-thread hit-testing which seems confused.
• The layout position doesn't seem to get transformed properly. If I do the position change using transform-origin (the second iframe) then it works.
• The ParentToChildConversionMatrix in BrowserParent.cpp has the right scale in both cases, but the translation is wrong for the first one.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1710059 - Fix order of transform application in WebRenderScrollDataWrapper. r=jrmuizel,botond

When we have a both bottom and top layer, we weren't applying the
transforms in the right order.

Do you know what the right way to test this is?

Comment 3

[Chris Peterson [:cpeterson]]

Tracking for Fission M7a because this bug blocks M7a bug 1699837.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3ca325d7f1bd
Fix order of transform application in WebRenderScrollDataWrapper. r=botond

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3ca325d7f1bd