Title: [176641] branches/safari-600.3-branch/Source/WebKit2
Revision
176641
Author
[email protected]
Date
2014-12-02 06:54:50 -0800 (Tue, 02 Dec 2014)

Log Message

Merged r176428. rdar://problems/19052196

Modified Paths

Diff

Modified: branches/safari-600.3-branch/Source/WebKit2/ChangeLog (176640 => 176641)


--- branches/safari-600.3-branch/Source/WebKit2/ChangeLog	2014-12-02 14:50:57 UTC (rev 176640)
+++ branches/safari-600.3-branch/Source/WebKit2/ChangeLog	2014-12-02 14:54:50 UTC (rev 176641)
@@ -1,3 +1,19 @@
+2014-12-02  Dana Burkart  <[email protected]>
+
+        Merge r176428. <rdar://problem/19052196>
+
+    2014-11-20  Conrad Shultz  <[email protected]>
+
+            Clicks on previews can be recognized multiple times
+            https://bugs.webkit.org/show_bug.cgi?id=138944
+
+            Reviewed by Tim Horton.
+
+            * UIProcess/mac/WKActionMenuController.mm:
+            (-[WKPagePreviewViewController _clickRecognized:]):
+            Only dispatch the delegate message on the appropriate gesture recognizer state transition.
+
+
 2014-11-20  Conrad Shultz  <[email protected]>
 
         Clients should be able to customize preview click behavior

Modified: branches/safari-600.3-branch/Source/WebKit2/UIProcess/mac/WKActionMenuController.mm (176640 => 176641)


--- branches/safari-600.3-branch/Source/WebKit2/UIProcess/mac/WKActionMenuController.mm	2014-12-02 14:50:57 UTC (rev 176640)
+++ branches/safari-600.3-branch/Source/WebKit2/UIProcess/mac/WKActionMenuController.mm	2014-12-02 14:54:50 UTC (rev 176641)
@@ -128,7 +128,8 @@
 
 - (void)_clickRecognized:(NSGestureRecognizer *)gestureRecognizer
 {
-    [_delegate pagePreviewViewControllerWasClicked:self];
+    if (gestureRecognizer.state == NSGestureRecognizerStateBegan)
+        [_delegate pagePreviewViewControllerWasClicked:self];
 }
 
 @end
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to