Title: [88704] trunk/Source/WebCore
- Revision
- 88704
- Author
- [email protected]
- Date
- 2011-06-13 15:06:18 -0700 (Mon, 13 Jun 2011)
Log Message
2011-06-13 Daniel Cheng <[email protected]>
Reviewed by Eric Seidel.
[chromium] Dragging images to Mail.app results in broken attachments
https://bugs.webkit.org/show_bug.cgi?id=62466
No new tests.
* platform/chromium/ClipboardChromium.cpp:
(WebCore::ClipboardChromium::declareAndWriteDragImage):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (88703 => 88704)
--- trunk/Source/WebCore/ChangeLog 2011-06-13 22:00:13 UTC (rev 88703)
+++ trunk/Source/WebCore/ChangeLog 2011-06-13 22:06:18 UTC (rev 88704)
@@ -1,3 +1,15 @@
+2011-06-13 Daniel Cheng <[email protected]>
+
+ Reviewed by Eric Seidel.
+
+ [chromium] Dragging images to Mail.app results in broken attachments
+ https://bugs.webkit.org/show_bug.cgi?id=62466
+
+ No new tests.
+
+ * platform/chromium/ClipboardChromium.cpp:
+ (WebCore::ClipboardChromium::declareAndWriteDragImage):
+
2011-06-13 Adrienne Walker <[email protected]>
Reviewed by Simon Fraser.
Modified: trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp (88703 => 88704)
--- trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp 2011-06-13 22:00:13 UTC (rev 88703)
+++ trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp 2011-06-13 22:06:18 UTC (rev 88704)
@@ -264,8 +264,11 @@
if (fullURL.isEmpty())
return;
- // Put img tag on the clipboard referencing the image
+#if !defined(OS_MACOSX)
+ // Put img tag on the clipboard referencing the image. We don't do this on Mac to match Safari
+ // behavior and to ensure that an image dragged to Mail.app is properly attached.
m_dataObject->setData(mimeTypeTextHTML, imageToMarkup(fullURL, element));
+#endif
}
void ClipboardChromium::writeURL(const KURL& url, const String& title, Frame*)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
