1018932 - [OSX] PDF viewer zoom issues when using Cmd + mouse control

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect, P3)

Description

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

Reproducible on:
Firefox 30 beta 9 - 20140529161749
 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:30.0) Gecko/20100101 Firefox/30.0
Latest Firefox 31 Aurora - 20140602004003 
 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Firefox/31.0
Latest Firefox 32 Nightly - 20140602030202
 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:32.0) Gecko/20100101 Firefox/32.0

Steps to reproduce:
1. Open a PDF (e.g. https://enregistrement.grandlyon.fr/Enregistrement/CG/CG.pdf).
2. Zoom in/out using Ctrl + mouse scroll -> the document properly zooms in/out while the PDF viewer control toolbar remains at a fixed size (same as when using the toolbar +/- zoom controls).
3. Zoom in/out using Cmd + mouse scroll.

Expected results:
The document properly zooms in/out, same as when using Ctrl + mouse scroll.

Actual results:
PDF viewer control toolbar zooms in together with the document itself.

Notes: 
1. Cmd + keyboard "+"/"-" zooms in/out same as Ctrl + keyboard "+"/"-". Cmd should behave just like Ctrl in this case, and zoom the document instead of the whole page.
2. Reproduces on Firefox 20.0 (when Cmd + mouse scroll was introduced for zoom control) so it's not a regression.

Comment 1

[u279076]

I missed that this is not a regression, therefore no need to set status flags for this one.

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Not adding to the Firefox team backlog since this is unlikely something that will be fixed by someone on the Firefox team.

Comment 3

[Taylor Alexander Brown]

I have created a [pull request](https://github.com/mozilla/pdf.js/pull/5926) to fix this issue.