Closed Bug 648511 Opened 9 years ago Closed 9 years ago

Firefox 4.0's PDF plugin scan doesn't find full Acrobat installation

Categories

(Core :: Plug-ins, defect)

x86
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla5
Tracking Status
blocking2.0 --- Macaw+
status2.0 --- .1-fixed

People

(Reporter: khym, Assigned: benjamin)

References

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0

Firefox is supposed to scan for Acrobat's installation directory if plugin.scan.Acrobat is set, but Firefox 4.0 doesn't find the "full" Acrobat's directory; it only finds the free Adobe Reader's install directory.

The problem seems to have been introduced in changeset 62841:04527f9407f9 for Bug 634819 - Use the nsIWindowsRegKey APIs instead of direct Win32 API calls. Prior to the change, it was looking for Acrobat's installation dir by first checking the HKLM\software\Adobe\Acrobat Reader registry key, then if that wasn't found, HKLM\software\Adobe\Adobe Acrobat. However, after the change, it first checks HKLM\software\Adobe\Acrobat Reader, then falls back to HKLM\software\Adobe\Adobe Reader.

Note the incorrect "Adobe Reader" vs. the correct "Adobe Acrobat".

Reproducible: Always

Steps to Reproduce:
1. Install Adobe Acrobat Standard (9.0 in my case). Do not install Adobe Reader
2. Install Firefox 4.0
3. Go to about:plugins
Actual Results:  
PDF plugin isn't listed in about:plugins

Expected Results:  
Expected PDF plugin to be listed in about:plugins
Should be trivial, but here's a patch anyway :)
Attachment #524606 - Flags: review+
Assignee: nobody → benjamin
Blocks: 634819
Status: UNCONFIRMED → NEW
Ever confirmed: true
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
Attachment #524606 - Flags: approval2.0?
blocking2.0: --- → ?
OMG I can't believe that I broke this.  Sorry :(

Christian, is this too late to be considered for Macaw?
blocking2.0: ? → Macaw+
Attachment #524606 - Flags: approval2.0? → approval2.0+
You need to log in before you can comment on or make changes to this bug.