Bug 141457

Summary: REGRESSION (iOS 8): PDFs embedded in <object>/<embed> tag don't load at all
Product: WebKit Reporter: Tim Horton <thorton>
Component: New BugsAssignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: mitz, sam, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch ap: review+

Tim Horton
Reported 2015-02-10 19:06:18 PST
[iOS] WebKit2 should render <object> and <embed> PDFs with PDFDocumentImage
Attachments
Patch (1.68 KB, patch)
2015-02-10 19:06 PST, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (5.03 KB, patch)
2015-02-11 13:26 PST, Tim Horton 		ap: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tim Horton
Comment 1 2015-02-10 19:06:33 PST
Created attachment 246368 [details] Patch
Tim Horton
Comment 2 2015-02-10 19:06:58 PST
<rdar://problem/18221851>
mitz
Comment 3 2015-02-10 19:14:46 PST
A better title for this bug might be “REGRESSION (iOS 8): PDFs embedded in <object>/<embed> tag don't load at all”
mitz
Comment 4 2015-02-10 19:20:05 PST
This can have a reftest with <img> on the -expected side, right?
Tim Horton
Comment 5 2015-02-10 19:26:17 PST
Seems reasonable.
Tim Horton
Comment 6 2015-02-11 13:26:00 PST
Created attachment 246409 [details] Patch
Alexey Proskuryakov
Comment 7 2015-02-11 13:28:30 PST
Comment on attachment 246409 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=246409&action=review > LayoutTests/TestExpectations:482 > +# The current expectations for this test only make sense for WebKit on iOS; other platforms can have very different behaviors. > +fast/replaced/pdf-as-object-and-embed.html [ Skip ] The test makes sense on all platforms though, so seems like it should be enabled, with different expectations elsewhere.
Tim Horton
Comment 8 2015-02-11 13:33:57 PST
(In reply to comment #7) > Comment on attachment 246409 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=246409&action=review > > > LayoutTests/TestExpectations:482 > > +# The current expectations for this test only make sense for WebKit on iOS; other platforms can have very different behaviors. > > +fast/replaced/pdf-as-object-and-embed.html [ Skip ] > > The test makes sense on all platforms though, so seems like it should be > enabled, with different expectations elsewhere. I filed https://bugs.webkit.org/show_bug.cgi?id=141488.
Tim Horton
Comment 9 2015-02-11 13:36:59 PST
http://trac.webkit.org/changeset/179955
Note You need to log in before you can comment on or make changes to this bug.