Title: [248609] branches/safari-608-branch/Source/WebKit
Revision
248609
Author
alanc...@apple.com
Date
2019-08-13 13:00:40 -0700 (Tue, 13 Aug 2019)

Log Message

Cherry-pick r248174. rdar://problem/54237758

    [iPadOS] slides.google.com: Full Screen API warning is presented when swiping through slides
    https://bugs.webkit.org/show_bug.cgi?id=200313
    <rdar://problem/53777001>

    Reviewed by Eric Carlson.

    Only run the secheuristic scoring during UIGestureRecognizerStateEnded, rather than both
    UIGestureRecognizerStateEnded and UIGestureRecognizerStateBegan. The goal of the heuristic is
    to detect fake on-screen keyboards by detecting gestures that look like "typing". Using only
    UIGestureRecognizerStateEnded still allows us to do this (as typing will usually have identical
    geometries for both Ended and Began) without generating false-positives during swipe gestures.

    * UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
    (-[WKFullScreenViewController _touchDetected:]):

    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248174 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Modified Paths

Diff

Modified: branches/safari-608-branch/Source/WebKit/ChangeLog (248608 => 248609)


--- branches/safari-608-branch/Source/WebKit/ChangeLog	2019-08-13 20:00:37 UTC (rev 248608)
+++ branches/safari-608-branch/Source/WebKit/ChangeLog	2019-08-13 20:00:40 UTC (rev 248609)
@@ -1,3 +1,42 @@
+2019-08-13  Alan Coon  <alanc...@apple.com>
+
+        Cherry-pick r248174. rdar://problem/54237758
+
+    [iPadOS] slides.google.com: Full Screen API warning is presented when swiping through slides
+    https://bugs.webkit.org/show_bug.cgi?id=200313
+    <rdar://problem/53777001>
+    
+    Reviewed by Eric Carlson.
+    
+    Only run the secheuristic scoring during UIGestureRecognizerStateEnded, rather than both
+    UIGestureRecognizerStateEnded and UIGestureRecognizerStateBegan. The goal of the heuristic is
+    to detect fake on-screen keyboards by detecting gestures that look like "typing". Using only
+    UIGestureRecognizerStateEnded still allows us to do this (as typing will usually have identical
+    geometries for both Ended and Began) without generating false-positives during swipe gestures.
+    
+    * UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
+    (-[WKFullScreenViewController _touchDetected:]):
+    
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248174 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2019-08-02  Jer Noble  <jer.no...@apple.com>
+
+            [iPadOS] slides.google.com: Full Screen API warning is presented when swiping through slides
+            https://bugs.webkit.org/show_bug.cgi?id=200313
+            <rdar://problem/53777001>
+
+            Reviewed by Eric Carlson.
+
+            Only run the secheuristic scoring during UIGestureRecognizerStateEnded, rather than both
+            UIGestureRecognizerStateEnded and UIGestureRecognizerStateBegan. The goal of the heuristic is
+            to detect fake on-screen keyboards by detecting gestures that look like "typing". Using only
+            UIGestureRecognizerStateEnded still allows us to do this (as typing will usually have identical
+            geometries for both Ended and Began) without generating false-positives during swipe gestures.
+
+            * UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
+            (-[WKFullScreenViewController _touchDetected:]):
+
 2019-08-12  Alan Coon  <alanc...@apple.com>
 
         Cherry-pick r248502. rdar://problem/54130670

Modified: branches/safari-608-branch/Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm (248608 => 248609)


--- branches/safari-608-branch/Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm	2019-08-13 20:00:37 UTC (rev 248608)
+++ branches/safari-608-branch/Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm	2019-08-13 20:00:40 UTC (rev 248609)
@@ -529,7 +529,7 @@
 
 - (void)_touchDetected:(id)sender
 {
-    if ([_touchGestureRecognizer state] == UIGestureRecognizerStateBegan || [_touchGestureRecognizer state] == UIGestureRecognizerStateEnded) {
+    if ([_touchGestureRecognizer state] == UIGestureRecognizerStateEnded) {
         double score = _secheuristic.scoreOfNextTouch([_touchGestureRecognizer locationInView:self.view]);
         if (score > requiredScore)
             [self _showPhishingAlert];
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to