1503428 - RDM is broken with enabling apz.allow_zooming

Status: RESOLVED WORKSFORME

(Core :: Graphics: WebRender, defect, P4)

Description

[Hiroyuki Ikezoe (:hiro)]

Attachment: A screenshot

Steps to reproduce

1. Enable apz.allow_zooming
2. Open a site (say https://gruntjs.com)
3. View it in RDM
4. Reload

See the attachment for the result.  I have no idea what's going on there at all.

I think this is probably not high priority.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Zooming is currently only partially implemented in WebRender. In particular, the only API we have to tell WR about zooming is the set_page_zoom/set_pinch_zoom, which applies to the entire content. So from the screenshot it looks like we are propagating the zoom to WR, but it's applying it to everything.

This is only a priority if we are planning/working on exposing APZ zoom in the RDM.

Comment 2

[Botond Ballo [:botond]]

What I said in bug 1503514 comment 3 applies here too, with the added twist that, as Kats mentioned, with WebRender zooming is in an even more primitive state (doesn't fully work on mobile yet either).