Changeset 104198 in webkit

Timestamp:

Jan 5, 2012, 12:48:17 PM (14 years ago)

Author:

ap@apple.com

Message:

WK2: Safari fails to open a PostScript file in Preview from context menu
​https://bugs.webkit.org/show_bug.cgi?id=75643
<rdar://problem/9823430>

Reviewed by Dan Bernstein.

• UIProcess/API/mac/PDFViewController.mm: (WebKit::PDFViewController::setPDFDocumentData): Append ".pdf" to file name when converting data, as suggested file name is created for original MIME type.

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/API/mac/PDFViewController.mm (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2012-01-05  Alexey Proskuryakov  <ap@apple.com>
+
+        WK2: Safari fails to open a PostScript file in Preview from context menu
+        https://bugs.webkit.org/show_bug.cgi?id=75643
+        <rdar://problem/9823430>
+
+        Reviewed by Dan Bernstein.
+
+        * UIProcess/API/mac/PDFViewController.mm: (WebKit::PDFViewController::setPDFDocumentData):
+        Append ".pdf" to file name when converting data, as suggested file name is created for
+        original MIME type.
+
 2012-01-05  Zeno Albisser  <zeno@webkit.org>
 

trunk/Source/WebKit2/UIProcess/API/mac/PDFViewController.mm

@@ -453 +453 @@
         if (!m_pdfData)
             return;
+        m_suggestedFilename = String(suggestedFilename + ".pdf");
     } else {
         // Make sure to copy the data.
         m_pdfData.adoptCF(CFDataCreate(0, dataReference.data(), dataReference.size()));
-    }
-
-    m_suggestedFilename = suggestedFilename;
+        m_suggestedFilename = suggestedFilename;
+    }
 
     RetainPtr<PDFDocument> pdfDocument(AdoptNS, [[pdfDocumentClass() alloc] initWithData:(NSData *)m_pdfData.get()]);