Title: [176428] trunk/Source/WebKit2
Revision
176428
Author
[email protected]
Date
2014-11-20 16:57:53 -0800 (Thu, 20 Nov 2014)

Log Message

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.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (176427 => 176428)


--- trunk/Source/WebKit2/ChangeLog	2014-11-21 00:36:45 UTC (rev 176427)
+++ trunk/Source/WebKit2/ChangeLog	2014-11-21 00:57:53 UTC (rev 176428)
@@ -1,5 +1,16 @@
 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
         https://bugs.webkit.org/show_bug.cgi?id=138940
 

Modified: trunk/Source/WebKit2/UIProcess/mac/WKActionMenuController.mm (176427 => 176428)


--- trunk/Source/WebKit2/UIProcess/mac/WKActionMenuController.mm	2014-11-21 00:36:45 UTC (rev 176427)
+++ trunk/Source/WebKit2/UIProcess/mac/WKActionMenuController.mm	2014-11-21 00:57:53 UTC (rev 176428)
@@ -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