Title: [182756] trunk/Source
Revision
182756
Author
beid...@apple.com
Date
2015-04-13 14:23:09 -0700 (Mon, 13 Apr 2015)

Log Message

Share sheets from Share menus appear outside the browser window.
<rdar://problem/20455592> and https://bugs.webkit.org/show_bug.cgi?id=143620

Reviewed by Darin Adler.

Source/WebCore:

* page/ContextMenuController.h:
(WebCore::ContextMenuController::client):

Source/WebKit/mac:

Refactor existing Sharing Service Picker code to also be used for the Share menu NSMenuItem.

* Misc/WebSharingServicePickerController.h:
(WebSharingServicePickerClient::webView):
* Misc/WebSharingServicePickerController.mm:
(WebSharingServicePickerClient::WebSharingServicePickerClient):
(WebSharingServicePickerClient::sharingServicePickerWillBeDestroyed):
(WebSharingServicePickerClient::pageForSharingServicePicker):
(WebSharingServicePickerClient::windowForSharingServicePicker):
(WebSharingServicePickerClient::screenRectForCurrentSharingServicePickerItem):
(WebSharingServicePickerClient::imageForCurrentSharingServicePickerItem):
(-[WebSharingServicePickerController initWithItems:includeEditorServices:client:style:]):
(-[WebSharingServicePickerController initWithSharingServicePicker:client:]):
(-[WebSharingServicePickerController sharingService:didShareItems:]):

* WebCoreSupport/WebContextMenuClient.h:
(WebContextMenuClient::webView): Deleted.
* WebCoreSupport/WebContextMenuClient.mm:
(WebContextMenuClient::WebContextMenuClient):
(WebContextMenuClient::pageForSharingServicePicker): Deleted.
(WebContextMenuClient::windowForSharingServicePicker): Deleted.

* WebCoreSupport/WebSelectionServiceController.h:
* WebCoreSupport/WebSelectionServiceController.mm:
(WebSelectionServiceController::WebSelectionServiceController):
(WebSelectionServiceController::handleSelectionServiceClick):
(WebSelectionServiceController::pageForSharingServicePicker): Deleted.
(WebSelectionServiceController::windowForSharingServicePicker): Deleted.
(WebSelectionServiceController::screenRectForCurrentSharingServicePickerItem): Deleted.
(WebSelectionServiceController::imageForCurrentSharingServicePickerItem): Deleted.

* WebView/WebHTMLView.mm:
(-[WebHTMLView menuForEvent:]):
* WebView/WebViewData.h:

Source/WebKit2:

Refactor existing Sharing Service Picker code to also be used for the Share menu NSMenuItem.

* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::internalShowContextMenu):
(WebKit::WebPageProxy::platformInitializeShareMenuItem):
* UIProcess/WebPageProxy.h:

* UIProcess/mac/WKSharingServicePickerDelegate.h: Added.
* UIProcess/mac/WKSharingServicePickerDelegate.mm: Added.
(+[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate]):
(-[WKSharingServicePickerDelegate menuProxy]):
(-[WKSharingServicePickerDelegate setMenuProxy:]):
(-[WKSharingServicePickerDelegate setPicker:]):
(-[WKSharingServicePickerDelegate setFiltersEditingServices:]):
(-[WKSharingServicePickerDelegate setHandlesEditingReplacement:]):
(-[WKSharingServicePickerDelegate sharingServicePicker:sharingServicesForItems:mask:proposedSharingServices:]):
(-[WKSharingServicePickerDelegate sharingServicePicker:delegateForSharingService:]):
(-[WKSharingServicePickerDelegate sharingService:willShareItems:]):
(-[WKSharingServicePickerDelegate sharingService:didShareItems:]):
(-[WKSharingServicePickerDelegate sharingService:sourceWindowForShareItems:sharingContentScope:]):

* UIProcess/mac/WebContextMenuProxyMac.mm:
(WebKit::WebContextMenuProxyMac::setupServicesMenu):
(+[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate]): Deleted.
(-[WKSharingServicePickerDelegate menuProxy]): Deleted.
(-[WKSharingServicePickerDelegate setMenuProxy:]): Deleted.
(-[WKSharingServicePickerDelegate setPicker:]): Deleted.
(-[WKSharingServicePickerDelegate setIncludeEditorServices:]): Deleted.
(-[WKSharingServicePickerDelegate sharingServicePicker:sharingServicesForItems:mask:proposedSharingServices:]): Deleted.
(-[WKSharingServicePickerDelegate sharingServicePicker:delegateForSharingService:]): Deleted.
(-[WKSharingServicePickerDelegate sharingService:willShareItems:]): Deleted.
(-[WKSharingServicePickerDelegate sharingService:didShareItems:]): Deleted.
(-[WKSharingServicePickerDelegate sharingService:sourceWindowForShareItems:sharingContentScope:]): Deleted.

* UIProcess/mac/WebPageProxyMac.mm:
(WebKit::WebPageProxy::platformInitializeShareMenuItem):

* WebKit2.xcodeproj/project.pbxproj:

Modified Paths

Added Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (182755 => 182756)


--- trunk/Source/WebCore/ChangeLog	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebCore/ChangeLog	2015-04-13 21:23:09 UTC (rev 182756)
@@ -1,3 +1,13 @@
+2015-04-13  Brady Eidson  <beid...@apple.com>
+
+        Share sheets from Share menus appear outside the browser window.
+        <rdar://problem/20455592> and https://bugs.webkit.org/show_bug.cgi?id=143620
+
+        Reviewed by Darin Adler.
+
+        * page/ContextMenuController.h:
+        (WebCore::ContextMenuController::client):
+
 2015-04-13  Andreas Kling  <akl...@apple.com>
 
         Rebaseline bindings tests.

Modified: trunk/Source/WebCore/page/ContextMenuController.h (182755 => 182756)


--- trunk/Source/WebCore/page/ContextMenuController.h	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebCore/page/ContextMenuController.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -49,6 +49,7 @@
     ~ContextMenuController();
 
     Page& page() { return m_page; }
+    ContextMenuClient& client() { return m_client; }
 
     ContextMenu* contextMenu() const { return m_contextMenu.get(); }
     WEBCORE_EXPORT void clearContextMenu();

Modified: trunk/Source/WebKit/mac/ChangeLog (182755 => 182756)


--- trunk/Source/WebKit/mac/ChangeLog	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/ChangeLog	2015-04-13 21:23:09 UTC (rev 182756)
@@ -1,3 +1,45 @@
+2015-04-13  Brady Eidson  <beid...@apple.com>
+
+        Share sheets from Share menus appear outside the browser window.
+        <rdar://problem/20455592> and https://bugs.webkit.org/show_bug.cgi?id=143620
+
+        Reviewed by Darin Adler.
+
+        Refactor existing Sharing Service Picker code to also be used for the Share menu NSMenuItem.
+
+        * Misc/WebSharingServicePickerController.h:
+        (WebSharingServicePickerClient::webView):
+        * Misc/WebSharingServicePickerController.mm:
+        (WebSharingServicePickerClient::WebSharingServicePickerClient):
+        (WebSharingServicePickerClient::sharingServicePickerWillBeDestroyed):
+        (WebSharingServicePickerClient::pageForSharingServicePicker):
+        (WebSharingServicePickerClient::windowForSharingServicePicker):
+        (WebSharingServicePickerClient::screenRectForCurrentSharingServicePickerItem):
+        (WebSharingServicePickerClient::imageForCurrentSharingServicePickerItem):
+        (-[WebSharingServicePickerController initWithItems:includeEditorServices:client:style:]):
+        (-[WebSharingServicePickerController initWithSharingServicePicker:client:]):
+        (-[WebSharingServicePickerController sharingService:didShareItems:]):
+
+        * WebCoreSupport/WebContextMenuClient.h:
+        (WebContextMenuClient::webView): Deleted.
+        * WebCoreSupport/WebContextMenuClient.mm:
+        (WebContextMenuClient::WebContextMenuClient):
+        (WebContextMenuClient::pageForSharingServicePicker): Deleted.
+        (WebContextMenuClient::windowForSharingServicePicker): Deleted.
+
+        * WebCoreSupport/WebSelectionServiceController.h:
+        * WebCoreSupport/WebSelectionServiceController.mm:
+        (WebSelectionServiceController::WebSelectionServiceController):
+        (WebSelectionServiceController::handleSelectionServiceClick):
+        (WebSelectionServiceController::pageForSharingServicePicker): Deleted.
+        (WebSelectionServiceController::windowForSharingServicePicker): Deleted.
+        (WebSelectionServiceController::screenRectForCurrentSharingServicePickerItem): Deleted.
+        (WebSelectionServiceController::imageForCurrentSharingServicePickerItem): Deleted.
+
+        * WebView/WebHTMLView.mm:
+        (-[WebHTMLView menuForEvent:]):
+        * WebView/WebViewData.h:
+
 2015-04-13  Beth Dakin  <bda...@apple.com>
 
         Add force property to MouseEvents

Modified: trunk/Source/WebKit/mac/Misc/WebSharingServicePickerController.h (182755 => 182756)


--- trunk/Source/WebKit/mac/Misc/WebSharingServicePickerController.h	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/Misc/WebSharingServicePickerController.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -26,10 +26,14 @@
 #if ENABLE(SERVICE_CONTROLS)
 
 #import <wtf/RetainPtr.h>
+
+#if PLATFORM(MAC)
 #import <WebCore/NSSharingServicePickerSPI.h>
 #import <WebCore/NSSharingServiceSPI.h>
+#endif
 
 @class WebSharingServicePickerController;
+@class WebView;
 
 namespace WebCore {
 class FloatRect;
@@ -42,21 +46,29 @@
 public:
     virtual ~WebSharingServicePickerClient() { }
 
-    virtual void sharingServicePickerWillBeDestroyed(WebSharingServicePickerController &) = 0;
-    virtual WebCore::Page* pageForSharingServicePicker(WebSharingServicePickerController &) = 0;
-    virtual RetainPtr<NSWindow> windowForSharingServicePicker(WebSharingServicePickerController &) = 0;
+    virtual void sharingServicePickerWillBeDestroyed(WebSharingServicePickerController &);
+    virtual WebCore::Page* pageForSharingServicePicker(WebSharingServicePickerController &);
+    virtual RetainPtr<NSWindow> windowForSharingServicePicker(WebSharingServicePickerController &);
 
-    virtual WebCore::FloatRect screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &) = 0;
-    virtual RetainPtr<NSImage> imageForCurrentSharingServicePickerItem(WebSharingServicePickerController &) = 0;
+    virtual WebCore::FloatRect screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &);
+    virtual RetainPtr<NSImage> imageForCurrentSharingServicePickerItem(WebSharingServicePickerController &);
+
+    WebView *webView() { return m_webView; }
+
+protected:
+    explicit WebSharingServicePickerClient(WebView *);
+    WebView *m_webView;
 };
 
 @interface WebSharingServicePickerController : NSObject <NSSharingServiceDelegate, NSSharingServicePickerDelegate> {
     WebSharingServicePickerClient* _pickerClient;
     RetainPtr<NSSharingServicePicker> _picker;
     BOOL _includeEditorServices;
+    BOOL _handleEditingReplacement;
 }
 
 - (instancetype)initWithItems:(NSArray *)items includeEditorServices:(BOOL)includeEditorServices client:(WebSharingServicePickerClient*)pickerClient style:(NSSharingServicePickerStyle)style;
+- (instancetype)initWithSharingServicePicker:(NSSharingServicePicker *)sharingServicePicker client:(WebSharingServicePickerClient&)pickerClient;
 - (NSMenu *)menu;
 - (void)didShareImageData:(NSData *)data confirmDataIsValidTIFFData:(BOOL)confirmData;
 - (void)clear;

Modified: trunk/Source/WebKit/mac/Misc/WebSharingServicePickerController.mm (182755 => 182756)


--- trunk/Source/WebKit/mac/Misc/WebSharingServicePickerController.mm	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/Misc/WebSharingServicePickerController.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -42,8 +42,38 @@
 
 using namespace WebCore;
 
+WebSharingServicePickerClient::WebSharingServicePickerClient(WebView *webView)
+    : m_webView(webView)
+{
+}
+
+void WebSharingServicePickerClient::sharingServicePickerWillBeDestroyed(WebSharingServicePickerController &)
+{
+}
+
+Page* WebSharingServicePickerClient::pageForSharingServicePicker(WebSharingServicePickerController &)
+{
+    return [m_webView page];
+}
+
+RetainPtr<NSWindow> WebSharingServicePickerClient::windowForSharingServicePicker(WebSharingServicePickerController &)
+{
+    return [m_webView window];
+}
+
+FloatRect WebSharingServicePickerClient::screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &)
+{
+    return FloatRect();
+}
+
+RetainPtr<NSImage> WebSharingServicePickerClient::imageForCurrentSharingServicePickerItem(WebSharingServicePickerController &)
+{
+    return nil;
+}
+
 @implementation WebSharingServicePickerController
 
+#if ENABLE(SERVICE_CONTROLS)
 - (instancetype)initWithItems:(NSArray *)items includeEditorServices:(BOOL)includeEditorServices client:(WebSharingServicePickerClient*)pickerClient style:(NSSharingServicePickerStyle)style
 {
 #ifndef __LP64__
@@ -57,12 +87,33 @@
     [_picker setDelegate:self];
 
     _includeEditorServices = includeEditorServices;
+    _handleEditingReplacement = includeEditorServices;
     _pickerClient = pickerClient;
 
     return self;
 #endif
 }
 
+- (instancetype)initWithSharingServicePicker:(NSSharingServicePicker *)sharingServicePicker client:(WebSharingServicePickerClient&)pickerClient
+{
+#ifndef __LP64__
+    return nil;
+#else
+    if (!(self = [super init]))
+        return nil;
+
+    _picker = sharingServicePicker;
+    [_picker setDelegate:self];
+
+    _includeEditorServices = YES;
+    _pickerClient = &pickerClient;
+
+    return self;
+#endif
+}
+#endif // ENABLE(SERVICE_CONTROLS)
+
+
 - (void)clear
 {
     // Protect self from being dealloc'ed partway through this method.
@@ -140,9 +191,7 @@
 
 - (void)sharingService:(NSSharingService *)sharingService didShareItems:(NSArray *)items
 {
-    // We only care about what item was shared if we were interested in editor services
-    // (i.e., if we plan on replacing the selection with the returned item)
-    if (!_includeEditorServices)
+    if (!_handleEditingReplacement)
         return;
 
     // We only send one item, so we should only get one item back.
@@ -211,4 +260,4 @@
 
 @end
 
-#endif
+#endif // ENABLE(SERVICE_CONTROLS)

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.h (182755 => 182756)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.h	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -26,6 +26,8 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#if ENABLE(CONTEXT_MENUS)
+
 #import "WebSharingServicePickerController.h"
 #import <WebCore/ContextMenuClient.h>
 #import <WebCore/IntRect.h>
@@ -39,7 +41,7 @@
 
 class WebContextMenuClient : public WebCore::ContextMenuClient
 #if ENABLE(SERVICE_CONTROLS)
-, public WebSharingServicePickerClient
+    , public WebSharingServicePickerClient
 #endif
 {
 public:
@@ -64,21 +66,21 @@
 #if ENABLE(SERVICE_CONTROLS)
     // WebSharingServicePickerClient
     virtual void sharingServicePickerWillBeDestroyed(WebSharingServicePickerController &) override;
-    virtual WebCore::Page* pageForSharingServicePicker(WebSharingServicePickerController &) override;
-    virtual RetainPtr<NSWindow> windowForSharingServicePicker(WebSharingServicePickerController &) override;
     virtual WebCore::FloatRect screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &) override;
     virtual RetainPtr<NSImage> imageForCurrentSharingServicePickerItem(WebSharingServicePickerController &) override;
 #endif
 
-    WebView *webView() { return m_webView; }
-        
 private:
     NSMenu *contextMenuForEvent(NSEvent *, NSView *, bool& isServicesMenu);
 
     bool clientFloatRectForNode(WebCore::Node&, WebCore::FloatRect&) const;
 
-    WebView *m_webView;
 #if ENABLE(SERVICE_CONTROLS)
     RetainPtr<WebSharingServicePickerController> m_sharingServicePickerController;
+#else
+    WebView* m_webView;
 #endif
 };
+
+#endif // ENABLE(CONTEXT_MENUS)
+

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.mm (182755 => 182756)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.mm	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -67,8 +67,12 @@
 - (void)speakString:(NSString *)string;
 @end
 
-WebContextMenuClient::WebContextMenuClient(WebView *webView) 
+WebContextMenuClient::WebContextMenuClient(WebView *webView)
+#if ENABLE(SERVICE_CONTROLS)
+    : WebSharingServicePickerClient(webView)
+#else
     : m_webView(webView)
+#endif
 {
 }
 
@@ -412,16 +416,6 @@
     m_sharingServicePickerController = nil;
 }
 
-WebCore::Page* WebContextMenuClient::pageForSharingServicePicker(WebSharingServicePickerController &)
-{
-    return [m_webView page];
-}
-
-RetainPtr<NSWindow> WebContextMenuClient::windowForSharingServicePicker(WebSharingServicePickerController &)
-{
-    return [m_webView window];
-}
-
 WebCore::FloatRect WebContextMenuClient::screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &)
 {
     Page* page = [m_webView page];

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebSelectionServiceController.h (182755 => 182756)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebSelectionServiceController.h	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebSelectionServiceController.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -51,14 +51,8 @@
 
     // WebSharingServicePickerClient
     virtual void sharingServicePickerWillBeDestroyed(WebSharingServicePickerController &) override;
-    virtual WebCore::Page* pageForSharingServicePicker(WebSharingServicePickerController &) override;
-    virtual RetainPtr<NSWindow> windowForSharingServicePicker(WebSharingServicePickerController &) override;
-    virtual WebCore::FloatRect screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &) override;
-    virtual RetainPtr<NSImage> imageForCurrentSharingServicePickerItem(WebSharingServicePickerController &) override;
 
 private:
-    WebView *m_webView;
-
     RetainPtr<WebSharingServicePickerController> m_sharingServicePickerController;
 };
 

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebSelectionServiceController.mm (182755 => 182756)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebSelectionServiceController.mm	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebSelectionServiceController.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -37,7 +37,7 @@
 using namespace WebCore;
 
 WebSelectionServiceController::WebSelectionServiceController(WebView *webView) 
-    : m_webView(webView)
+    : WebSharingServicePickerClient(webView)
 {
 }
 
@@ -58,6 +58,7 @@
     NSArray *items = @[ attributedSelection.get() ];
 
     bool isEditable = selection.selection().isContentEditable();
+    
     m_sharingServicePickerController = adoptNS([[WebSharingServicePickerController alloc] initWithItems:items includeEditorServices:isEditable client:this style:NSSharingServicePickerStyleTextSelection]);
 
     RetainPtr<NSMenu> menu = adoptNS([[m_sharingServicePickerController menu] copy]);
@@ -96,24 +97,4 @@
     m_sharingServicePickerController = nil;
 }
 
-WebCore::Page* WebSelectionServiceController::pageForSharingServicePicker(WebSharingServicePickerController &)
-{
-    return [m_webView page];
-}
-
-RetainPtr<NSWindow> WebSelectionServiceController::windowForSharingServicePicker(WebSharingServicePickerController &)
-{
-    return [m_webView window];
-}
-
-FloatRect WebSelectionServiceController::screenRectForCurrentSharingServicePickerItem(WebSharingServicePickerController &)
-{
-    return FloatRect();
-}
-
-RetainPtr<NSImage> WebSelectionServiceController::imageForCurrentSharingServicePickerItem(WebSharingServicePickerController &)
-{
-    return nil;
-}
-
 #endif

Modified: trunk/Source/WebKit/mac/WebView/WebHTMLView.mm (182755 => 182756)


--- trunk/Source/WebKit/mac/WebView/WebHTMLView.mm	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/WebView/WebHTMLView.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -38,6 +38,7 @@
 #import "WebActionMenuController.h"
 #import "WebArchive.h"
 #import "WebClipView.h"
+#import "WebContextMenuClient.h"
 #import "WebDOMOperationsInternal.h"
 #import "WebDataSourceInternal.h"
 #import "WebDefaultUIDelegate.h"
@@ -67,6 +68,7 @@
 #import "WebPreferences.h"
 #import "WebPreferencesPrivate.h"
 #import "WebResourcePrivate.h"
+#import "WebSharingServicePickerController.h"
 #import "WebTextCompletionController.h"
 #import "WebTypesInternal.h"
 #import "WebUIDelegatePrivate.h"
@@ -624,6 +626,10 @@
 #ifndef NDEBUG
     BOOL enumeratingSubviews;
 #endif
+
+#if ENABLE(SERVICE_CONTROLS)
+    RetainPtr<WebSharingServicePickerController> currentSharingServicePickerController;
+#endif
 }
 - (void)clear;
 @end
@@ -3358,6 +3364,21 @@
     if (!coreMenu)
         return nil;
 
+    auto menuItemVector = contextMenuItemVector(coreMenu->platformDescription());
+    for (auto& menuItem : menuItemVector) {
+        if (menuItem.action() != ContextMenuItemTagShareMenu)
+            continue;
+
+        NSMenuItem *nsItem = menuItem.platformDescription();
+        if (![nsItem.representedObject isKindOfClass:[NSSharingServicePicker class]]) {
+            ASSERT_NOT_REACHED();
+            continue;
+        }
+#if ENABLE(SERVICE_CONTROLS)
+        _private->currentSharingServicePickerController = adoptNS([[WebSharingServicePickerController alloc] initWithSharingServicePicker:nsItem.representedObject client:static_cast<WebContextMenuClient&>(page->contextMenuController().client())]);
+#endif
+    }
+
     NSArray* menuItems = coreMenu->platformDescription();
     if (!menuItems)
         return nil;

Modified: trunk/Source/WebKit/mac/WebView/WebViewData.h (182755 => 182756)


--- trunk/Source/WebKit/mac/WebView/WebViewData.h	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit/mac/WebView/WebViewData.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -93,7 +93,10 @@
 
 class LayerFlushController;
 class WebViewGroup;
+
+#if ENABLE(SERVICE_CONTROLS)
 class WebSelectionServiceController;
+#endif
 
 class WebViewLayerFlushScheduler : public WebCore::LayerFlushScheduler {
 public:

Modified: trunk/Source/WebKit2/ChangeLog (182755 => 182756)


--- trunk/Source/WebKit2/ChangeLog	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit2/ChangeLog	2015-04-13 21:23:09 UTC (rev 182756)
@@ -1,3 +1,49 @@
+2015-04-13  Brady Eidson  <beid...@apple.com>
+
+        Share sheets from Share menus appear outside the browser window.
+        <rdar://problem/20455592> and https://bugs.webkit.org/show_bug.cgi?id=143620
+
+        Reviewed by Darin Adler.
+
+        Refactor existing Sharing Service Picker code to also be used for the Share menu NSMenuItem.
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::internalShowContextMenu):
+        (WebKit::WebPageProxy::platformInitializeShareMenuItem):
+        * UIProcess/WebPageProxy.h:
+
+        * UIProcess/mac/WKSharingServicePickerDelegate.h: Added.
+        * UIProcess/mac/WKSharingServicePickerDelegate.mm: Added.
+        (+[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate]):
+        (-[WKSharingServicePickerDelegate menuProxy]):
+        (-[WKSharingServicePickerDelegate setMenuProxy:]):
+        (-[WKSharingServicePickerDelegate setPicker:]):
+        (-[WKSharingServicePickerDelegate setFiltersEditingServices:]):
+        (-[WKSharingServicePickerDelegate setHandlesEditingReplacement:]):
+        (-[WKSharingServicePickerDelegate sharingServicePicker:sharingServicesForItems:mask:proposedSharingServices:]):
+        (-[WKSharingServicePickerDelegate sharingServicePicker:delegateForSharingService:]):
+        (-[WKSharingServicePickerDelegate sharingService:willShareItems:]):
+        (-[WKSharingServicePickerDelegate sharingService:didShareItems:]):
+        (-[WKSharingServicePickerDelegate sharingService:sourceWindowForShareItems:sharingContentScope:]):
+
+        * UIProcess/mac/WebContextMenuProxyMac.mm:
+        (WebKit::WebContextMenuProxyMac::setupServicesMenu):
+        (+[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate]): Deleted.
+        (-[WKSharingServicePickerDelegate menuProxy]): Deleted.
+        (-[WKSharingServicePickerDelegate setMenuProxy:]): Deleted.
+        (-[WKSharingServicePickerDelegate setPicker:]): Deleted.
+        (-[WKSharingServicePickerDelegate setIncludeEditorServices:]): Deleted.
+        (-[WKSharingServicePickerDelegate sharingServicePicker:sharingServicesForItems:mask:proposedSharingServices:]): Deleted.
+        (-[WKSharingServicePickerDelegate sharingServicePicker:delegateForSharingService:]): Deleted.
+        (-[WKSharingServicePickerDelegate sharingService:willShareItems:]): Deleted.
+        (-[WKSharingServicePickerDelegate sharingService:didShareItems:]): Deleted.
+        (-[WKSharingServicePickerDelegate sharingService:sourceWindowForShareItems:sharingContentScope:]): Deleted.
+
+        * UIProcess/mac/WebPageProxyMac.mm:
+        (WebKit::WebPageProxy::platformInitializeShareMenuItem):
+
+        * WebKit2.xcodeproj/project.pbxproj:
+
 2015-04-13  Anders Carlsson  <ander...@apple.com>
 
         Begin cleaning up the shared memory interface

Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (182755 => 182756)


--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2015-04-13 21:23:09 UTC (rev 182756)
@@ -4064,8 +4064,10 @@
         }
 
         ContextMenuItem coreItem = ContextMenuItem::shareMenuItem(absoluteLinkURL, downloadableMediaURL, image.get(), contextMenuContextData.selectedText());
-        if (!coreItem.isNull())
+        if (!coreItem.isNull()) {
+            platformInitializeShareMenuItem(coreItem);
             proposedAPIItems.append(WebContextMenuItem::create(coreItem));
+        }
     }
 
     Vector<RefPtr<WebContextMenuItem>> clientItems;
@@ -4081,6 +4083,12 @@
     m_contextMenuClient->contextMenuDismissed(*this);
 }
 
+#if !PLATFORM(MAC)
+void WebPageProxy::platformInitializeShareMenuItem(ContextMenuItem&)
+{
+}
+#endif
+
 void WebPageProxy::contextMenuItemSelected(const WebContextMenuItemData& item)
 {
     // Application custom items don't need to round-trip through to WebCore in the WebProcess.

Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.h (182755 => 182756)


--- trunk/Source/WebKit2/UIProcess/WebPageProxy.h	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -1233,6 +1233,8 @@
     };
     void showContextMenu(const WebCore::IntPoint& menuLocation, const ContextMenuContextData&, const Vector<WebContextMenuItemData>&, const UserData&);
     void internalShowContextMenu(const WebCore::IntPoint& menuLocation, const ContextMenuContextData&, const Vector<WebContextMenuItemData>&, ContextMenuClientEligibility, const UserData&);
+
+    void platformInitializeShareMenuItem(WebCore::ContextMenuItem&);
 #endif
 
 #if ENABLE(TELEPHONE_NUMBER_DETECTION)

Added: trunk/Source/WebKit2/UIProcess/mac/WKSharingServicePickerDelegate.h (0 => 182756)


--- trunk/Source/WebKit2/UIProcess/mac/WKSharingServicePickerDelegate.h	                        (rev 0)
+++ trunk/Source/WebKit2/UIProcess/mac/WKSharingServicePickerDelegate.h	2015-04-13 21:23:09 UTC (rev 182756)
@@ -0,0 +1,55 @@
+/*
+ * Copyright (C) 2015 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef WKSharingServicePickerDelegate_h
+#define WKSharingServicePickerDelegate_h
+
+#if ENABLE(SERVICE_CONTROLS)
+
+#include <wtf/RetainPtr.h>
+
+namespace WebKit {
+class WebContextMenuProxyMac;
+}
+
+@class NSSharingServicePicker;
+
+@interface WKSharingServicePickerDelegate : NSObject <NSSharingServiceDelegate, NSSharingServicePickerDelegate> {
+    WebKit::WebContextMenuProxyMac* _menuProxy;
+    RetainPtr<NSSharingServicePicker> _picker;
+    BOOL _filterEditingServices;
+    BOOL _handleEditingReplacement;
+}
+
++ (WKSharingServicePickerDelegate *)sharedSharingServicePickerDelegate;
+- (WebKit::WebContextMenuProxyMac*)menuProxy;
+- (void)setMenuProxy:(WebKit::WebContextMenuProxyMac*)menuProxy;
+- (void)setPicker:(NSSharingServicePicker *)picker;
+- (void)setFiltersEditingServices:(BOOL)filtersEditingServices;
+- (void)setHandlesEditingReplacement:(BOOL)handlesEditingReplacement;
+@end
+
+#endif // ENABLE(SERVICE_CONTROLS)
+#endif // WKSharingServicePickerDelegate_h

Added: trunk/Source/WebKit2/UIProcess/mac/WKSharingServicePickerDelegate.mm (0 => 182756)


--- trunk/Source/WebKit2/UIProcess/mac/WKSharingServicePickerDelegate.mm	                        (rev 0)
+++ trunk/Source/WebKit2/UIProcess/mac/WKSharingServicePickerDelegate.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -0,0 +1,142 @@
+/*
+ * Copyright (C) 2015 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#import "config.h"
+#import "WKSharingServicePickerDelegate.h"
+
+#if ENABLE(SERVICE_CONTROLS)
+
+#import "DataReference.h"
+#import "WebContextMenuProxyMac.h"
+#import "WebPageProxy.h"
+#import <WebCore/NSSharingServicePickerSPI.h>
+#import <WebCore/NSSharingServiceSPI.h>
+#import <wtf/text/WTFString.h>
+
+// FIXME: We probably need to hang on the picker itself until the context menu operation is done, and this object will probably do that.
+@implementation WKSharingServicePickerDelegate
+
++ (WKSharingServicePickerDelegate*)sharedSharingServicePickerDelegate
+{
+    static WKSharingServicePickerDelegate* delegate = [[WKSharingServicePickerDelegate alloc] init];
+    return delegate;
+}
+
+- (WebKit::WebContextMenuProxyMac*)menuProxy
+{
+    return _menuProxy;
+}
+
+- (void)setMenuProxy:(WebKit::WebContextMenuProxyMac*)menuProxy
+{
+    _menuProxy = menuProxy;
+}
+
+- (void)setPicker:(NSSharingServicePicker *)picker
+{
+    _picker = picker;
+}
+
+- (void)setFiltersEditingServices:(BOOL)filtersEditingServices
+{
+    _filterEditingServices = filtersEditingServices;
+}
+
+- (void)setHandlesEditingReplacement:(BOOL)handlesEditingReplacement
+{
+    _handleEditingReplacement = handlesEditingReplacement;
+}
+
+- (NSArray *)sharingServicePicker:(NSSharingServicePicker *)sharingServicePicker sharingServicesForItems:(NSArray *)items mask:(NSSharingServiceMask)mask proposedSharingServices:(NSArray *)proposedServices
+{
+    if (!_filterEditingServices)
+        return proposedServices;
+
+    NSMutableArray *services = [[NSMutableArray alloc] initWithCapacity:[proposedServices count]];
+    
+    for (NSSharingService *service in proposedServices) {
+        if (service.type != NSSharingServiceTypeEditor)
+            [services addObject:service];
+    }
+    
+    return services;
+}
+
+- (id <NSSharingServiceDelegate>)sharingServicePicker:(NSSharingServicePicker *)sharingServicePicker delegateForSharingService:(NSSharingService *)sharingService
+{
+    return self;
+}
+
+- (void)sharingService:(NSSharingService *)sharingService willShareItems:(NSArray *)items
+{
+    _menuProxy->clearServicesMenu();
+}
+
+- (void)sharingService:(NSSharingService *)sharingService didShareItems:(NSArray *)items
+{
+    // We only care about what item was shared if we were interested in editor services
+    // (i.e., if we plan on replacing the selection with the returned item)
+    if (!_handleEditingReplacement)
+        return;
+
+    Vector<String> types;
+    IPC::DataReference dataReference;
+
+    id item = [items objectAtIndex:0];
+
+    if ([item isKindOfClass:[NSAttributedString class]]) {
+        NSData *data = "" RTFDFromRange:NSMakeRange(0, [item length]) documentAttributes:nil];
+        dataReference = IPC::DataReference(static_cast<const uint8_t*>([data bytes]), [data length]);
+
+        types.append(NSPasteboardTypeRTFD);
+        types.append(NSRTFDPboardType);
+    } else if ([item isKindOfClass:[NSData class]]) {
+        NSData *data = "" *)item;
+        RetainPtr<CGImageSourceRef> source = adoptCF(CGImageSourceCreateWithData((CFDataRef)data, NULL));
+        RetainPtr<CGImageRef> image = adoptCF(CGImageSourceCreateImageAtIndex(source.get(), 0, NULL));
+
+        if (!image)
+            return;
+
+        dataReference = IPC::DataReference(static_cast<const uint8_t*>([data bytes]), [data length]);
+        types.append(NSPasteboardTypeTIFF);
+    } else {
+        LOG_ERROR("sharingService:didShareItems: - Unknown item type returned\n");
+        return;
+    }
+
+    // FIXME: We should adopt replaceSelectionWithAttributedString instead of bouncing through the (fake) pasteboard.
+    _menuProxy->page().replaceSelectionWithPasteboardData(types, dataReference);
+}
+
+- (NSWindow *)sharingService:(NSSharingService *)sharingService sourceWindowForShareItems:(NSArray *)items sharingContentScope:(NSSharingContentScope *)sharingContentScope
+{
+    return _menuProxy->window();
+}
+
+@end
+
+#endif // ENABLE(SERVICE_CONTROLS)
+

Modified: trunk/Source/WebKit2/UIProcess/mac/WebContextMenuProxyMac.mm (182755 => 182756)


--- trunk/Source/WebKit2/UIProcess/mac/WebContextMenuProxyMac.mm	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit2/UIProcess/mac/WebContextMenuProxyMac.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -34,10 +34,11 @@
 #import "ServicesController.h"
 #import "ShareableBitmap.h"
 #import "StringUtilities.h"
+#import "WKSharingServicePickerDelegate.h"
+#import "WKView.h"
 #import "WebContextMenuItem.h"
 #import "WebContextMenuItemData.h"
 #import "WebProcessProxy.h"
-#import "WKView.h"
 #import <WebCore/GraphicsContext.h>
 #import <WebCore/IntRect.h>
 #import <WebCore/NSSharingServicePickerSPI.h>
@@ -147,119 +148,6 @@
 
 @end
 
-#if ENABLE(SERVICE_CONTROLS)
-@interface WKSharingServicePickerDelegate : NSObject <NSSharingServiceDelegate, NSSharingServicePickerDelegate> {
-    WebKit::WebContextMenuProxyMac* _menuProxy;
-    RetainPtr<NSSharingServicePicker> _picker;
-    BOOL _includeEditorServices;
-}
-
-+ (WKSharingServicePickerDelegate *)sharedSharingServicePickerDelegate;
-- (WebKit::WebContextMenuProxyMac*)menuProxy;
-- (void)setMenuProxy:(WebKit::WebContextMenuProxyMac*)menuProxy;
-- (void)setPicker:(NSSharingServicePicker *)picker;
-- (void)setIncludeEditorServices:(BOOL)includeEditorServices;
-@end
-
-// FIXME: We probably need to hang on the picker itself until the context menu operation is done, and this object will probably do that.
-@implementation WKSharingServicePickerDelegate
-+ (WKSharingServicePickerDelegate*)sharedSharingServicePickerDelegate
-{
-    static WKSharingServicePickerDelegate* delegate = [[WKSharingServicePickerDelegate alloc] init];
-    return delegate;
-}
-
-- (WebKit::WebContextMenuProxyMac*)menuProxy
-{
-    return _menuProxy;
-}
-
-- (void)setMenuProxy:(WebKit::WebContextMenuProxyMac*)menuProxy
-{
-    _menuProxy = menuProxy;
-}
-
-- (void)setPicker:(NSSharingServicePicker *)picker
-{
-    _picker = picker;
-}
-
-- (void)setIncludeEditorServices:(BOOL)includeEditorServices
-{
-    _includeEditorServices = includeEditorServices;
-}
-
-- (NSArray *)sharingServicePicker:(NSSharingServicePicker *)sharingServicePicker sharingServicesForItems:(NSArray *)items mask:(NSSharingServiceMask)mask proposedSharingServices:(NSArray *)proposedServices
-{
-    if (_includeEditorServices)
-        return proposedServices;
-
-    NSMutableArray *services = [[NSMutableArray alloc] initWithCapacity:[proposedServices count]];
-    
-    for (NSSharingService *service in proposedServices) {
-        if (service.type != NSSharingServiceTypeEditor)
-            [services addObject:service];
-    }
-    
-    return services;
-}
-
-- (id <NSSharingServiceDelegate>)sharingServicePicker:(NSSharingServicePicker *)sharingServicePicker delegateForSharingService:(NSSharingService *)sharingService
-{
-    return self;
-}
-
-- (void)sharingService:(NSSharingService *)sharingService willShareItems:(NSArray *)items
-{
-    _menuProxy->clearServicesMenu();
-}
-
-- (void)sharingService:(NSSharingService *)sharingService didShareItems:(NSArray *)items
-{
-    // We only care about what item was shared if we were interested in editor services
-    // (i.e., if we plan on replacing the selection with the returned item)
-    if (!_includeEditorServices)
-        return;
-
-    Vector<String> types;
-    IPC::DataReference dataReference;
-
-    id item = [items objectAtIndex:0];
-
-    if ([item isKindOfClass:[NSAttributedString class]]) {
-        NSData *data = "" RTFDFromRange:NSMakeRange(0, [item length]) documentAttributes:nil];
-        dataReference = IPC::DataReference(static_cast<const uint8_t*>([data bytes]), [data length]);
-
-        types.append(NSPasteboardTypeRTFD);
-        types.append(NSRTFDPboardType);
-    } else if ([item isKindOfClass:[NSData class]]) {
-        NSData *data = "" *)item;
-        RetainPtr<CGImageSourceRef> source = adoptCF(CGImageSourceCreateWithData((CFDataRef)data, NULL));
-        RetainPtr<CGImageRef> image = adoptCF(CGImageSourceCreateImageAtIndex(source.get(), 0, NULL));
-
-        if (!image)
-            return;
-
-        dataReference = IPC::DataReference(static_cast<const uint8_t*>([data bytes]), [data length]);
-        types.append(NSPasteboardTypeTIFF);
-    } else {
-        LOG_ERROR("sharingService:didShareItems: - Unknown item type returned\n");
-        return;
-    }
-
-    // FIXME: We should adopt replaceSelectionWithAttributedString instead of bouncing through the (fake) pasteboard.
-    _menuProxy->page().replaceSelectionWithPasteboardData(types, dataReference);
-}
-
-- (NSWindow *)sharingService:(NSSharingService *)sharingService sourceWindowForShareItems:(NSArray *)items sharingContentScope:(NSSharingContentScope *)sharingContentScope
-{
-    return _menuProxy->window();
-}
-
-@end
-
-#endif
-
 namespace WebKit {
 
 WebContextMenuProxyMac::WebContextMenuProxyMac(WKView* webView, WebPageProxy* page)
@@ -392,7 +280,8 @@
     [picker setStyle:hasControlledImage ? NSSharingServicePickerStyleRollover : NSSharingServicePickerStyleTextSelection];
     [picker setDelegate:[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate]];
     [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setPicker:picker.get()];
-    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setIncludeEditorServices:includeEditorServices];
+    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setFiltersEditingServices:!includeEditorServices];
+    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setHandlesEditingReplacement:includeEditorServices];
 
     m_servicesMenu = adoptNS([[picker menu] copy]);
 

Modified: trunk/Source/WebKit2/UIProcess/mac/WebPageProxyMac.mm (182755 => 182756)


--- trunk/Source/WebKit2/UIProcess/mac/WebPageProxyMac.mm	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit2/UIProcess/mac/WebPageProxyMac.mm	2015-04-13 21:23:09 UTC (rev 182756)
@@ -43,6 +43,8 @@
 #import "StringUtilities.h"
 #import "TextChecker.h"
 #import "WKBrowsingContextControllerInternal.h"
+#import "WKSharingServicePickerDelegate.h"
+#import "WebContextMenuProxyMac.h"
 #import "WebPageMessages.h"
 #import "WebProcessProxy.h"
 #import <WebCore/DictationAlternative.h>
@@ -745,7 +747,24 @@
     }
 #endif
 }
+
+void WebPageProxy::platformInitializeShareMenuItem(ContextMenuItem& item)
+{
+#if ENABLE(SERVICE_CONTROLS)
+    NSMenuItem *nsItem = item.platformDescription();
+
+    NSSharingServicePicker *sharingServicePicker = [nsItem representedObject];
+    sharingServicePicker.delegate = [WKSharingServicePickerDelegate sharedSharingServicePickerDelegate];
     
+    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setFiltersEditingServices:NO];
+    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setHandlesEditingReplacement:NO];
+    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setMenuProxy:static_cast<WebContextMenuProxyMac*>(m_activeContextMenu.get())];
+
+    // Setting the picker lets the delegate retain it to keep it alive, but this picker is kept alive by the menu item.
+    [[WKSharingServicePickerDelegate sharedSharingServicePickerDelegate] setPicker:nil];
+#endif
+}
+    
 } // namespace WebKit
 
 #endif // PLATFORM(MAC)

Modified: trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj (182755 => 182756)


--- trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj	2015-04-13 21:18:29 UTC (rev 182755)
+++ trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj	2015-04-13 21:23:09 UTC (rev 182756)
@@ -926,6 +926,8 @@
 		513A163D163088F6005D7D22 /* NetworkProcessProxyMessages.h in Headers */ = {isa = PBXBuildFile; fileRef = 513A163B163088F6005D7D22 /* NetworkProcessProxyMessages.h */; };
 		513A164C1630A9BF005D7D22 /* NetworkConnectionToWebProcess.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 513A16491630A9BF005D7D22 /* NetworkConnectionToWebProcess.cpp */; };
 		513A164D1630A9BF005D7D22 /* NetworkConnectionToWebProcess.h in Headers */ = {isa = PBXBuildFile; fileRef = 513A164A1630A9BF005D7D22 /* NetworkConnectionToWebProcess.h */; };
+		513E462D1AD837560016234A /* WKSharingServicePickerDelegate.h in Headers */ = {isa = PBXBuildFile; fileRef = 513E462B1AD837560016234A /* WKSharingServicePickerDelegate.h */; };
+		513E462E1AD837560016234A /* WKSharingServicePickerDelegate.mm in Sources */ = {isa = PBXBuildFile; fileRef = 513E462C1AD837560016234A /* WKSharingServicePickerDelegate.mm */; };
 		514BDED316C98EDD00E4E25E /* StatisticsRequest.h in Headers */ = {isa = PBXBuildFile; fileRef = 514BDED216C98EDD00E4E25E /* StatisticsRequest.h */; };
 		514D9F5719119D35000063A7 /* ServicesController.h in Headers */ = {isa = PBXBuildFile; fileRef = 514D9F5519119D35000063A7 /* ServicesController.h */; };
 		514D9F5819119D35000063A7 /* ServicesController.mm in Sources */ = {isa = PBXBuildFile; fileRef = 514D9F5619119D35000063A7 /* ServicesController.mm */; };
@@ -3117,6 +3119,8 @@
 		513A16491630A9BF005D7D22 /* NetworkConnectionToWebProcess.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = NetworkConnectionToWebProcess.cpp; path = NetworkProcess/NetworkConnectionToWebProcess.cpp; sourceTree = "<group>"; };
 		513A164A1630A9BF005D7D22 /* NetworkConnectionToWebProcess.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = NetworkConnectionToWebProcess.h; path = NetworkProcess/NetworkConnectionToWebProcess.h; sourceTree = "<group>"; };
 		513A164B1630A9BF005D7D22 /* NetworkConnectionToWebProcess.messages.in */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; name = NetworkConnectionToWebProcess.messages.in; path = NetworkProcess/NetworkConnectionToWebProcess.messages.in; sourceTree = "<group>"; };
+		513E462B1AD837560016234A /* WKSharingServicePickerDelegate.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WKSharingServicePickerDelegate.h; sourceTree = "<group>"; };
+		513E462C1AD837560016234A /* WKSharingServicePickerDelegate.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WKSharingServicePickerDelegate.mm; sourceTree = "<group>"; };
 		514BDED216C98EDD00E4E25E /* StatisticsRequest.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = StatisticsRequest.h; sourceTree = "<group>"; };
 		514D9F5519119D35000063A7 /* ServicesController.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ServicesController.h; sourceTree = "<group>"; };
 		514D9F5619119D35000063A7 /* ServicesController.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = ServicesController.mm; sourceTree = "<group>"; };
@@ -7066,6 +7070,8 @@
 				0FCB4E5D18BBE3D9000FCFC9 /* WKPrintingView.mm */,
 				0FCB4E5E18BBE3D9000FCFC9 /* WKTextInputWindowController.h */,
 				0FCB4E5F18BBE3D9000FCFC9 /* WKTextInputWindowController.mm */,
+				513E462B1AD837560016234A /* WKSharingServicePickerDelegate.h */,
+				513E462C1AD837560016234A /* WKSharingServicePickerDelegate.mm */,
 			);
 			path = mac;
 			sourceTree = "<group>";
@@ -7939,6 +7945,7 @@
 				1A5E4DA412D3BD3D0099A2BB /* TextCheckerState.h in Headers */,
 				CE1A0BD71A48E6C60054EF74 /* TextInputSPI.h in Headers */,
 				1AAF263914687C39004A1E8A /* TiledCoreAnimationDrawingArea.h in Headers */,
+				513E462D1AD837560016234A /* WKSharingServicePickerDelegate.h in Headers */,
 				1AF05D8714688348008B1E81 /* TiledCoreAnimationDrawingAreaProxy.h in Headers */,
 				1AFE436618B6C081009C7A48 /* UIDelegate.h in Headers */,
 				CEE4AE2B1A5DCF430002F49B /* UIKitSPI.h in Headers */,
@@ -9868,6 +9875,7 @@
 				31A2EC48148997C200810D71 /* WebNotification.cpp in Sources */,
 				31099973146C75A20029DEB9 /* WebNotificationClient.cpp in Sources */,
 				31312620148FF82C00BA2A39 /* WebNotificationManager.cpp in Sources */,
+				513E462E1AD837560016234A /* WKSharingServicePickerDelegate.mm in Sources */,
 				31BA924D148831260062EDB5 /* WebNotificationManagerMessageReceiver.cpp in Sources */,
 				31A2EC4A148997C200810D71 /* WebNotificationManagerProxy.cpp in Sources */,
 				31A2EC4D148997C200810D71 /* WebNotificationProvider.cpp in Sources */,
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to