1315608 - Null PDF zoom level between Page Width and 50% displayed while navigating with keyboard

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect)

Description

[Bogdan Maris [:bogdan_maris], Release Desktop QA]

Attachment: Gif showing the issue

[Note]:
- This is the same behavior as in bug 1194700 but with e10s disabled.

[Affected versions]:
- Firefox 45.5.0ESR
- Firefox 50.0 RC
- Latest Nightly 52.0a1
- Developer Edition 51.0a1

[Affected platforms]:
- Mac OS X 10.12
- Windows 7 64bit
- Windows 10 64bit
- Ubuntu 14.04 32bit

[Steps to reproduce]:
1. Start Firefox
2. Open any PDF (ex: http://www.cbu.edu.zm/downloads/pdf-sample.pdf)
3. Click on predefined zoom level button
4. Hit ESC button to close the dropdown but to keep the focus on it
5. Using left/right arrow keys go through the zoom levels
6. Hit Enter when the values are changed to activate the zoom level selected

[Expected result]:
- All the correct zoom levels are shown while navigating using keyboard.

[Actual result]:
- Between Page Width and 50% zoom levels there is a null value.

[Regression range]:
- This is NOT a recent regression. Not sure if this is an older regression though, I was able to reproduce it on old Nightly from 42.0a1 with e10s disabled, will investigate further ASAP.

[Additional notes]:
- Gif showing the issue is attached.

Comment 1

[Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)]

Thanks Bogdan. Since this looks like an older and minor regression I am not going to worry about it for ESR45 or for the 50/51 release. We can take a look at it for 52 though.

Comment 2

[Bogdan Maris [:bogdan_maris], Release Desktop QA]

I tracked down a regression using mozregression, but it's something very old, since pdf.js was not an add-on anymore.

Last good revision: 1158503601be (2012-05-21)
First bad revision: b038090f07c2 (2012-05-22)

Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1158503601be&tochange=b038090f07c2

d50e4a17308b	Benjamin Smedberg — Bug 740795 part C - Don't ship pdf.js as an extension, build changes, r=ted

Comment 3

[Benjamin Smedberg]

No comment. The PDF team should triage this with everything else.

Comment 5

[Bogdan Maris [:bogdan_maris], Release Desktop QA]

I just verified this is no longer an issue on Firefox 52 beta 8 across platforms (Windows 10 64bit, macOS 10.12.3 and Ubuntu 16.04 32bit).