The WebPDFViewPlaceholder changes in r173265 completely broke loading by ... removing the code that completes the load. Let's put it back. rdar://problem/18524717
Created attachment 239192 [details] patch
Comment on attachment 239192 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=239192&action=review > Source/WebKit/ios/WebView/WebPDFViewPlaceholder.mm:287 > + // Dump the data provider as soon as possible since the CGPDFDocument will not hold onto it. I'm not sure this comment makes sense to me (I'm just restoring it, but maybe we should refrain).
Comment on attachment 239192 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=239192&action=review > Source/WebKit/ios/WebView/WebPDFViewPlaceholder.mm:285 > + _document = CGPDFDocumentCreateWithProvider(provider); No idea why this is manually memory-managed like this, but we should fix that in a different patch.
Comment on attachment 239192 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=239192&action=review r=me >> Source/WebKit/ios/WebView/WebPDFViewPlaceholder.mm:287 >> + // Dump the data provider as soon as possible since the CGPDFDocument will not hold onto it. > > I'm not sure this comment makes sense to me (I'm just restoring it, but maybe we should refrain). Yes, let's remove the comment. It doesn't appear to add anything useful.
http://trac.webkit.org/changeset/174282