[iOS] A copied text selection is pasted as a web archive attachment in the entry...
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Dec 2018 06:10:28 +0000 (06:10 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Dec 2018 06:10:28 +0000 (06:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=192842
<rdar://problem/46823586>

Reviewed by Tim Horton.

Source/WebCore:

Temporarily revert a behavior change introduced by r238661, where we now add "com.apple.webarchive" as a
registered UTI when dragging or copying a text selection. This broke the Messages app on iOS, which currently
inserts a copied or dragged text selection from WebKit-based views as a web archive file attachment. A fix for
this is internally tracked in <rdar://problem/46830277>.

* platform/ios/PlatformPasteboardIOS.mm:
(WebCore::PlatformPasteboard::write):

Tools:

Adjusts an existing API test to verify that "com.apple.webarchive" is not one of the types registered to the
item provider when dragging a rich text selection.

* TestWebKitAPI/Tests/ios/DragAndDropTestsIOS.mm:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239369 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/ios/PlatformPasteboardIOS.mm
Tools/ChangeLog
Tools/TestWebKitAPI/Tests/ios/DragAndDropTestsIOS.mm

index 8b8bd98..ed5f04e 100644 (file)
@@ -1,3 +1,19 @@
+2018-12-18  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [iOS] A copied text selection is pasted as a web archive attachment in the entry view in Messages
+        https://bugs.webkit.org/show_bug.cgi?id=192842
+        <rdar://problem/46823586>
+
+        Reviewed by Tim Horton.
+
+        Temporarily revert a behavior change introduced by r238661, where we now add "com.apple.webarchive" as a
+        registered UTI when dragging or copying a text selection. This broke the Messages app on iOS, which currently
+        inserts a copied or dragged text selection from WebKit-based views as a web archive file attachment. A fix for
+        this is internally tracked in <rdar://problem/46830277>.
+
+        * platform/ios/PlatformPasteboardIOS.mm:
+        (WebCore::PlatformPasteboard::write):
+
 2018-12-18  Michael Catanzaro  <mcatanzaro@igalia.com>
 
         Unreviewed, add a missing UNUSED_PARAM()
index 8d89bb3..e9c3a31 100644 (file)
@@ -369,10 +369,13 @@ void PlatformPasteboard::write(const PasteboardWebContent& content)
 
     if (content.dataInWebArchiveFormat) {
         auto webArchiveData = content.dataInWebArchiveFormat->createNSData();
-#if !PLATFORM(IOSMAC)
-        [representationsToRegister addData:webArchiveData.get() forType:WebArchivePboardType];
+#if PLATFORM(IOSMAC)
+        NSString *webArchiveType = (__bridge NSString *)kUTTypeWebArchive;
+#else
+        // FIXME: We should additionally register "com.apple.webarchive" once <rdar://problem/46830277> is fixed.
+        NSString *webArchiveType = WebArchivePboardType;
 #endif
-        [representationsToRegister addData:webArchiveData.get() forType:(__bridge NSString *)kUTTypeWebArchive];
+        [representationsToRegister addData:webArchiveData.get() forType:webArchiveType];
     }
 
     if (content.dataInAttributedStringFormat) {
index 201a7fc..1b01b1d 100644 (file)
@@ -1,3 +1,16 @@
+2018-12-18  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [iOS] A copied text selection is pasted as a web archive attachment in the entry view in Messages
+        https://bugs.webkit.org/show_bug.cgi?id=192842
+        <rdar://problem/46823586>
+
+        Reviewed by Tim Horton.
+
+        Adjusts an existing API test to verify that "com.apple.webarchive" is not one of the types registered to the
+        item provider when dragging a rich text selection.
+
+        * TestWebKitAPI/Tests/ios/DragAndDropTestsIOS.mm:
+
 2018-12-18  David Quesada  <david_quesada@apple.com>
 
         REGRESSION: [iOS] API test TestWebKitAPI._WKDownload.OriginatingWebView is a flaky failure
index 8ef395e..44bc150 100644 (file)
@@ -372,6 +372,9 @@ TEST(DragAndDropTests, ContentEditableToContentEditable)
     checkSelectionRectsWithLogging(@[ makeCGRectValue(1, 201, 961, 227) ], [simulator finalSelectionRects]);
     checkRichTextTypePrecedesPlainTextType(simulator.get());
     EXPECT_TRUE([simulator lastKnownDropProposal].precise);
+
+    // FIXME: Once <rdar://problem/46830277> is fixed, we should add "com.apple.webarchive" as a registered pasteboard type and rebaseline this expectation.
+    EXPECT_FALSE([[[simulator sourceItemProviders].firstObject registeredTypeIdentifiers] containsObject:(__bridge NSString *)kUTTypeWebArchive]);
 }
 
 TEST(DragAndDropTests, ContentEditableToTextarea)