64934 2011-07-21 05:23:01 -0700 [Mac] Embedding PDF with <object> or <embed> fails when there is no MIME type specified 2011-10-11 22:43:53 -0700 1 1 1 Unclassified WebKit PDF 528+ (Nightly build) Mac All RESOLVED FIXED InRadar P2 Normal --- 1 lvschie ap aestes andersca oldest_to_newest 440356 0 lvschie 2011-07-21 05:23:01 -0700 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 440496 1 ap 2011-07-21 11:05:29 -0700 <rdar://problem/9816330> 443489 2 lvschie 2011-07-28 07:41:01 -0700 I sure hope that either you guys at Webkit, or Apple, fix this issue. It's driving me nuts, to say the least. 481779 3 ap 2011-10-11 11:46:43 -0700 This largely works in nightlies already. One exception is the latter example, where <embed> doesn't have a type. Patch forthcoming. 481785 4 110552 ap 2011-10-11 11:51:43 -0700 Created attachment 110552 proposed fix 481788 5 110552 mitz 2011-10-11 11:56:01 -0700 Comment on attachment 110552 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. 481792 6 ap 2011-10-11 12:00:54 -0700 Committed <http://trac.webkit.org/changeset/97167>. 482229 7 lvschie 2011-10-11 22:43:53 -0700 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? 110552 2011-10-11 11:51:43 -0700 2011-10-11 11:56:01 -0700 proposed fix PDFNoMIMEType.txt text/plain 1586 ap SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDk3MTY2KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTEtMTAtMTEgIEFsZXhleSBQ cm9za3VyeWFrb3YgIDxhcEBhcHBsZS5jb20+CisKKyAgICAgICAgRW1iZWRkaW5nIG9mIFBERidz IHdpdGggPG9iamVjdD4gb3IgPGVtYmVkPiBmYWlscyB3aGVuIHRoZXJlIGlzIG5vIE1JTUUgdHlw ZSBzcGVjaWZpZWQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp P2lkPTY0OTM0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg ICAgKiBXZWJQcm9jZXNzL1dlYlBhZ2UvV2ViUGFnZS5jcHA6IChXZWJLaXQ6OldlYlBhZ2U6OmNy ZWF0ZVBsdWdpbik6IENoZWNrIFVSTCBleHRlbnNpb24sIHRvbywKKyAgICAgICAgbGlrZSBHZXRQ bHVnaW5QYXRoIGRvZXMgaW4gVUkgcHJvY2Vzcy4gT25lIGRpZmZlcmVuY2UgaXMgdGhhdCBVSSBw cm9jZXNzIGFkZGl0aW9uYWxseSB0cmllcworICAgICAgICBwbGF0Zm9ybSBzcGVjaWZpYyBleHRl bnNpb24gdG8gTUlNRSB0eXBlIG1hcHBpbmcsIGJ1dCB0aGF0IGRvZXNuJ3QgbWF0dGVyIGZvciBQ REYsIHdoaWNoCisgICAgICAgIGRvZXMgbm90IGhhdmUgb3RoZXIgc3RhbmRhcmQgZXh0ZW5zaW9u cy4KKwogMjAxMS0xMC0xMSAgQWxleGV5IFByb3NrdXJ5YWtvdiAgPGFwQGFwcGxlLmNvbT4KIAog ICAgICAgICBbTWFjXSBTaG91bGQgYmUgYWJsZSB0byBkaXNwbGF5IG11bHRpLXBhZ2UgUERGcyBp biBzdWJmcmFtZXMKSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQ YWdlLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2Uv V2ViUGFnZS5jcHAJKHJldmlzaW9uIDk3MTU0KQorKysgU291cmNlL1dlYktpdDIvV2ViUHJvY2Vz cy9XZWJQYWdlL1dlYlBhZ2UuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zMzIsNyArMzMyLDggQEAg UGFzc1JlZlB0cjxQbHVnaW4+IFdlYlBhZ2U6OmNyZWF0ZVBsdWdpbgogCiAgICAgaWYgKHBsdWdp blBhdGguaXNOdWxsKCkpIHsKICNpZiBQTEFURk9STShNQUMpCi0gICAgICAgIGlmIChwYXJhbWV0 ZXJzLm1pbWVUeXBlID09ICJhcHBsaWNhdGlvbi9wZGYiKQorICAgICAgICBpZiAocGFyYW1ldGVy cy5taW1lVHlwZSA9PSAiYXBwbGljYXRpb24vcGRmIgorICAgICAgICAgICAgfHwgKHBhcmFtZXRl cnMubWltZVR5cGUuaXNFbXB0eSgpICYmIHBhcmFtZXRlcnMudXJsLnN0cmluZygpLmxvd2VyKCku ZW5kc1dpdGgoIi5wZGYiKSkpCiAgICAgICAgICAgICByZXR1cm4gQnVpbHRJblBERlZpZXc6OmNy ZWF0ZShtX3BhZ2UuZ2V0KCkpOwogI2VuZGlmCiAgICAgICAgIHJldHVybiAwOwo=