- fixed <rdar://problem/3512801>: REGRESSION (Safari 100-116):
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Dec 2003 17:36:57 +0000 (17:36 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Dec 2003 17:36:57 +0000 (17:36 +0000)
        Mike Hay's Magic 8-ball game ignores slow clicks

        Reviewed by Darin.

        * WebView.subproj/WebHTMLView.m:
        (-[WebHTMLView _handleAutoscrollForMouseDragged:]):
        start the autoscroll timer here, so the timer only runs when KHTML is
        handling the event.
        (-[WebHTMLView mouseDown:]):
        don't start the autoscroll timer here.

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

WebKit/ChangeLog
WebKit/WebView.subproj/WebHTMLView.m

index 1ea08e1850bb15b24f84c41e53173924fa82ed16..71c3878b92854156a013490b8a5970ec59d45ddd 100644 (file)
@@ -1,3 +1,17 @@
+2003-12-17  John Sullivan  <sullivan@apple.com>
+
+        - fixed <rdar://problem/3512801>: REGRESSION (Safari 100-116): 
+        Mike Hay's Magic 8-ball game ignores slow clicks
+
+        Reviewed by Darin.
+
+        * WebView.subproj/WebHTMLView.m:
+        (-[WebHTMLView _handleAutoscrollForMouseDragged:]):
+        start the autoscroll timer here, so the timer only runs when KHTML is
+        handling the event.
+        (-[WebHTMLView mouseDown:]):
+        don't start the autoscroll timer here.
+
 2003-12-16  Ken Kocienda  <kocienda@apple.com>
 
         Reviewed by me
index 7a955d5acb376044d6108a1186457ad5b0479bfa..cbef0445d4c20d279788839ecc63228c4e655a91 100644 (file)
@@ -692,8 +692,8 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)_handleAutoscrollForMouseDragged:(NSEvent *)event
 {
-    // FIXME: this really needs to be based on a timer
     [self autoscroll:event];
+    [self _startAutoscrollTimer:event];
 }
 
 - (BOOL)_mayStartDragWithMouseDragged:(NSEvent *)mouseDraggedEvent
@@ -1444,7 +1444,6 @@ static WebHTMLView *lastHitView = nil;
     }
     
     _private->ignoringMouseDraggedEvents = NO;
-    [self _startAutoscrollTimer:event];
     
     // Record the mouse down position so we can determine drag hysteresis.
     [_private->mouseDownEvent release];
@@ -1453,7 +1452,8 @@ static WebHTMLView *lastHitView = nil;
     // Don't do any mouseover while the mouse is down.
     [NSObject cancelPreviousPerformRequestsWithTarget:self selector:@selector(_updateMouseoverWithFakeEvent) object:nil];
 
-    // Let KHTML get a chance to deal with the event.
+    // Let KHTML get a chance to deal with the event. This will call back to us
+    // to start the autoscroll timer if appropriate.
     [[self _bridge] mouseDown:event];
 }