Changeset 211852 in webkit

Timestamp:

Feb 7, 2017 5:47:31 PM (7 years ago)

Author:

Wenson Hsieh

Message:

WebItemProviderPasteboard should use -registerLoadHandlersToItemProvider: when creating a new UIItemProvider
​https://bugs.webkit.org/show_bug.cgi?id=167918
<rdar://problem/30382347>

Reviewed by Tim Horton.

Adopts SPI in WebItemProviderPasteboard for object types that the platform knows how to serialize. Since we use
-createObjectOfClass: to initialize data when reading off of the pasteboard, we need to match the format that
objects conforming to UIItemProviderReading will expect. Thus, for all given objects that conform to
UIItemProviderWriting, we have them register themselves to the item provider.

We register other UTI types due to the fact that PlatformPasteboardIOS does not care about the specific
pasteboard used. This should not be necessary, however, since data written to the WebItemProviderPasteboard
should never need to be read by an actual UIPasteboard. This will be refactored in a future patch to add a
special type of WebItemProviderPasteboard-aware PlatformPasteboard.

Also fixes some reference counting issues in WebItemProviderPasteboard by changing the array of _itemProviders
to be a RetainPtr.

• platform/ios/WebItemProviderPasteboard.mm:

(-[WebItemProviderPasteboard init]):
(-[WebItemProviderPasteboard pasteboardTypes]):
(-[WebItemProviderPasteboard itemProviders]):
(-[WebItemProviderPasteboard setItemProviders:]):
(-[WebItemProviderPasteboard numberOfItems]):
(-[WebItemProviderPasteboard setItems:]):
(-[WebItemProviderPasteboard valuesForPasteboardType:inItemSet:]):
(-[WebItemProviderPasteboard itemProviderAtIndex:]):
(-[WebItemProviderPasteboard dealloc]): Deleted.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/ios/WebItemProviderPasteboard.mm (modified) (8 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-02-07  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        WebItemProviderPasteboard should use -registerLoadHandlersToItemProvider: when creating a new UIItemProvider
+        https://bugs.webkit.org/show_bug.cgi?id=167918
+        <rdar://problem/30382347>
+
+        Reviewed by Tim Horton.
+
+        Adopts SPI in WebItemProviderPasteboard for object types that the platform knows how to serialize. Since we use
+        -createObjectOfClass: to initialize data when reading off of the pasteboard, we need to match the format that
+        objects conforming to UIItemProviderReading will expect. Thus, for all given objects that conform to
+        UIItemProviderWriting, we have them register themselves to the item provider.
+
+        We register other UTI types due to the fact that PlatformPasteboardIOS does not care about the specific
+        pasteboard used. This should not be necessary, however, since data written to the WebItemProviderPasteboard
+        should never need to be read by an actual UIPasteboard. This will be refactored in a future patch to add a
+        special type of WebItemProviderPasteboard-aware PlatformPasteboard.
+
+        Also fixes some reference counting issues in WebItemProviderPasteboard by changing the array of _itemProviders
+        to be a RetainPtr.
+
+        * platform/ios/WebItemProviderPasteboard.mm:
+        (-[WebItemProviderPasteboard init]):
+        (-[WebItemProviderPasteboard pasteboardTypes]):
+        (-[WebItemProviderPasteboard itemProviders]):
+        (-[WebItemProviderPasteboard setItemProviders:]):
+        (-[WebItemProviderPasteboard numberOfItems]):
+        (-[WebItemProviderPasteboard setItems:]):
+        (-[WebItemProviderPasteboard valuesForPasteboardType:inItemSet:]):
+        (-[WebItemProviderPasteboard itemProviderAtIndex:]):
+        (-[WebItemProviderPasteboard dealloc]): Deleted.
+
 2017-02-06  Ryosuke Niwa  <rniwa@webkit.org>
 

trunk/Source/WebCore/platform/ios/WebItemProviderPasteboard.mm

@@ -35 +35 @@
 #import <UIKit/UIColor.h>
 #import <UIKit/UIImage.h>
+#import <UIKit/UIItemProviderWriting.h>
+#import <wtf/RetainPtr.h>
 
 SOFT_LINK_FRAMEWORK(UIKit)
@@ -77 +79 @@
 @end
 
-@implementation WebItemProviderPasteboard
+@implementation WebItemProviderPasteboard {
+    RetainPtr<NSArray> _itemProviders;
+}
 
 + (instancetype)sharedInstance
@@ -92 +96 @@
 {
     if (self = [super init]) {
-        _itemProviders = [[NSArray alloc] init];
+        _itemProviders = adoptNS([[NSArray alloc] init]);
         _changeCount = 0;
         _pendingOperationCount = 0;
@@ -99 +103 @@
 }
 
-- (void)dealloc
-{
-    [super dealloc];
-    [_itemProviders release];
-}
-
 - (NSArray<NSString *> *)pasteboardTypes
 {
     NSMutableSet<NSString *> *allTypes = [NSMutableSet set];
-    for (UIItemProvider *provider in _itemProviders)
+    for (UIItemProvider *provider in _itemProviders.get())
         [allTypes addObjectsFromArray:provider.registeredTypeIdentifiers];
     return allTypes.allObjects;
+}
+
+- (NSArray<UIItemProvider *> *)itemProviders
+{
+    return _itemProviders.get();
 }
 
@@ -118 +121 @@
     if (_itemProviders == itemProviders || [_itemProviders isEqualToArray:itemProviders])
         return;
-    _itemProviders = [itemProviders copy];
+
+    _itemProviders = itemProviders;
     _changeCount++;
 }
@@ -124 +128 @@
 - (NSInteger)numberOfItems
 {
-    return _itemProviders.count;
+    return [_itemProviders count];
 }
 
 - (void)setItems:(NSArray *)items
 {
-    NSMutableArray *providers = [[NSMutableArray alloc] init];
+    NSMutableArray *providers = [NSMutableArray array];
     for (NSDictionary *item in items) {
         if (!item.count)
             continue;
-        UIItemProvider *itemProvider = [[getUIItemProviderClass() alloc] init];
-        for (NSString *typeIdentifier in item) {
+        RetainPtr<UIItemProvider> itemProvider = adoptNS([[getUIItemProviderClass() alloc] init]);
+        RetainPtr<NSMutableDictionary> itemRepresentationsCopy = adoptNS([item mutableCopy]);
+        // First, let the platform write all the default object types it can recognize, such as NSString and NSURL.
+        for (NSString *typeIdentifier in [itemRepresentationsCopy allKeys]) {
+            id representingObject = [itemRepresentationsCopy objectForKey:typeIdentifier];
+            if (![representingObject conformsToProtocol:@protocol(UIItemProviderWriting)])
+                continue;
+
+            id <UIItemProviderWriting> objectToWrite = (id <UIItemProviderWriting>)representingObject;
+            if (![objectToWrite.writableTypeIdentifiersForItemProvider containsObject:typeIdentifier])
+                continue;
+
+            [itemRepresentationsCopy removeObjectForKey:typeIdentifier];
+            [objectToWrite registerLoadHandlersToItemProvider:itemProvider.get()];
+        }
+
+        // Secondly, WebKit uses some custom type representations and/or type identifiers, so we need to write these as well.
+        for (NSString *typeIdentifier in itemRepresentationsCopy.get()) {
             [itemProvider registerDataRepresentationForTypeIdentifier:typeIdentifier options:nil loadHandler:^NSProgress *(UIItemProviderDataLoadCompletionBlock completionBlock)
             {
-                completionBlock(item[typeIdentifier], nil);
+                completionBlock([itemRepresentationsCopy objectForKey:typeIdentifier], nil);
                 return [NSProgress discreteProgressWithTotalUnitCount:100];
             }];
         }
-        [providers addObject:itemProvider];
+        [providers addObject:itemProvider.get()];
     }
     _changeCount++;
@@ -175 +195 @@
             return;
 
-        if (isImageType(pasteboardType) && [self _tryToCreateAndAppendObjectOfClass:[NSString class] toArray:values usingProvider:provider])
+        if (isImageType(pasteboardType) && [self _tryToCreateAndAppendObjectOfClass:[getUIImageClass() class] toArray:values usingProvider:provider])
             return;
 
@@ -211 +231 @@
 - (UIItemProvider *)itemProviderAtIndex:(NSInteger)index
 {
-    return 0 <= index && index < (NSInteger)_itemProviders.count ? _itemProviders[index] : nil;
+    return 0 <= index && index < (NSInteger)[_itemProviders count] ? [_itemProviders objectAtIndex:index] : nil;
 }