553934 - [OOPP] PDF opened in browser showing huge zoom value

Status: RESOLVED WORKSFORME

(Core Graveyard :: Plug-ins, defect)

Description

[Jim Jeffery not reading bug-mail 1/2/11]

Using Adobe Reader 9.3.1 set to open PDF's in the browser open:

http://www.av-comparatives.org/images/stories/test/ondret/avc_report25.pdf

Note that zoom level shows over 5000% zoom, and clicking on the document gives a modal diaglog box about unable to read document and an Error (14) 

Seems not all pdf's exibit this problem, as this one is OK:
http://www.in.gov/idoc/files/IDOCRecidivism.pdf

Turning off OOPP on the first URL will allow it to load normally.

Comment 1

[Jim Jeffery not reading bug-mail 1/2/11]

Not sure when this regressed as this is the first I've seen this problem.

Testing using the nightly for today:
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre) Gecko/20100321 Minefield/3.7a4pre Firefox/3.6 ID:20100321073057

Comment 2

[Jim Jeffery not reading bug-mail 1/2/11]

This regressed between: 

3/8/10 nightly
http://hg.mozilla.org/mozilla-central/rev/afc7c1521284  ok

3/9/10 nightly
http://hg.mozilla.org/mozilla-central/rev/dcfcfca09b45  bad

The zip-archive is really dog-slow for some reason, will try to narrow it more if I can get some downloads between the 8th and 9th.

Comment 3

[Jim Jeffery not reading bug-mail 1/2/11]

better range

good with cset:
463ffd8cf98e

bad with cset:
02dbaba64e8b

Now, if only I knew how to find patches..

Comment 4

[Jim Jeffery not reading bug-mail 1/2/11]

I'm confused;

cset 463ffd8cf98e  is OK added patch for bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=488850

cset 02dbaba64e8b is bad and was a config change:
Doug Turner — Ensuring that js/src/config/config.mk matches top level config.mk

Comment 5

[Jim Jeffery not reading bug-mail 1/2/11]

OK, my bad... got mixed up, too many builds tested:

cset 463ffd8cf98e  is the bad set which added patch for bug:

https://bugzilla.mozilla.org/show_bug.cgi?id=488850

488850 seems to be security sensitive as I can't add it to Blocks..

Comment 6

[Jim Jeffery not reading bug-mail 1/2/11]

Now, I'm not sure.. there are not enough zip archive files to be really sure..

The nightly on 3/8 with cset
http://hg.mozilla.org/mozilla-central/rev/afc7c1521284 is OK

1st available zip archive that has not already been purged is the one noted in comment #5 above - there are still several patches between the nightly and the cset noted in comment #5.  

sorry for all the bug-spam ..  hope this helps someone figure which patch(s) is responsible.

Comment 7

[Jim Jeffery not reading bug-mail 1/2/11]

Unknown when this was fixed or by what.. but its now WFM using today's nightly trunk m-c build.