Reviewed by Justin Garcia.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Jul 2005 21:52:11 +0000 (21:52 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Jul 2005 21:52:11 +0000 (21:52 +0000)
        Mail (running on tip of tree WebKit) was running into an assertion I recently added.
        The assertion is actually correct, catching an old bug in this code.

        * WebView.subproj/WebView.m:
        (-[WebView selectedFrame]):
        if the first responder is a WebFrameView, then we've found the WebFrameView we're
        looking for, and we shouldn't look at its superviews.

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

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

index 1babdab..e49c64e 100644 (file)
@@ -1,5 +1,17 @@
 2005-07-22  John Sullivan  <sullivan@apple.com>
 
+        Reviewed by Justin Garcia.
+        
+        Mail (running on tip of tree WebKit) was running into an assertion I recently added.
+        The assertion is actually correct, catching an old bug in this code.
+
+        * WebView.subproj/WebView.m:
+        (-[WebView selectedFrame]):
+        if the first responder is a WebFrameView, then we've found the WebFrameView we're 
+        looking for, and we shouldn't look at its superviews.
+
+2005-07-22  John Sullivan  <sullivan@apple.com>
+
         Reviewed by Chris Blumenberg.
 
         - some changes in the direction of weaning all the form-related SPI from NSView
index a1692df..77ed88a 100644 (file)
@@ -2548,7 +2548,9 @@ static WebFrame *incrementFrame(WebFrame *curr, BOOL forward, BOOL wrapFlag)
     // where the focused frame doesn't actually contain a selection.
     NSResponder *resp = [[self window] firstResponder];
     if (resp && [resp isKindOfClass:[NSView class]] && [(NSView *)resp isDescendantOf:self]) {
-        WebFrameView *frameView = (WebFrameView *)[(NSView *)resp _web_superviewOfClass:[WebFrameView class]];
+        WebFrameView *frameView = [resp isKindOfClass:[WebFrameView class]] 
+            ? (WebFrameView *)resp 
+            : (WebFrameView *)[(NSView *)resp _web_superviewOfClass:[WebFrameView class]];
         ASSERT(frameView != nil);
 #ifndef NDEBUG
         WebFrame *frameWithSelection = [self _findSelectedFrame];