]> git.saurik.com Git - wxWidgets.git/blobdiff - src/cocoa/window.mm
fix ownerdrawn button label drawing in ODS_NOACCEL case (#3519)
[wxWidgets.git] / src / cocoa / window.mm
index 01c776073b85c3ecac6adf824b85bb1660dfdcb6..5b54eb71a456bb0757db0f9eab9455dfdab299d8 100644 (file)
     #include "wx/log.h"
     #include "wx/window.h"
     #include "wx/dc.h"
+    #include "wx/dcclient.h"
     #include "wx/utils.h"
 #endif //WX_PRECOMP
 
 #include "wx/tooltip.h"
 
+#include "wx/cocoa/dc.h"
 #include "wx/cocoa/autorelease.h"
 #include "wx/cocoa/string.h"
 #include "wx/cocoa/trackingrectmanager.h"
-#include "wx/mac/corefoundation/cfref.h"
+#include "wx/cocoa/private/scrollview.h"
+#include "wx/osx/core/cfref.h"
+#include "wx/cocoa/ObjcRef.h"
 
 #import <Foundation/NSArray.h>
 #import <Foundation/NSRunLoop.h>
 #include "wx/cocoa/objc/NSView.h"
 #import <AppKit/NSEvent.h>
 #import <AppKit/NSScrollView.h>
+#import <AppKit/NSScroller.h>
 #import <AppKit/NSColor.h>
 #import <AppKit/NSClipView.h>
 #import <Foundation/NSException.h>
@@ -285,29 +290,6 @@ private:
     wxWindowCocoaHider();
 };
 
-// ========================================================================
-// wxWindowCocoaScrollView
-// ========================================================================
-class wxWindowCocoaScrollView: protected wxCocoaNSView
-{
-    DECLARE_NO_COPY_CLASS(wxWindowCocoaScrollView)
-public:
-    wxWindowCocoaScrollView(wxWindow *owner);
-    virtual ~wxWindowCocoaScrollView();
-    inline WX_NSScrollView GetNSScrollView() { return m_cocoaNSScrollView; }
-    void ClientSizeToSize(int &width, int &height);
-    void DoGetClientSize(int *x, int *y) const;
-    void Encapsulate();
-    void Unencapsulate();
-protected:
-    wxWindowCocoa *m_owner;
-    WX_NSScrollView m_cocoaNSScrollView;
-    virtual void Cocoa_FrameChanged(void);
-    virtual void Cocoa_synthesizeMouseMoved(void) {}
-private:
-    wxWindowCocoaScrollView();
-};
-
 // ========================================================================
 // wxDummyNSView
 // ========================================================================
@@ -327,6 +309,12 @@ WX_IMPLEMENT_GET_OBJC_CLASS(wxDummyNSView,NSView)
 
 // ========================================================================
 // wxWindowCocoaHider
+//  NOTE: This class and method of hiding predates setHidden: support in
+//  the toolkit.  The hack used here is to replace the view with a stand-in
+//  that will be subject to the usual Cocoa resizing rules.
+//  When possible (i.e. when running on 10.3 or higher) we make it hidden
+//  mostly as an optimization so Cocoa doesn't have to consider it when
+//  drawing or finding key views.
 // ========================================================================
 wxWindowCocoaHider::wxWindowCocoaHider(wxWindow *owner)
 :   m_owner(owner)
@@ -337,6 +325,9 @@ wxWindowCocoaHider::wxWindowCocoaHider(wxWindow *owner)
         initWithFrame:[owner->GetNSViewForHiding() frame]];
     [m_dummyNSView setAutoresizingMask: [owner->GetNSViewForHiding() autoresizingMask]];
     AssociateNSView(m_dummyNSView);
+
+    if([m_dummyNSView respondsToSelector:@selector(setHidden:)])
+        [m_dummyNSView setHidden:YES];
 }
 
 wxWindowCocoaHider::~wxWindowCocoaHider()
@@ -364,6 +355,140 @@ bool wxWindowCocoaHider::Cocoa_drawRect(const NSRect& rect)
 }
 #endif //def WXCOCOA_FILL_DUMMY_VIEW
 
+
+/*! @class WXManualScrollView
+    @abstract   An NSScrollView subclass which implements wx scroll behavior
+    @discussion
+    Overrides default behavior of NSScrollView such that this class receives
+    the scroller action messages and allows the wxCocoaScrollView to act
+    on them accordingly.  In particular, because the NSScrollView will not
+    receive action messages from the scroller, it will not adjust the
+    document view.  This must be done manually using the ScrollWindow
+    method of wxWindow.
+ */
+@interface WXManualScrollView : NSScrollView
+{
+    /*! @field      m_wxCocoaScrollView
+     */
+    wxWindowCocoaScrollView *m_wxCocoaScrollView;
+}
+
+// Override these to set up the target/action correctly
+- (void)setHorizontalScroller:(NSScroller *)aScroller;
+- (void)setVerticalScroller:(NSScroller *)aScroller;
+- (void)setHasHorizontalScroller:(BOOL)flag;
+- (void)setHasVerticalScroller:(BOOL)flag;
+
+// NOTE: _wx_ prefix means "private" method like _ that Apple (and only Apple) uses.
+- (wxWindowCocoaScrollView*)_wx_wxCocoaScrollView;
+- (void)_wx_setWxCocoaScrollView:(wxWindowCocoaScrollView*)theWxScrollView;
+
+/*! @method     _wx_doScroller
+    @abstract   Handles action messages from the scrollers
+    @discussion
+    Similar to Apple's _doScroller: method which is private and not documented.
+    We do not, however, call that method.  Instead, we effectively override
+    it entirely.  We don't override it by naming ourself the same thing because
+    the base class code may or may not call that method for other reasons we
+    simply cannot know about.
+ */
+- (void)_wx_doScroller:(id)sender;
+
+@end
+
+
+@implementation WXManualScrollView : NSScrollView
+
+static inline void WXManualScrollView_DoSetScrollerTargetAction(WXManualScrollView *self, NSScroller *aScroller)
+{
+    if(aScroller != NULL && [self _wx_wxCocoaScrollView] != NULL)
+    {
+        [aScroller setTarget:self];
+        [aScroller setAction:@selector(_wx_doScroller:)];
+    }
+}
+
+- (void)setHorizontalScroller:(NSScroller *)aScroller
+{
+    [super setHorizontalScroller:aScroller];
+    WXManualScrollView_DoSetScrollerTargetAction(self, aScroller);
+}
+
+- (void)setVerticalScroller:(NSScroller *)aScroller
+{
+    [super setVerticalScroller:aScroller];
+    WXManualScrollView_DoSetScrollerTargetAction(self, aScroller);
+}
+
+- (void)setHasHorizontalScroller:(BOOL)flag
+{
+    [super setHasHorizontalScroller:flag];
+    WXManualScrollView_DoSetScrollerTargetAction(self, [self horizontalScroller]);
+}
+
+- (void)setHasVerticalScroller:(BOOL)flag
+{
+    [super setHasVerticalScroller:flag];
+    WXManualScrollView_DoSetScrollerTargetAction(self, [self verticalScroller]);
+}
+
+- (wxWindowCocoaScrollView*)_wx_wxCocoaScrollView
+{   return m_wxCocoaScrollView; }
+
+- (void)_wx_setWxCocoaScrollView:(wxWindowCocoaScrollView*)theWxScrollView
+{
+    m_wxCocoaScrollView = theWxScrollView;
+    [self setHorizontalScroller:[self horizontalScroller]];
+    [self setVerticalScroller:[self verticalScroller]];
+}
+
+- (void)_wx_doScroller:(id)sender
+{
+    if(m_wxCocoaScrollView != NULL)
+        m_wxCocoaScrollView->_wx_doScroller(sender);
+    else
+        wxLogError(wxT("Unexpected action message received from NSScroller"));
+}
+
+- (void)reflectScrolledClipView:(NSClipView *)aClipView
+{
+    struct _ScrollerBackup
+    {
+        _ScrollerBackup(NSScroller *aScroller)
+        :   m_scroller(aScroller)
+        ,   m_floatValue(aScroller!=nil?[aScroller floatValue]:0.0)
+        ,   m_knobProportion(aScroller!=nil?[aScroller knobProportion]:1.0)
+        ,   m_isEnabled(aScroller!=nil?[aScroller isEnabled]:false)
+        {
+        }
+        NSScroller *m_scroller;
+        CGFloat m_floatValue;
+        CGFloat m_knobProportion;
+        BOOL m_isEnabled;
+        ~_ScrollerBackup()
+        {
+            if(m_scroller != nil)
+            {
+                [m_scroller setFloatValue:m_floatValue knobProportion:m_knobProportion];
+                [m_scroller setEnabled:m_isEnabled];
+            }
+        }
+    private:
+        _ScrollerBackup();
+        _ScrollerBackup(_ScrollerBackup const&);
+        _ScrollerBackup& operator=(_ScrollerBackup const&);
+    };
+    _ScrollerBackup _horizontalBackup([self horizontalScroller]);
+    _ScrollerBackup _verticalBackup([self verticalScroller]);
+    // We MUST call super's implementation or else nothing seems to work right at all.
+    // However, we need our scrollers not to change values due to the document window
+    // moving so we cheat and save/restore their values across this call.
+    [super reflectScrolledClipView: aClipView];
+}
+
+@end
+WX_IMPLEMENT_GET_OBJC_CLASS(WXManualScrollView,NSScrollView)
+
 // ========================================================================
 // wxFlippedNSClipView
 // ========================================================================
@@ -386,13 +511,35 @@ WX_IMPLEMENT_GET_OBJC_CLASS(wxFlippedNSClipView,NSClipView)
 // ========================================================================
 wxWindowCocoaScrollView::wxWindowCocoaScrollView(wxWindow *owner)
 :   m_owner(owner)
+,   m_cocoaNSScrollView() // nil
+,   m_scrollRange() // {0,0}
+,   m_scrollThumb() // {0,0}
+,   m_virtualOrigin(0,0)
 {
     wxAutoNSAutoreleasePool pool;
     wxASSERT(owner);
     wxASSERT(owner->GetNSView());
-    m_cocoaNSScrollView = [[NSScrollView alloc]
+    m_isNativeView = ![owner->GetNSView() isKindOfClass:[WX_GET_OBJC_CLASS(WXNSView) class]];
+    m_cocoaNSScrollView = [(m_isNativeView?[NSScrollView alloc]:[WXManualScrollView alloc])
         initWithFrame:[owner->GetNSView() frame]];
     AssociateNSView(m_cocoaNSScrollView);
+    if(m_isNativeView)
+    {
+        /*  Set a bezel border around the entire thing because it looks funny without it.
+            TODO: Be sure to undo any borders on the real view (if any) and apply them
+            to this view if necessary. Right now, there is no border support in wxCocoa
+            so this isn't an issue.
+         */
+        [m_cocoaNSScrollView setBorderType:NSBezelBorder];
+    }
+    else
+    {
+        [(WXManualScrollView*)m_cocoaNSScrollView _wx_setWxCocoaScrollView: this];
+        // Don't set a bezel because we might be creating a scroll view due to being
+        // the "target window" of a wxScrolledWindow.  That is to say that the user
+        // has absolutely no intention of scrolling the clip view used by this
+        // NSScrollView.
+    }
 
     /* Replace the default NSClipView with a flipped one.  This ensures
        scrolling is "pinned" to the top-left instead of bottom-right. */
@@ -401,9 +548,7 @@ wxWindowCocoaScrollView::wxWindowCocoaScrollView(wxWindow *owner)
     [m_cocoaNSScrollView setContentView:flippedClip];
     [flippedClip release];
 
-    [m_cocoaNSScrollView setBackgroundColor: [NSColor windowBackgroundColor]];
-    [m_cocoaNSScrollView setHasHorizontalScroller: YES];
-    [m_cocoaNSScrollView setHasVerticalScroller: YES];
+    // In all cases we must encapsulate the real NSView properly
     Encapsulate();
 }
 
@@ -432,6 +577,10 @@ void wxWindowCocoaScrollView::Unencapsulate()
 wxWindowCocoaScrollView::~wxWindowCocoaScrollView()
 {
     DisassociateNSView(m_cocoaNSScrollView);
+    if(!m_isNativeView)
+    {
+        [(WXManualScrollView*)m_cocoaNSScrollView _wx_setWxCocoaScrollView:NULL];
+    }
     [m_cocoaNSScrollView release];
 }
 
@@ -455,12 +604,322 @@ void wxWindowCocoaScrollView::DoGetClientSize(int *x, int *y) const
         *y = (int)nssize.height;
 }
 
+static inline void SetCocoaScroller(NSScroller *aScroller, int WXUNUSED(orientation), int position, int thumbSize, int range)
+{
+    wxCHECK_RET(aScroller != nil, wxT("Expected the NSScrollView to have a scroller"));
+
+    // NOTE: thumbSize is already ensured to be >= 1 and <= range by our caller
+    // unless range = 0 in which case we shouldn't have been be called.
+    wxCHECK_RET(range > 0, wxT("Internal wxCocoa bug: shouldn't have been called with 0 range"));
+
+    // Range of valid position values is from 0 to effectiveRange
+    // NOTE: if thumbSize == range then effectiveRange is 0.
+    // thumbSize is at least 1 which gives range from 0 to range - 1 inclusive
+    // which is exactly what we want.
+    int effectiveRange = range - thumbSize;
+
+    // knobProportion is hopefully easy to understand
+    // Note that thumbSize is already guaranteed >= 1 by our caller.
+    CGFloat const knobProportion = CGFloat(thumbSize)/CGFloat(range);
+
+    // NOTE: When effectiveRange is zero there really is no valid position
+    // We arbitrarily pick 0.0 which is the same as a scroller in the home position.
+    CGFloat const floatValue = (effectiveRange != 0)?CGFloat(position)/CGFloat(effectiveRange):0.0;
+
+    [aScroller setFloatValue:floatValue knobProportion: knobProportion];
+    // Make sure it's visibly working
+    [aScroller setEnabled:YES];
+}
+
+void wxWindowCocoaScrollView::SetScrollPos(wxOrientation orientation, int position)
+{
+    // NOTE: Rather than using only setFloatValue: (which we could do) we instead
+    // simply share the SetCocoaScroller call because all but the knobProportion
+    // calculations have to be done anyway.
+    if(orientation & wxHORIZONTAL)
+    {
+        NSScroller *aScroller = [m_cocoaNSScrollView horizontalScroller];
+        if(aScroller != nil)
+            SetCocoaScroller(aScroller, orientation, position, m_scrollThumb[0], m_scrollRange[0]);
+    }
+    if(orientation & wxVERTICAL)
+    {
+        NSScroller *aScroller = [m_cocoaNSScrollView verticalScroller];
+        if(aScroller != nil)
+            SetCocoaScroller(aScroller, orientation, position, m_scrollThumb[1], m_scrollRange[1]);
+    }
+}
+
+void wxWindowCocoaScrollView::SetScrollbar(int orientation, int position, int thumbSize, int range)
+{
+    // FIXME: API assumptions:
+    // 1. If the user wants to remove a scroller he gives range 0.
+    // 2. If the user wants to disable a scroller he sets thumbSize == range
+    //    in which case it is logically impossible to scroll.
+    //    The scroller shall still be displayed.
+
+    // Ensure that range is >= 0.
+    wxASSERT(range >= 0);
+    if(range < 0)
+        range = 0;
+
+    // Ensure that thumbSize <= range
+    wxASSERT(thumbSize <= range);
+    // Also ensure thumbSize >= 1 but don't complain if it isn't
+    if(thumbSize < 1)
+        thumbSize = 1;
+    // Now make sure it's really less than range, even if we just set it to 1
+    if(thumbSize > range)
+        thumbSize = range;
+
+    bool needScroller = (range != 0);
+
+    // Can potentially set both horizontal and vertical at the same time although this is
+    // probably not very useful.
+    if(orientation & wxHORIZONTAL)
+    {
+        m_scrollRange[0] = range;
+        m_scrollThumb[0] = thumbSize;
+        if(!m_isNativeView)
+        {
+            [m_cocoaNSScrollView setHasHorizontalScroller:needScroller];
+            if(needScroller)
+                SetCocoaScroller([m_cocoaNSScrollView horizontalScroller], orientation, position, thumbSize, range);
+        }
+    }
+
+    if(orientation & wxVERTICAL)
+    {
+        m_scrollRange[1] = range;
+        m_scrollThumb[1] = thumbSize;
+        if(!m_isNativeView)
+        {
+            [m_cocoaNSScrollView setHasVerticalScroller:needScroller];
+            if(needScroller)
+                SetCocoaScroller([m_cocoaNSScrollView verticalScroller], orientation, position, thumbSize, range);
+        }
+    }
+}
+
+int wxWindowCocoaScrollView::GetScrollPos(wxOrientation orient)
+{
+    if((orient & wxBOTH) == wxBOTH)
+    {
+        wxLogError(wxT("GetScrollPos called for wxHORIZONTAL and wxVERTICAL together which makes no sense"));
+        return 0;
+    }
+    int effectiveScrollRange;
+    NSScroller *cocoaScroller;
+    if(orient & wxHORIZONTAL)
+    {
+        effectiveScrollRange = m_scrollRange[0] - m_scrollThumb[0];
+        cocoaScroller = [m_cocoaNSScrollView horizontalScroller];
+    }
+    else if(orient & wxVERTICAL)
+    {
+        effectiveScrollRange = m_scrollRange[1] - m_scrollThumb[1];
+        cocoaScroller = [m_cocoaNSScrollView verticalScroller];
+    }
+    else
+    {
+        wxLogError(wxT("GetScrollPos called without an orientation which makes no sense"));
+        return 0;
+    }
+    if(cocoaScroller == nil)
+    {   // Document is not scrolled
+        return 0;
+    }
+    /*
+        The effective range of a scroll bar as defined by wxWidgets is from 0 to (range - thumbSize).
+        That is a scroller at the left/top position is at 0 and a scroller at the bottom/right
+        position is at range-thumbsize.
+
+        The range of an NSScroller is 0.0 to 1.0.  Much easier! NOTE: Apple doesn't really specify
+        but GNUStep docs do say that 0.0 is top/left and 1.0 is bottom/right.  This is actualy
+        in contrast to NSSlider which generally has 1.0 at the TOP when it's done vertically.
+     */
+    CGFloat cocoaScrollPos = [cocoaScroller floatValue];
+    return effectiveScrollRange * cocoaScrollPos;
+}
+
+int wxWindowCocoaScrollView::GetScrollRange(wxOrientation orient)
+{
+    if((orient & wxBOTH) == wxBOTH)
+    {
+        wxLogError(wxT("GetScrollRange called for wxHORIZONTAL and wxVERTICAL together which makes no sense"));
+        return 0;
+    }
+    if(orient & wxHORIZONTAL)
+    {
+        return m_scrollRange[0];
+    }
+    else if(orient & wxVERTICAL)
+    {
+        return m_scrollRange[1];
+    }
+    else
+    {
+        wxLogError(wxT("GetScrollPos called without an orientation which makes no sense"));
+        return 0;
+    }
+}
+
+int wxWindowCocoaScrollView::GetScrollThumb(wxOrientation orient)
+{
+    if((orient & wxBOTH) == wxBOTH)
+    {
+        wxLogError(wxT("GetScrollThumb called for wxHORIZONTAL and wxVERTICAL together which makes no sense"));
+        return 0;
+    }
+    if(orient & wxHORIZONTAL)
+    {
+        return m_scrollThumb[0];
+    }
+    else if(orient & wxVERTICAL)
+    {
+        return m_scrollThumb[1];
+    }
+    else
+    {
+        wxLogError(wxT("GetScrollThumb called without an orientation which makes no sense"));
+        return 0;
+    }
+}
+
+/*!
+    Moves the contents (all existing drawing as well as all all child wxWindow) by the specified
+    amount expressed in the wxWindow's own coordinate system.  This is used to implement scrolling
+    but the usage is rather interesting.  When scrolling right (e.g. increasing the value of
+    the scroller) you must give a negative delta x (e.g. moving the contents LEFT).  Likewise,
+    when scrolling the window down, increasing the value of the scroller, you give a negative
+    delta y which moves the contents up.
+
+    wxCocoa notes: To accomplish this trick in Cocoa we basically do what NSScrollView would
+    have done and that is adjust the content view's bounds origin.  The content view is somewhat
+    confusingly the NSClipView which is more or less sort of the pImpl for NSScrollView
+    The real NSView with the user's content (e.g. the "virtual area" in wxWidgets parlance)
+    is called the document view in NSScrollView parlance.
+
+    The bounds origin is basically the exact opposite concept.  Whereas in Windows the client
+    coordinate system remains constant and the content must shift left/up for increases
+    of scrolling, in Cocoa the coordinate system is actually the virtual one.  So we must
+    instead shift the bounds rectangle right/down to get the effect of the content moving
+    left/up.  Basically, it's a higher level interface than that provided by wxWidgets
+    so essentially we're implementing the low-level move content interface in terms of
+    the high-level move the viewport (the bounds) over top that content (the document
+    view which is the virtual area to wx).
+
+    For all intents and purposes that basically just means that we subtract the deltas
+    from the bounds origin and thus a negative delta actually increases the bounds origin
+    and a positive delta actually decreases it.  This is absolutely true for the horizontal
+    axis but there's a catch in the vertical axis.  If the content view (the clip view) is
+    flipped (and we do this by default) then it works exactly like the horizontal axis.
+    If it is not flipped (i.e. it is in postscript coordinates which are opposite to
+    wxWidgets) then the sense needs to be reversed.
+
+    However, this plays hell with window positions.  The frame rects of any child views
+    do not change origin and this is actually important because if they did, the views
+    would send frame changed notifications, not to mention that Cocoa just doesn't really
+    do scrolling that way, it does it the way we do it here.
+
+    To fix this we implement GetPosition for child windows to not merely consult its
+    superview at the Cocoa level in order to do proper Cocoa->wx coordinate transform
+    but to actually consult is parent wxWindow because it makes a big difference if
+    the parent is scrolled. Argh.  (FIXME: This isn't actually implemented yet)
+ */
+void wxWindowCocoaScrollView::ScrollWindow(int dx, int dy, const wxRect*)
+{
+    // Update our internal origin so we know how much the application code
+    // expects us to have been scrolled.
+    m_virtualOrigin.x += dx;
+    m_virtualOrigin.y += dy;
+
+    // Scroll the window using the standard Cocoa method of adjusting the
+    // clip view's bounds in the opposite fashion.
+    NSClipView *contentView = [m_cocoaNSScrollView contentView];
+    NSRect clipViewBoundsRect = [contentView bounds];
+    clipViewBoundsRect.origin.x -= dx;
+    if([contentView isFlipped])
+        clipViewBoundsRect.origin.y -= dy;
+    else
+        clipViewBoundsRect.origin.y += dy;
+    [contentView scrollToPoint:clipViewBoundsRect.origin];
+}
+
+void wxWindowCocoaScrollView::_wx_doScroller(NSScroller *sender)
+{
+    wxOrientation orientation;
+    if(sender == [m_cocoaNSScrollView horizontalScroller])
+        orientation = wxHORIZONTAL;
+    else if(sender == [m_cocoaNSScrollView verticalScroller])
+        orientation = wxVERTICAL;
+    else
+    {
+        wxLogDebug(wxT("Received action message from unexpected NSScroller"));
+        return;
+    }
+    // NOTE: Cocoa does not move the scroller for page up/down or line
+    // up/down events.  That means the value will be the old value.
+    // For thumbtrack events, the value is the new value.
+    int scrollpos = GetScrollPos(orientation);
+    int commandType;
+    switch([sender hitPart])
+    {
+    default:
+    case NSScrollerNoPart:
+    case NSScrollerKnob:        // Drag of knob
+    case NSScrollerKnobSlot:    // Jump directly to position
+        commandType = wxEVT_SCROLLWIN_THUMBTRACK;
+        break;
+    case NSScrollerDecrementPage:
+        commandType = wxEVT_SCROLLWIN_PAGEUP;
+        break;
+    case NSScrollerIncrementPage:
+        commandType = wxEVT_SCROLLWIN_PAGEDOWN;
+        break;
+    case NSScrollerDecrementLine:
+        commandType = wxEVT_SCROLLWIN_LINEUP;
+        break;
+    case NSScrollerIncrementLine:
+        commandType = wxEVT_SCROLLWIN_LINEDOWN;
+        break;
+    }
+    wxScrollWinEvent event(commandType, scrollpos, orientation);
+    event.SetEventObject(m_owner);
+    m_owner->HandleWindowEvent(event);
+}
+
+void wxWindowCocoaScrollView::UpdateSizes()
+{
+    // Using the virtual size, figure out what the document frame size should be
+    // NOTE: Assume that the passed in virtualSize is already >= the client size
+    wxSize virtualSize = m_owner->GetVirtualSize();
+
+    // Get the document's current frame
+    NSRect documentViewFrame = [m_owner->GetNSView() frame];
+    NSRect newFrame = documentViewFrame;
+    newFrame.size = NSMakeSize(virtualSize.x, virtualSize.y);
+
+    if(!NSEqualRects(newFrame, documentViewFrame))
+    {
+        [m_owner->GetNSView() setFrame:newFrame];
+    }
+}
+
 void wxWindowCocoaScrollView::Cocoa_FrameChanged(void)
 {
-    wxLogTrace(wxTRACE_COCOA,wxT("Cocoa_FrameChanged"));
+    wxLogTrace(wxTRACE_COCOA,wxT("wxWindowCocoaScrollView=%p::Cocoa_FrameChanged for wxWindow %p"), this, m_owner);
     wxSizeEvent event(m_owner->GetSize(), m_owner->GetId());
     event.SetEventObject(m_owner);
-    m_owner->GetEventHandler()->ProcessEvent(event);
+    m_owner->HandleWindowEvent(event);
+
+    /*  If the view is not a native one then it's being managed by wx.  In this case the control
+        may decide to change its virtual size and we must update the document view's size to
+        match.  For native views the virtual size will never have been set so we do not want
+        to use it at all.
+     */
+    if(!m_isNativeView)
+        UpdateSizes();
 }
 
 // ========================================================================
@@ -556,8 +1015,8 @@ void wxWindowCocoa::SetNSView(WX_NSView cocoaNSView)
     bool need_debug = cocoaNSView || m_cocoaNSView;
     if(need_debug) wxLogTrace(wxTRACE_COCOA_RetainRelease,wxT("wxWindowCocoa=%p::SetNSView [m_cocoaNSView=%p retainCount]=%d"),this,m_cocoaNSView,[m_cocoaNSView retainCount]);
     DisassociateNSView(m_cocoaNSView);
-    [cocoaNSView retain];
-    [m_cocoaNSView release];
+    wxGCSafeRetain(cocoaNSView);
+    wxGCSafeRelease(m_cocoaNSView);
     m_cocoaNSView = cocoaNSView;
     AssociateNSView(m_cocoaNSView);
     if(need_debug) wxLogTrace(wxTRACE_COCOA_RetainRelease,wxT("wxWindowCocoa=%p::SetNSView [cocoaNSView=%p retainCount]=%d"),this,cocoaNSView,[cocoaNSView retainCount]);
@@ -606,7 +1065,7 @@ NSRect wxWindowCocoa::CocoaTransformWxToBounds(NSRect rectWx)
 WX_NSAffineTransform wxWindowCocoa::CocoaGetWxToBoundsTransform()
 {
     // TODO: Handle scrolling offset
-    NSAffineTransform *transform = wxDC::CocoaGetWxToBoundsTransform([GetNSView() isFlipped], [GetNSView() bounds].size.height);
+    NSAffineTransform *transform = wxCocoaDCImpl::CocoaGetWxToBoundsTransform([GetNSView() isFlipped], [GetNSView() bounds].size.height);
     return transform;
 }
 
@@ -640,7 +1099,7 @@ bool wxWindowCocoa::Cocoa_drawRect(const NSRect &rect)
 
     wxPaintEvent event(m_windowId);
     event.SetEventObject(this);
-    bool ret = GetEventHandler()->ProcessEvent(event);
+    bool ret = HandleWindowEvent(event);
     m_isInPaint = false;
     return ret;
 }
@@ -651,6 +1110,44 @@ void wxWindowCocoa::InitMouseEvent(wxMouseEvent& event, WX_NSEvent cocoaEvent)
     // Mouse events happen at the NSWindow level so we need to convert
     // into our bounds coordinates then convert to wx coordinates.
     NSPoint cocoaPoint = [m_cocoaNSView convertPoint:[(NSEvent*)cocoaEvent locationInWindow] fromView:nil];
+    if( m_wxCocoaScrollView != NULL)
+    {
+        // This gets the wx client area (i.e. the visible portion of the content) in
+        // the coordinate system of our (the doucment) view.
+       NSRect documentVisibleRect = [[m_wxCocoaScrollView->GetNSScrollView() contentView] documentVisibleRect];
+        // For horizontal, simply subtract the origin.
+        // e.g. if the origin is at 123 and the user clicks as far left as possible then
+        // the coordinate that wx wants is 0.
+       cocoaPoint.x -= documentVisibleRect.origin.x;
+        if([m_cocoaNSView isFlipped])
+        {
+            // In the flipped view case this works exactly like horizontal.
+            cocoaPoint.y -= documentVisibleRect.origin.y;
+        }
+        // For vertical we have to mind non-flipped (e.g. y=0 at bottom) views.
+        // We also need to mind the fact that we're still in Cocoa coordinates
+        // and not wx coordinates.  The wx coordinate translation will still occur
+        // and that is going to be wxY = boundsH - cocoaY for non-flipped views.
+
+        // When we consider scrolling we are truly interested in how far the top
+        // edge of the bounds rectangle is scrolled off the screen.
+        // Assuming the bounds origin is 0 (which is an assumption we make in
+        // wxCocoa since wxWidgets has no analog to it) then the top edge of
+        // the bounds rectangle is simply its height.  The top edge of the
+        // documentVisibleRect (e.g. the client area) is its height plus
+        // its origin.
+        // Thus, we simply need add the distance between the bounds top
+        // and the client (docuemntVisibleRect) top.
+        // Or putting it another way, we subtract the distance between the
+        // client top and the bounds top.
+        else
+        {
+            NSRect bounds = [m_cocoaNSView bounds];
+            CGFloat scrollYOrigin = (bounds.size.height - (documentVisibleRect.origin.y + documentVisibleRect.size.height));
+            cocoaPoint.y += scrollYOrigin;
+        }
+    }
+
     NSPoint pointWx = CocoaTransformBoundsToWx(cocoaPoint);
     // FIXME: Should we be adjusting for client area origin?
     const wxPoint &clientorigin = GetClientAreaOrigin();
@@ -672,7 +1169,7 @@ bool wxWindowCocoa::Cocoa_mouseMoved(WX_NSEvent theEvent)
     wxMouseEvent event(wxEVT_MOTION);
     InitMouseEvent(event,theEvent);
     wxLogTrace(wxTRACE_COCOA,wxT("wxWindow=%p::Cocoa_mouseMoved @%d,%d"),this,event.m_x,event.m_y);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 void wxWindowCocoa::Cocoa_synthesizeMouseMoved()
@@ -693,7 +1190,7 @@ void wxWindowCocoa::Cocoa_synthesizeMouseMoved()
     event.SetId(GetId());
 
     wxLogTrace(wxTRACE_COCOA,wxT("wxwin=%p Synthesized Mouse Moved @%d,%d"),this,event.m_x,event.m_y);
-    GetEventHandler()->ProcessEvent(event);
+    HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_mouseEntered(WX_NSEvent theEvent)
@@ -711,7 +1208,7 @@ bool wxWindowCocoa::Cocoa_mouseEntered(WX_NSEvent theEvent)
         wxMouseEvent event(wxEVT_ENTER_WINDOW);
         InitMouseEvent(event,theEvent);
         wxLogTrace(wxTRACE_COCOA_TrackingRect,wxT("wxwin=%p Mouse Entered TR#%d @%d,%d"),this,[theEvent trackingNumber], event.m_x,event.m_y);
-        return GetEventHandler()->ProcessEvent(event);
+        return HandleWindowEvent(event);
     }
     else
         return false;
@@ -726,7 +1223,7 @@ bool wxWindowCocoa::Cocoa_mouseExited(WX_NSEvent theEvent)
         wxMouseEvent event(wxEVT_LEAVE_WINDOW);
         InitMouseEvent(event,theEvent);
         wxLogTrace(wxTRACE_COCOA_TrackingRect,wxT("wxwin=%p Mouse Exited TR#%d @%d,%d"),this,[theEvent trackingNumber],event.m_x,event.m_y);
-        return GetEventHandler()->ProcessEvent(event);
+        return HandleWindowEvent(event);
     }
     else
         return false;
@@ -737,7 +1234,7 @@ bool wxWindowCocoa::Cocoa_mouseDown(WX_NSEvent theEvent)
     wxMouseEvent event([theEvent clickCount]<2?wxEVT_LEFT_DOWN:wxEVT_LEFT_DCLICK);
     InitMouseEvent(event,theEvent);
     wxLogTrace(wxTRACE_COCOA,wxT("Mouse Down @%d,%d num clicks=%d"),event.m_x,event.m_y,[theEvent clickCount]);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_mouseDragged(WX_NSEvent theEvent)
@@ -746,7 +1243,7 @@ bool wxWindowCocoa::Cocoa_mouseDragged(WX_NSEvent theEvent)
     InitMouseEvent(event,theEvent);
     event.m_leftDown = true;
     wxLogTrace(wxTRACE_COCOA,wxT("Mouse Drag @%d,%d"),event.m_x,event.m_y);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_mouseUp(WX_NSEvent theEvent)
@@ -754,7 +1251,7 @@ bool wxWindowCocoa::Cocoa_mouseUp(WX_NSEvent theEvent)
     wxMouseEvent event(wxEVT_LEFT_UP);
     InitMouseEvent(event,theEvent);
     wxLogTrace(wxTRACE_COCOA,wxT("Mouse Up @%d,%d"),event.m_x,event.m_y);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseDown(WX_NSEvent theEvent)
@@ -762,7 +1259,7 @@ bool wxWindowCocoa::Cocoa_rightMouseDown(WX_NSEvent theEvent)
     wxMouseEvent event([theEvent clickCount]<2?wxEVT_RIGHT_DOWN:wxEVT_RIGHT_DCLICK);
     InitMouseEvent(event,theEvent);
     wxLogDebug(wxT("Mouse Down @%d,%d num clicks=%d"),event.m_x,event.m_y,[theEvent clickCount]);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseDragged(WX_NSEvent theEvent)
@@ -771,7 +1268,7 @@ bool wxWindowCocoa::Cocoa_rightMouseDragged(WX_NSEvent theEvent)
     InitMouseEvent(event,theEvent);
     event.m_rightDown = true;
     wxLogDebug(wxT("Mouse Drag @%d,%d"),event.m_x,event.m_y);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseUp(WX_NSEvent theEvent)
@@ -779,7 +1276,7 @@ bool wxWindowCocoa::Cocoa_rightMouseUp(WX_NSEvent theEvent)
     wxMouseEvent event(wxEVT_RIGHT_UP);
     InitMouseEvent(event,theEvent);
     wxLogDebug(wxT("Mouse Up @%d,%d"),event.m_x,event.m_y);
-    return GetEventHandler()->ProcessEvent(event);
+    return HandleWindowEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_otherMouseDown(WX_NSEvent theEvent)
@@ -799,12 +1296,25 @@ bool wxWindowCocoa::Cocoa_otherMouseUp(WX_NSEvent theEvent)
 
 void wxWindowCocoa::Cocoa_FrameChanged(void)
 {
-    wxLogTrace(wxTRACE_COCOA,wxT("wxWindow=%p::Cocoa_FrameChanged"),this);
-    if(m_visibleTrackingRectManager != NULL)
-        m_visibleTrackingRectManager->RebuildTrackingRect();
-    wxSizeEvent event(GetSize(), m_windowId);
-    event.SetEventObject(this);
-    GetEventHandler()->ProcessEvent(event);
+    // We always get this message for the real NSView which may have been
+    // enclosed in an NSScrollView.  If that's the case then what we're
+    // effectively getting here is a notifcation that the
+    // virtual sized changed.. which we don't need to send on since
+    // wx has no concept of this whatsoever.
+    bool isViewForSuperview = (m_wxCocoaScrollView == NULL);
+    if(isViewForSuperview)
+    {
+        wxLogTrace(wxTRACE_COCOA,wxT("wxWindow=%p::Cocoa_FrameChanged"),this);
+        if(m_visibleTrackingRectManager != NULL)
+            m_visibleTrackingRectManager->RebuildTrackingRect();
+        wxSizeEvent event(GetSize(), m_windowId);
+        event.SetEventObject(this);
+        HandleWindowEvent(event);
+    }
+    else
+    {
+        wxLogTrace(wxTRACE_COCOA,wxT("wxWindow=%p::Cocoa_FrameChanged ignored"),this);
+    }
 }
 
 bool wxWindowCocoa::Cocoa_resetCursorRects()
@@ -902,17 +1412,48 @@ bool wxWindow::Show(bool show)
         // If state isn't changing, return false
         if(!m_cocoaHider)
             return false;
+
+        // Replace the stand-in view with the real one and destroy the dummy view
         CocoaReplaceView(m_cocoaHider->GetNSView(), cocoaView);
         wxASSERT(![m_cocoaHider->GetNSView() superview]);
         delete m_cocoaHider;
         m_cocoaHider = NULL;
         wxASSERT([cocoaView superview]);
+
+        // Schedule an update of the key view loop (NOTE: 10.4+ only.. argh)
+        NSWindow *window = [cocoaView window];
+        if(window != nil)
+        {
+            // Delay this until returning to the event loop for a couple of reasons:
+            // 1. If a lot of stuff is shown/hidden we avoid recalculating needlessly
+            // 2. NSWindow does not seem to see the newly shown views if we do it right now.
+            if([window respondsToSelector:@selector(recalculateKeyViewLoop)])
+                [window performSelector:@selector(recalculateKeyViewLoop) withObject:nil afterDelay:0.0];
+        }
     }
     else
     {
         // If state isn't changing, return false
         if(m_cocoaHider)
             return false;
+
+        // Handle the first responder
+        NSWindow *window = [cocoaView window];
+        if(window != nil)
+        {
+            NSResponder *firstResponder = [window firstResponder];
+            if([firstResponder isKindOfClass:[NSView class]] && [(NSView*)firstResponder isDescendantOf:cocoaView])
+            {
+                BOOL didResign = [window makeFirstResponder:nil];
+                // If the current first responder (one of our subviews) refuses to give
+                // up its status, then fail now and don't hide this view
+                if(didResign == NO)
+                    return false;
+            }
+        }
+
+        // Create a new view to stand in for the real one (via wxWindowCocoaHider) and replace
+        // the real one with the stand in.
         m_cocoaHider = new wxWindowCocoaHider(this);
         // NOTE: replaceSubview:with will cause m_cocaNSView to be
         // (auto)released which balances out addSubview
@@ -1180,7 +1721,7 @@ void wxWindow::GetTextExtent(const wxString& string, int *outX, int *outY,
     // transformations.  However, it's better than nothing.
     // We don't create a wxClientDC because we don't want to accidently be able to use
     // it for drawing.
-    wxDC tmpdc;
+    wxClientDC tmpdc(const_cast<wxWindow*>(this));
     return tmpdc.GetTextExtent(string, outX, outY, outDescent, outExternalLeading, inFont);
 }
 
@@ -1192,27 +1733,34 @@ void wxWindow::WarpPointer (int x_pos, int y_pos)
 
 int wxWindow::GetScrollPos(int orient) const
 {
-    // TODO
-    return 0;
+    if(m_wxCocoaScrollView != NULL)
+        return m_wxCocoaScrollView->GetScrollPos(static_cast<wxOrientation>(orient & wxBOTH));
+    else
+        return 0;
 }
 
 // This now returns the whole range, not just the number
 // of positions that we can scroll.
 int wxWindow::GetScrollRange(int orient) const
 {
-    // TODO
-    return 0;
+    if(m_wxCocoaScrollView != NULL)
+        return m_wxCocoaScrollView->GetScrollRange(static_cast<wxOrientation>(orient & wxBOTH));
+    else
+        return 0;
 }
 
 int wxWindow::GetScrollThumb(int orient) const
 {
-    // TODO
-    return 0;
+    if(m_wxCocoaScrollView != NULL)
+        return m_wxCocoaScrollView->GetScrollThumb(static_cast<wxOrientation>(orient & wxBOTH));
+    else
+        return 0;
 }
 
 void wxWindow::SetScrollPos(int orient, int pos, bool refresh)
 {
-    // TODO
+    if(m_wxCocoaScrollView != NULL)
+        return m_wxCocoaScrollView->SetScrollPos(static_cast<wxOrientation>(orient & wxBOTH), pos);
 }
 
 void wxWindow::CocoaCreateNSScrollView()
@@ -1228,38 +1776,15 @@ void wxWindow::SetScrollbar(int orient, int pos, int thumbVisible,
     int range, bool refresh)
 {
     CocoaCreateNSScrollView();
-    // TODO
+    m_wxCocoaScrollView->SetScrollbar(orient, pos, thumbVisible, range);
+    // TODO: Handle refresh (if we even need to)
 }
 
 // Does a physical scroll
 void wxWindow::ScrollWindow(int dx, int dy, const wxRect *rect)
 {
-    // TODO
-}
-
-static inline int _DoFixupDistance(int vDistance, int cDistance)
-{
-    // If the virtual distance is wxDefaultCoord, set it to the client distance
-    // This definitely has to be done or else we literally get views with a -1 size component!
-    if(vDistance == wxDefaultCoord)
-        vDistance = cDistance;
-    // NOTE: Since cDistance should always be >= 0 and since wxDefaultCoord is -1, the above
-    // test is more or less useless because it gets covered by the next one.  However, just in
-    // case anyone decides that the next test is not correct, I want them to be aware that
-    // the above test would still be needed.
-
-    // I am not entirely sure about this next one but I believe it makes sense because
-    // otherwise the virtual view (which is the m_cocoaNSView that got wrapped by the scrolling
-    // machinery) can be smaller than the NSClipView (the client area) which
-    // means that, for instance, mouse clicks inside the client area as wx sees it but outside
-    // the virtual area as wx sees it won't be seen by the m_cocoaNSView.
-    // We make the assumption that if a virtual distance is less than the client distance that
-    // the real view must already be or will soon be positioned at coordinate 0  within the
-    // NSClipView that represents the client area.  This way, when we increase the distance to
-    // be the client distance, the real view will exactly fit in the clip view.
-    else if(vDistance < cDistance)
-        vDistance = cDistance;
-    return vDistance;
+    if(m_wxCocoaScrollView != NULL)
+        m_wxCocoaScrollView->ScrollWindow(dx, dy, rect);
 }
 
 void wxWindow::DoSetVirtualSize( int x, int y )
@@ -1267,14 +1792,14 @@ void wxWindow::DoSetVirtualSize( int x, int y )
     // Call wxWindowBase method which will set m_virtualSize to the appropriate value,
     // possibly not what the caller passed in.  For example, the current implementation
     // clamps the width and height to within the min/max virtual ranges.
-    // wxDefaultCoord is passed through unchanged which means we need to handle it ourselves
-    // which we do by using the _DoFixupDistance helper method.
+    // wxDefaultCoord is passed through unchanged but then GetVirtualSize() will correct
+    // that by returning effectively max(virtual, client)
     wxWindowBase::DoSetVirtualSize(x,y);
     // Create the scroll view if it hasn't been already.
     CocoaCreateNSScrollView();
-    // Now use fixed-up distances when setting the frame size
-    wxSize clientSize = GetClientSize();
-    [m_cocoaNSView setFrameSize:NSMakeSize(_DoFixupDistance(m_virtualSize.x, clientSize.x), _DoFixupDistance(m_virtualSize.y, clientSize.y))];
+
+    // The GetVirtualSize automatically increases the size to max(client,virtual)
+    m_wxCocoaScrollView->UpdateSizes();
 }
 
 bool wxWindow::SetFont(const wxFont& font)