WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
64934
[Mac] Embedding PDF with <object> or <embed> fails when there is no MIME type specified
https://bugs.webkit.org/show_bug.cgi?id=64934
Summary
[Mac] Embedding PDF with <object> or <embed> fails when there is no MIME type...
lvschie
Reported
2011-07-21 05:23:01 PDT
In Safari 5.1 and the most recent versions of WebKit the embedding of PDF's in the HTML tags OBJECT or EMBED fail (at least under Lion) and display the message "Missing plugin". In earlier versions of Safari (5.0x and lower) this worked like a charm. This is a major issue for us, because our software relies heavily on this and there are not alternatives available for Lion yet (Acrobat PDF plugin not supported onder OS X 10.7 Lion). It seems that the embedded PDF viewer only displays PDF's that are loaded directly into a page correctly, but not those embedded in the OBJECT and/or EMBED tags. Example of code: < object data="file.pdf" width="80%" height="80%" type="application/pdf" >Object embed not supported by this browser < /object > or: < embed src="file.pdf" width="80%" height="80%" >< /embed > Example of website that shows the issue:
http://blogs.adobe.com/pdfdevjunkie/2007/08/using_the_html_embed_tag_to_di.html
Attachments
proposed fix
(1.55 KB, patch)
2011-10-11 11:51 PDT
,
Alexey Proskuryakov
mitz: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1
2011-07-21 11:05:29 PDT
<
rdar://problem/9816330
>
lvschie
Comment 2
2011-07-28 07:41:01 PDT
I sure hope that either you guys at Webkit, or Apple, fix this issue. It's driving me nuts, to say the least.
Alexey Proskuryakov
Comment 3
2011-10-11 11:46:43 PDT
This largely works in nightlies already. One exception is the latter example, where <embed> doesn't have a type. Patch forthcoming.
Alexey Proskuryakov
Comment 4
2011-10-11 11:51:43 PDT
Created
attachment 110552
[details]
proposed fix
mitz
Comment 5
2011-10-11 11:56:01 PDT
Comment on
attachment 110552
[details]
proposed fix View in context:
https://bugs.webkit.org/attachment.cgi?id=110552&action=review
> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:336 > + || (parameters.mimeType.isEmpty() && parameters.url.string().lower().endsWith(".pdf")))
This should check the path component of the url rather than the whole string.
Alexey Proskuryakov
Comment 6
2011-10-11 12:00:54 PDT
Committed <
http://trac.webkit.org/changeset/97167
>.
lvschie
Comment 7
2011-10-11 22:43:53 PDT
Great, it works partially. Only thing missing now are the zoom, download and page forward/back buttons when a PDF is loaded. I assume that these will be visible when Mac OS X 10.7.2 is launched?
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug