ASSERT when right clicking on SVG Image generating Share menu - can break Web Inspector
[WebKit-https.git] / Source / WebKit2 / ChangeLog
index 98b600bd168e44bbf197228e0cce8e44f45dfaa9..3fca7dc7e5bb04fae028167b466dc9e2ba5363c7 100644 (file)
@@ -1,3 +1,22 @@
+2015-10-20  Joseph Pecoraro  <pecoraro@apple.com>
+
+        ASSERT when right clicking on SVG Image generating Share menu - can break Web Inspector
+        https://bugs.webkit.org/show_bug.cgi?id=150374
+
+        Reviewed by Brian Burg.
+
+        When there is no absolute URL or downloadable media URL no system share
+        menu is created, but we were still appending the null context menu to
+        the list of context menu items.
+
+        * UIProcess/mac/WebContextMenuProxyMac.mm:
+        (WebKit::WebContextMenuProxyMac::shareMenuItem):
+        Bail early if the context menu is null.
+
+        (WebKit::WebContextMenuProxyMac::showContextMenu):
+        Only append the share context menu item if it is non-null.
+        This matches ContextMenuController.cpp in WebCore.
+
 2015-10-20  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r191306.