1556777 - While in presentation mode the background webpage and browser UI can be zoomed in and out

Status: RESOLVED DUPLICATE of bug 1643508

(Core :: Panning and Zooming, defect, P3)

Description

[Daniel Cicas [:dcicas], Release QA]

Affected versions

• 68.0b7

Affected platforms

• ARM Windows 10

Steps to reproduce

1. Open a PDF file http://www23.statcan.gc.ca/imdb-bmdi/pub/instrument/3901_Q2_V3-eng.pdf
2. Enter presentation mode.
3. Zoom in and out using your fingers on the touchscreen.

Expected result

• The PDF file is not zoomed in or out.

Actual result

• The PDF presentation zooms and gets re-rendered. Also the browsers UI gets zoomed in and out in the background.

Regression range
Will return with a regression range as soon as possible.

Additional notes
All other zooming methods do not work while in presentation mode.
After exiting the presentation mode the browser and webpage is zoomed in/out

Comment 1

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

Did you manage to get a regression range for this? If it occurs prior to 2017 then don't bother; the range probably won't be that useful.

Comment 2

[Daniel Cicas [:dcicas], Release QA]

Hello,
As far as I know the ARM build (aarch 64 build) was introduced only with Firefox 67.0a1 (I might be mistaken, please correct me on this). I just tested this and it happens since the first Firefox 67 nightly build. Please tell me if I can help with anymore information.

Comment 3

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

Ok, thanks. It seems unlikely that this would be specific to ARM builds. More likely we just haven't run into it on touchscreen x86 machines yet. But I can test and triage accordingly.

Comment 4

[Botond Ballo [:botond]]

Indeed, I can reproduce this on an x86 Windows touchscreen.

(In reply to Daniel Cicas [:dcicas], Release QA from comment #0)

• The PDF presentation zooms and gets re-rendered. Also the browsers UI gets zoomed in and out in the background.

To clarify, full-zoom gets applied to the chrome of the PDF.js viewer, but not the browser chrome itself.

Comment 5

[Botond Ballo [:botond]]

Regression window goes back to bug 1298886, where we introduced the "pinch-gesture activates full zoom" codepath.

Comment 6

[Botond Ballo [:botond]]

Setting to fix-optional as this is a not a high-impact regression.

Comment 7

[Botond Ballo [:botond]]

Sorry, I did not mean to change components here. While this is related to reflowing zoom, the bug only occurs when the reflowing zoom is activated using a pinch-gesture, suggesting that it's an issue in APZCCallbackHelper::NotifyPinchGesture or thereabouts.

Comment 8

[Botond Ballo [:botond]]

Forward-duping to bug 1643508 which has a patch.