Changeset 88704 in webkit

Timestamp:

Jun 13, 2011 3:06:18 PM (13 years ago)

Author:

dcheng@chromium.org

Message:

2011-06-13 Daniel Cheng <​dcheng@chromium.org>

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/chromium/ClipboardChromium.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-06-13  Daniel Cheng  <dcheng@chromium.org>
+
+        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  <enne@google.com>
 

trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp

@@ -265 +265 @@
         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
 }