Use IsThisEnabled() to test if AUI child is enabled, not IsEnabled().

[wxWidgets.git] / src / osx / uiaction_osx.cpp

diff --git a/src/osx/uiaction_osx.cpp b/src/osx/uiaction_osx.cpp
index 603b3039774b9678b0708c76dbc42d206f1c777a..60cd4da9d8d6edc2556508835e0ee00b8304847c 100644 (file)
--- a/src/osx/uiaction_osx.cpp
+++ b/src/osx/uiaction_osx.cpp
@@ -1,11 +1,13 @@
 /////////////////////////////////////////////////////////////////////////////
 // Name:        src/osx/uiaction_osx.cpp
 // Purpose:     wxUIActionSimulator implementation
-// Author:      Kevin Ollivier
+// Author:      Kevin Ollivier, Steven Lamerton, Vadim Zeitlin
 // Modified by:
 // Created:     2010-03-06
-// RCS-ID:      $Id: menu.cpp 54129 2008-06-11 19:30:52Z SC $
+// RCS-ID:      $Id$
 // Copyright:   (c) Kevin Ollivier
+//              (c) 2010 Steven Lamerton
+//              (c) 2010 Vadim Zeitlin
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
@@ -15,61 +17,63 @@
 
 #include "wx/uiaction.h"
 
-#include <ApplicationServices/ApplicationServices.h>
+#include "wx/log.h"
 
+#include "wx/osx/private.h"
+#include "wx/osx/core/cfref.h"
+
+namespace
+{
+    
 CGEventTapLocation tap = kCGSessionEventTap;
 
 CGEventType CGEventTypeForMouseButton(int button, bool isDown)
 {
-    switch (button)
+    switch ( button )
     {
         case wxMOUSE_BTN_LEFT:
-            if (isDown)
-                return kCGEventLeftMouseDown;
-            else
-                return kCGEventLeftMouseUp;
-        case wxMOUSE_BTN_RIGHT:
-            if (isDown)
-                return kCGEventRightMouseDown;
-            else
-                return kCGEventRightMouseUp;
+            return isDown ? kCGEventLeftMouseDown : kCGEventLeftMouseUp;
 
-        // Apparently all other buttons use the constant OtherMouseDown
+        case wxMOUSE_BTN_RIGHT:
+            return isDown ? kCGEventRightMouseDown : kCGEventRightMouseUp;
 
+        // All the other buttons use the constant OtherMouseDown but we still
+        // want to check for invalid parameters so assert first
         default:
-            if (isDown)
-                return kCGEventOtherMouseDown;
-            else
-                return kCGEventOtherMouseUp;
-    }
-}
+            wxFAIL_MSG("Unsupported button passed in.");
+            // fall back to the only known remaining case
 
-void SendCharCode(CGCharCode keycode, bool isDown)
-{
-    CGEventRef event = CGEventCreateKeyboardEvent(NULL, keycode, isDown);
-    if (event)
-    {
-        CGEventPost(kCGHIDEventTap, event);
+        case wxMOUSE_BTN_MIDDLE:
+            return isDown ? kCGEventOtherMouseDown : kCGEventOtherMouseUp;
     }
-    CFRelease(event);
 }
 
-bool wxUIActionSimulator::MouseDown(int button)
+CGPoint GetMousePosition()
 {
-    CGPoint pos;
     int x, y;
     wxGetMousePosition(&x, &y);
+
+    CGPoint pos;
     pos.x = x;
     pos.y = y;
+
+    return pos;
+}
+
+} // anonymous namespace
+
+bool wxUIActionSimulator::MouseDown(int button)
+{
     CGEventType type = CGEventTypeForMouseButton(button, true);
-    CGEventRef event = CGEventCreateMouseEvent(NULL, type, pos, button);
+    wxCFRef<CGEventRef> event(
+            CGEventCreateMouseEvent(NULL, type, GetMousePosition(), button));
+
+    if ( !event )
+        return false;
+
     CGEventSetType(event, type);
+    CGEventPost(tap, event);
 
-    if (event)
-    {
-        CGEventPost(tap, event);
-    }
-    CFRelease(event);
     return true;
 }
 
@@ -78,57 +82,46 @@ bool wxUIActionSimulator::MouseMove(long x, long y)
     CGPoint pos;
     pos.x = x;
     pos.y = y;
+
     CGEventType type = kCGEventMouseMoved;
-    CGEventRef event = CGEventCreateMouseEvent(NULL, type, pos, kCGMouseButtonLeft);
-    CGEventSetType(event, type);
+    wxCFRef<CGEventRef> event(
+            CGEventCreateMouseEvent(NULL, type, pos, kCGMouseButtonLeft));
 
-    if (event)
-    {
-        CGEventPost(tap, event);
-    }
-    CFRelease(event);
+    if ( !event )
+        return false;
+
+    CGEventSetType(event, type);
+    CGEventPost(tap, event);
 
     return true;
 }
 
 bool wxUIActionSimulator::MouseUp(int button)
 {
-    CGPoint pos;
-    int x, y;
-    wxGetMousePosition(&x, &y);
-    pos.x = x;
-    pos.y = y;
     CGEventType type = CGEventTypeForMouseButton(button, false);
-    CGEventRef event = CGEventCreateMouseEvent(NULL, type, pos, button);
-    CGEventSetType(event, type);
+    wxCFRef<CGEventRef> event(
+            CGEventCreateMouseEvent(NULL, type, GetMousePosition(), button));
 
-    if (event)
-    {
-        CGEventPost(tap, event);
-    }
-    CFRelease(event);
+    if ( !event )
+        return false;
+
+    CGEventSetType(event, type);
+    CGEventPost(tap, event);
 
     return true;
 }
 
-bool wxUIActionSimulator::Key(int keycode, bool isDown, bool shiftDown, bool cmdDown, bool altDown)
+bool
+wxUIActionSimulator::DoKey(int keycode, int WXUNUSED(modifiers), bool isDown)
 {
-    if (shiftDown)
-        SendCharCode((CGCharCode)56, true);
-    if (altDown)
-        SendCharCode((CGCharCode)58, true);
-    if (cmdDown)
-        SendCharCode((CGCharCode)55, true);
-
-    SendCharCode((CGCharCode)keycode, isDown);
-
-    if (shiftDown)
-        SendCharCode((CGCharCode)56, false);
-    if (altDown)
-        SendCharCode((CGCharCode)58, false);
-    if (cmdDown)
-        SendCharCode((CGCharCode)55, false);
+    CGKeyCode cgcode = wxCharCodeWXToOSX((wxKeyCode)keycode);
+
+    wxCFRef<CGEventRef>
+        event(CGEventCreateKeyboardEvent(NULL, cgcode, isDown));
+    if ( !event )
+        return false;
 
+    CGEventPost(kCGHIDEventTap, event);
     return true;
 }