Reviewed by Adele.
authorggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Jul 2006 08:07:05 +0000 (08:07 +0000)
committerggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Jul 2006 08:07:05 +0000 (08:07 +0000)
        - Build fix: don't need BLOCK_OBJC_EXCEPTIONS because we already have
        one surrounding this function, and the nested one makes the compiler
        think our local variable is volatile (seems like a compiler bug to me).

        * bridge/mac/FrameMac.mm:
        (WebCore::FrameMac::sendContextMenuEvent):

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

WebCore/ChangeLog
WebCore/bridge/mac/FrameMac.mm

index 0c6abbc..5600867 100644 (file)
@@ -1,3 +1,14 @@
+2006-07-14  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Adele.
+
+        - Build fix: don't need BLOCK_OBJC_EXCEPTIONS because we already have
+        one surrounding this function, and the nested one makes the compiler
+        think our local variable is volatile (seems like a compiler bug to me).
+        
+        * bridge/mac/FrameMac.mm:
+        (WebCore::FrameMac::sendContextMenuEvent):
+
 2006-07-14  Timothy Hatcher  <timothy@apple.com>
 
         Reviewed by Maciej.
index aa9648c..c20dd26 100644 (file)
@@ -2174,11 +2174,8 @@ bool FrameMac::sendContextMenuEvent(NSEvent *event)
         // Since we're about to make a new selection in this view,
         // make sure this view is first responder
         NSView *view = d->m_view->getDocumentView();
-        BEGIN_BLOCK_OBJC_EXCEPTIONS;
-        if ([_bridge firstResponder] != view) {
+        if ([_bridge firstResponder] != view)
             [_bridge makeFirstResponder:view];
-        }
-        END_BLOCK_OBJC_EXCEPTIONS;
 
         _mouseDownMayStartSelect = true; // context menu events are always allowed to perform a selection
         selectClosestWordFromMouseEvent(mouseEvent, mev.targetNode());