byte length for interim UniChar String corrected

[wxWidgets.git] / src / cocoa / window.mm

diff --git a/src/cocoa/window.mm b/src/cocoa/window.mm
index b255cb12123716157cbb52c73e8c7b8c923b0b50..91e48b8579dc1ef5fea0cad49850e7ae863b1f64 100644 (file)
--- a/src/cocoa/window.mm
+++ b/src/cocoa/window.mm
@@ -9,13 +9,26 @@
 // Licence:    wxWindows license
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:    wxWindows license
 /////////////////////////////////////////////////////////////////////////////
 

-#include "wx/window.h"
-#include "wx/log.h"
+#include "wx/wxprec.h"
+#ifndef WX_PRECOMP
+    #include "wx/log.h"
+    #include "wx/window.h"
+#endif //WX_PRECOMP

 
 #include "wx/cocoa/autorelease.h"
 
 
 #include "wx/cocoa/autorelease.h"
 

-#import <Appkit/NSView.h>
+#import <AppKit/NSView.h>

 #import <AppKit/NSEvent.h>
 #import <AppKit/NSEvent.h>

+#import <AppKit/NSScrollView.h>
+#import <AppKit/NSColor.h>
+#import <AppKit/NSClipView.h>
+
+// Turn this on to paint green over the dummy views for debugging
+#undef WXCOCOA_FILL_DUMMY_VIEW
+
+#ifdef WXCOCOA_FILL_DUMMY_VIEW
+#import <AppKit/NSBezierPath.h>
+#endif //def WXCOCOA_FILL_DUMMY_VIEW

 
 // ========================================================================
 // wxWindowCocoaHider
 
 // ========================================================================
 // wxWindowCocoaHider


@@ -31,10 +44,50 @@ protected:
     wxWindowCocoa *m_owner;
     WX_NSView m_dummyNSView;
     virtual void Cocoa_FrameChanged(void);
     wxWindowCocoa *m_owner;
     WX_NSView m_dummyNSView;
     virtual void Cocoa_FrameChanged(void);

+#ifdef WXCOCOA_FILL_DUMMY_VIEW
+    virtual bool Cocoa_drawRect(const NSRect& rect);
+#endif //def WXCOCOA_FILL_DUMMY_VIEW

 private:
     wxWindowCocoaHider();
 };
 
 private:
     wxWindowCocoaHider();
 };
 

+// ========================================================================
+// wxWindowCocoaScroller
+// ========================================================================
+class wxWindowCocoaScroller: protected wxCocoaNSView
+{
+    DECLARE_NO_COPY_CLASS(wxWindowCocoaScroller)
+public:
+    wxWindowCocoaScroller(wxWindow *owner);
+    virtual ~wxWindowCocoaScroller();
+    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);
+private:
+    wxWindowCocoaScroller();
+};
+
+// ========================================================================
+// wxDummyNSView
+// ========================================================================
+@interface wxDummyNSView : NSView
+- (NSView *)hitTest:(NSPoint)aPoint;
+@end
+
+@implementation wxDummyNSView : NSView
+- (NSView *)hitTest:(NSPoint)aPoint
+{
+    return nil;
+}
+
+@end
+

 // ========================================================================
 // wxWindowCocoaHider
 // ========================================================================
 // ========================================================================
 // wxWindowCocoaHider
 // ========================================================================


@@ -43,8 +96,9 @@ wxWindowCocoaHider::wxWindowCocoaHider(wxWindow *owner)
 {
     wxASSERT(owner);
     wxASSERT(owner->GetNSViewForHiding());
 {
     wxASSERT(owner);
     wxASSERT(owner->GetNSViewForHiding());

-    m_dummyNSView = [[NSView alloc]
+    m_dummyNSView = [[wxDummyNSView alloc]

         initWithFrame:[owner->GetNSViewForHiding() frame]];
         initWithFrame:[owner->GetNSViewForHiding() frame]];

+    [m_dummyNSView setAutoresizingMask: [owner->GetNSViewForHiding() autoresizingMask]];

     AssociateNSView(m_dummyNSView);
 }
 
     AssociateNSView(m_dummyNSView);
 }
 


@@ -61,6 +115,113 @@ void wxWindowCocoaHider::Cocoa_FrameChanged(void)
     [m_owner->GetNSViewForHiding() setFrame:[m_dummyNSView frame]];
 }
 
     [m_owner->GetNSViewForHiding() setFrame:[m_dummyNSView frame]];
 }
 

+#ifdef WXCOCOA_FILL_DUMMY_VIEW
+bool wxWindowCocoaHider::Cocoa_drawRect(const NSRect& rect)
+{
+    NSBezierPath *bezpath = [NSBezierPath bezierPathWithRect:rect];
+    [[NSColor greenColor] set];
+    [bezpath stroke];
+    [bezpath fill];
+    return true;
+}
+#endif //def WXCOCOA_FILL_DUMMY_VIEW
+
+// ========================================================================
+// wxFlippedNSClipView
+// ========================================================================
+@interface wxFlippedNSClipView : NSClipView
+- (BOOL)isFlipped;
+@end
+
+@implementation wxFlippedNSClipView : NSClipView
+- (BOOL)isFlipped
+{
+    return YES;
+}
+
+@end
+
+// ========================================================================
+// wxWindowCocoaScroller
+// ========================================================================
+wxWindowCocoaScroller::wxWindowCocoaScroller(wxWindow *owner)
+:   m_owner(owner)
+{
+    wxASSERT(owner);
+    wxASSERT(owner->GetNSView());
+    m_cocoaNSScrollView = [[NSScrollView alloc]
+        initWithFrame:[owner->GetNSView() frame]];
+    AssociateNSView(m_cocoaNSScrollView);
+
+    /* Replace the default NSClipView with a flipped one.  This ensures
+       scrolling is "pinned" to the top-left instead of bottom-right. */
+    NSClipView *flippedClip = [[wxFlippedNSClipView alloc]
+        initWithFrame: [[m_cocoaNSScrollView contentView] frame]];
+    [m_cocoaNSScrollView setContentView:flippedClip];
+    [flippedClip release];
+
+    [m_cocoaNSScrollView setBackgroundColor: [NSColor windowBackgroundColor]];
+    [m_cocoaNSScrollView setHasHorizontalScroller: YES];
+    [m_cocoaNSScrollView setHasVerticalScroller: YES];
+    Encapsulate();
+}
+
+void wxWindowCocoaScroller::Encapsulate()
+{
+    // Set the scroll view autoresizingMask to match the current NSView
+    [m_cocoaNSScrollView setAutoresizingMask: [m_owner->GetNSView() autoresizingMask]];
+    [m_owner->GetNSView() setAutoresizingMask: NSViewNotSizable];
+    // NOTE: replaceSubView will cause m_cocaNSView to be released
+    // except when it hasn't been added into an NSView hierarchy in which
+    // case it doesn't need to be and this should work out to a no-op
+    m_owner->CocoaReplaceView(m_owner->GetNSView(), m_cocoaNSScrollView);
+    // The NSView is still retained by owner
+    [m_cocoaNSScrollView setDocumentView: m_owner->GetNSView()];
+    // Now it's also retained by the NSScrollView
+}
+
+void wxWindowCocoaScroller::Unencapsulate()
+{
+    [m_cocoaNSScrollView setDocumentView: nil];
+    m_owner->CocoaReplaceView(m_cocoaNSScrollView, m_owner->GetNSView());
+    if(![[m_owner->GetNSView() superview] isFlipped])
+        [m_owner->GetNSView() setAutoresizingMask: NSViewMinYMargin];
+}
+
+wxWindowCocoaScroller::~wxWindowCocoaScroller()
+{
+    DisassociateNSView(m_cocoaNSScrollView);
+    [m_cocoaNSScrollView release];
+}
+
+void wxWindowCocoaScroller::ClientSizeToSize(int &width, int &height)
+{
+    NSSize frameSize = [NSScrollView
+        frameSizeForContentSize: NSMakeSize(width,height)
+        hasHorizontalScroller: [m_cocoaNSScrollView hasHorizontalScroller]
+        hasVerticalScroller: [m_cocoaNSScrollView hasVerticalScroller]
+        borderType: [m_cocoaNSScrollView borderType]];
+    width = (int)frameSize.width;
+    height = (int)frameSize.height;
+}
+
+void wxWindowCocoaScroller::DoGetClientSize(int *x, int *y) const
+{
+    NSSize nssize = [m_cocoaNSScrollView contentSize];
+    if(x)
+        *x = (int)nssize.width;
+    if(y)
+        *y = (int)nssize.height;
+}
+
+void wxWindowCocoaScroller::Cocoa_FrameChanged(void)
+{
+    wxLogTrace(wxTRACE_COCOA,wxT("Cocoa_FrameChanged"));
+    wxSizeEvent event(m_owner->GetSize(), m_owner->GetId());
+    event.SetEventObject(m_owner);
+    m_owner->GetEventHandler()->ProcessEvent(event);
+}
+

 // ========================================================================
 // wxWindowCocoa
 // ========================================================================
 // ========================================================================
 // wxWindowCocoa
 // ========================================================================


@@ -79,12 +240,12 @@ wxWindow *wxWindowCocoa::sm_capturedWindow = NULL;
 // Constructor
 void wxWindowCocoa::Init()
 {
 // Constructor
 void wxWindowCocoa::Init()
 {

-    InitBase();
-

     m_cocoaNSView = NULL;
     m_cocoaHider = NULL;
     m_cocoaNSView = NULL;
     m_cocoaHider = NULL;

+    m_cocoaScroller = NULL;

     m_isBeingDeleted = FALSE;
     m_isInPaint = FALSE;
     m_isBeingDeleted = FALSE;
     m_isInPaint = FALSE;

+    m_shouldBeEnabled = true;

 }
 
 // Constructor
 }
 
 // Constructor


@@ -98,15 +259,15 @@ bool wxWindow::Create(wxWindow *parent, wxWindowID winid,
         return false;
 
     // TODO: create the window
         return false;
 
     // TODO: create the window

-    NSRect cocoaRect = NSMakeRect(10,10,20,20);

     m_cocoaNSView = NULL;
     m_cocoaNSView = NULL;

-    SetNSView([[NSView alloc] initWithFrame: cocoaRect]);
+    SetNSView([[NSView alloc] initWithFrame: MakeDefaultNSRect(size)]);

     [m_cocoaNSView release];
 
     if (m_parent)
     {
         m_parent->AddChild(this);
         m_parent->CocoaAddChild(this);
     [m_cocoaNSView release];
 
     if (m_parent)
     {
         m_parent->AddChild(this);
         m_parent->CocoaAddChild(this);

+        SetInitialFrameRect(pos,size);

     }
 
     return TRUE;
     }
 
     return TRUE;


@@ -118,11 +279,12 @@ wxWindow::~wxWindow()
     wxAutoNSAutoreleasePool pool;
     DestroyChildren();
 
     wxAutoNSAutoreleasePool pool;
     DestroyChildren();
 

-    if(m_parent)
-        m_parent->RemoveChild(this);
-
-    CocoaRemoveFromParent();
+    // Make sure our parent (in the wxWindows sense) is our superview
+    // before we go removing from it.
+    if(m_parent && m_parent->GetNSView()==[GetNSViewForSuperview() superview])
+        CocoaRemoveFromParent();

     delete m_cocoaHider;
     delete m_cocoaHider;

+    delete m_cocoaScroller;

     SetNSView(NULL);
 }
 
     SetNSView(NULL);
 }
 


@@ -142,37 +304,45 @@ void wxWindowCocoa::CocoaRemoveFromParent(void)
 
 void wxWindowCocoa::SetNSView(WX_NSView cocoaNSView)
 {
 
 void wxWindowCocoa::SetNSView(WX_NSView cocoaNSView)
 {

+    // Assume setting the NSView to NULL means this wxWindow is being destroyed
+    if(m_cocoaNSView && !cocoaNSView)
+        SendDestroyEvent();
+

     bool need_debug = cocoaNSView || m_cocoaNSView;
     bool need_debug = cocoaNSView || m_cocoaNSView;

-    if(need_debug) wxLogDebug("wxWindowCocoa=%p::SetNSView [m_cocoaNSView=%p retainCount]=%d",this,m_cocoaNSView,[m_cocoaNSView retainCount]);
+    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];
     m_cocoaNSView = cocoaNSView;
     AssociateNSView(m_cocoaNSView);
     DisassociateNSView(m_cocoaNSView);
     [cocoaNSView retain];
     [m_cocoaNSView release];
     m_cocoaNSView = cocoaNSView;
     AssociateNSView(m_cocoaNSView);

-    if(need_debug) wxLogDebug("wxWindowCocoa=%p::SetNSView [cocoaNSView=%p retainCount]=%d",this,cocoaNSView,[cocoaNSView retainCount]);
+    if(need_debug) wxLogTrace(wxTRACE_COCOA_RetainRelease,wxT("wxWindowCocoa=%p::SetNSView [cocoaNSView=%p retainCount]=%d"),this,cocoaNSView,[cocoaNSView retainCount]);

 }
 
 WX_NSView wxWindowCocoa::GetNSViewForSuperview() const
 {
     return m_cocoaHider
         ?   m_cocoaHider->GetNSView()
 }
 
 WX_NSView wxWindowCocoa::GetNSViewForSuperview() const
 {
     return m_cocoaHider
         ?   m_cocoaHider->GetNSView()

-        :   m_cocoaNSView;
+        :   m_cocoaScroller
+            ?   m_cocoaScroller->GetNSScrollView()
+            :   m_cocoaNSView;

 }
 
 WX_NSView wxWindowCocoa::GetNSViewForHiding() const
 {
 }
 
 WX_NSView wxWindowCocoa::GetNSViewForHiding() const
 {

-    return m_cocoaNSView;
+    return m_cocoaScroller
+        ?   m_cocoaScroller->GetNSScrollView()
+        :   m_cocoaNSView;

 }
 
 bool wxWindowCocoa::Cocoa_drawRect(const NSRect &rect)
 {
 }
 
 bool wxWindowCocoa::Cocoa_drawRect(const NSRect &rect)
 {

-    wxLogDebug("Cocoa_drawRect");
+    wxLogTrace(wxTRACE_COCOA,wxT("Cocoa_drawRect"));

     // Recursion can happen if the event loop runs from within the paint
     // handler.  For instance, if an assertion dialog is shown.
     // FIXME: This seems less than ideal.
     if(m_isInPaint)
     {
     // Recursion can happen if the event loop runs from within the paint
     // handler.  For instance, if an assertion dialog is shown.
     // FIXME: This seems less than ideal.
     if(m_isInPaint)
     {

-        wxLogDebug("Paint event recursion!");
+        wxLogDebug(wxT("Paint event recursion!"));

         return false;
     }
     //FIXME: should probably turn that rect into the update region
         return false;
     }
     //FIXME: should probably turn that rect into the update region


@@ -186,7 +356,7 @@ bool wxWindowCocoa::Cocoa_drawRect(const NSRect &rect)
 
 void wxWindowCocoa::InitMouseEvent(wxMouseEvent& event, WX_NSEvent cocoaEvent)
 {
 
 void wxWindowCocoa::InitMouseEvent(wxMouseEvent& event, WX_NSEvent cocoaEvent)
 {

-    wxASSERT_MSG([m_cocoaNSView window]==[cocoaEvent window],"Mouse event for different NSWindow");
+    wxASSERT_MSG([m_cocoaNSView window]==[cocoaEvent window],wxT("Mouse event for different NSWindow"));

     NSPoint cocoaPoint = [m_cocoaNSView convertPoint:[(NSEvent*)cocoaEvent locationInWindow] fromView:nil];
     NSRect cocoaRect = [m_cocoaNSView frame];
     const wxPoint &clientorigin = GetClientAreaOrigin();
     NSPoint cocoaPoint = [m_cocoaNSView convertPoint:[(NSEvent*)cocoaEvent locationInWindow] fromView:nil];
     NSRect cocoaRect = [m_cocoaNSView frame];
     const wxPoint &clientorigin = GetClientAreaOrigin();


@@ -207,7 +377,7 @@ bool wxWindowCocoa::Cocoa_mouseMoved(WX_NSEvent theEvent)
 {
     wxMouseEvent event(wxEVT_MOTION);
     InitMouseEvent(event,theEvent);
 {
     wxMouseEvent event(wxEVT_MOTION);
     InitMouseEvent(event,theEvent);

-    wxLogDebug("Mouse Drag @%d,%d",event.m_x,event.m_y);
+    wxLogTrace(wxTRACE_COCOA,wxT("Mouse Drag @%d,%d"),event.m_x,event.m_y);

     return GetEventHandler()->ProcessEvent(event);
 }
 
     return GetEventHandler()->ProcessEvent(event);
 }
 


@@ -225,7 +395,7 @@ bool wxWindowCocoa::Cocoa_mouseDown(WX_NSEvent theEvent)
 {
     wxMouseEvent event([theEvent clickCount]<2?wxEVT_LEFT_DOWN:wxEVT_LEFT_DCLICK);
     InitMouseEvent(event,theEvent);
 {
     wxMouseEvent event([theEvent clickCount]<2?wxEVT_LEFT_DOWN:wxEVT_LEFT_DCLICK);
     InitMouseEvent(event,theEvent);

-    wxLogDebug("Mouse Down @%d,%d num clicks=%d",event.m_x,event.m_y,[theEvent clickCount]);
+    wxLogTrace(wxTRACE_COCOA,wxT("Mouse Down @%d,%d num clicks=%d"),event.m_x,event.m_y,[theEvent clickCount]);

     return GetEventHandler()->ProcessEvent(event);
 }
 
     return GetEventHandler()->ProcessEvent(event);
 }
 


@@ -234,7 +404,7 @@ bool wxWindowCocoa::Cocoa_mouseDragged(WX_NSEvent theEvent)
     wxMouseEvent event(wxEVT_MOTION);
     InitMouseEvent(event,theEvent);
     event.m_leftDown = true;
     wxMouseEvent event(wxEVT_MOTION);
     InitMouseEvent(event,theEvent);
     event.m_leftDown = true;

-    wxLogDebug("Mouse Drag @%d,%d",event.m_x,event.m_y);
+    wxLogTrace(wxTRACE_COCOA,wxT("Mouse Drag @%d,%d"),event.m_x,event.m_y);

     return GetEventHandler()->ProcessEvent(event);
 }
 
     return GetEventHandler()->ProcessEvent(event);
 }
 


@@ -242,23 +412,33 @@ bool wxWindowCocoa::Cocoa_mouseUp(WX_NSEvent theEvent)
 {
     wxMouseEvent event(wxEVT_LEFT_UP);
     InitMouseEvent(event,theEvent);
 {
     wxMouseEvent event(wxEVT_LEFT_UP);
     InitMouseEvent(event,theEvent);

-    wxLogDebug("Mouse Up @%d,%d",event.m_x,event.m_y);
+    wxLogTrace(wxTRACE_COCOA,wxT("Mouse Up @%d,%d"),event.m_x,event.m_y);

     return GetEventHandler()->ProcessEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseDown(WX_NSEvent theEvent)
 {
     return GetEventHandler()->ProcessEvent(event);
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseDown(WX_NSEvent theEvent)
 {

-    return false;
+    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);

 }
 
 bool wxWindowCocoa::Cocoa_rightMouseDragged(WX_NSEvent theEvent)
 {
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseDragged(WX_NSEvent theEvent)
 {

-    return false;
+    wxMouseEvent event(wxEVT_MOTION);
+    InitMouseEvent(event,theEvent);
+    event.m_rightDown = true;
+    wxLogDebug(wxT("Mouse Drag @%d,%d"),event.m_x,event.m_y);
+    return GetEventHandler()->ProcessEvent(event);

 }
 
 bool wxWindowCocoa::Cocoa_rightMouseUp(WX_NSEvent theEvent)
 {
 }
 
 bool wxWindowCocoa::Cocoa_rightMouseUp(WX_NSEvent theEvent)
 {

-    return false;
+    wxMouseEvent event(wxEVT_RIGHT_UP);
+    InitMouseEvent(event,theEvent);
+    wxLogDebug(wxT("Mouse Up @%d,%d"),event.m_x,event.m_y);
+    return GetEventHandler()->ProcessEvent(event);

 }
 
 bool wxWindowCocoa::Cocoa_otherMouseDown(WX_NSEvent theEvent)
 }
 
 bool wxWindowCocoa::Cocoa_otherMouseDown(WX_NSEvent theEvent)


@@ -278,7 +458,7 @@ bool wxWindowCocoa::Cocoa_otherMouseUp(WX_NSEvent theEvent)
 
 void wxWindowCocoa::Cocoa_FrameChanged(void)
 {
 
 void wxWindowCocoa::Cocoa_FrameChanged(void)
 {

-    wxLogDebug("Cocoa_FrameChanged");
+    wxLogTrace(wxTRACE_COCOA,wxT("Cocoa_FrameChanged"));

     wxSizeEvent event(GetSize(), m_windowId);
     event.SetEventObject(this);
     GetEventHandler()->ProcessEvent(event);
     wxSizeEvent event(GetSize(), m_windowId);
     event.SetEventObject(this);
     GetEventHandler()->ProcessEvent(event);


@@ -286,7 +466,9 @@ void wxWindowCocoa::Cocoa_FrameChanged(void)
 
 bool wxWindow::Close(bool force)
 {
 
 bool wxWindow::Close(bool force)
 {

-    return false;
+    // The only reason this function exists is that it is virtual and
+    // wxTopLevelWindowCocoa will override it.
+    return wxWindowBase::Close(force);

 }
 
 void wxWindow::CocoaReplaceView(WX_NSView oldView, WX_NSView newView)
 }
 
 void wxWindow::CocoaReplaceView(WX_NSView oldView, WX_NSView newView)


@@ -294,13 +476,41 @@ void wxWindow::CocoaReplaceView(WX_NSView oldView, WX_NSView newView)
     [[oldView superview] replaceSubview:oldView with:newView];
 }
 
     [[oldView superview] replaceSubview:oldView with:newView];
 }
 

+bool wxWindow::EnableSelfAndChildren(bool enable)
+{
+    // If the state isn't changing, don't do anything
+    if(!wxWindowBase::Enable(enable && m_shouldBeEnabled))
+        return false;
+    // Set the state of the Cocoa window
+    CocoaSetEnabled(m_isEnabled);
+    // Disable all children or (if enabling) return them to their proper state
+    for(wxWindowList::compatibility_iterator node = GetChildren().GetFirst();
+        node; node = node->GetNext())
+    {
+        node->GetData()->EnableSelfAndChildren(enable);
+    }
+    return true;
+}
+
+bool wxWindow::Enable(bool enable)
+{
+    // Keep track of what the window SHOULD be doing
+    m_shouldBeEnabled = enable;
+    // If the parent is disabled for any reason, then this window will be too.
+    if(!IsTopLevel() && GetParent())
+    {
+        enable = enable && GetParent()->IsEnabled();
+    }
+    return EnableSelfAndChildren(enable);
+}
+

 bool wxWindow::Show(bool show)
 {
     wxAutoNSAutoreleasePool pool;
     // If the window is marked as visible, then it shouldn't have a dummy view
     // If the window is marked hidden, then it should have a dummy view
     // wxSpinCtrl (generic) abuses m_isShown, don't use it for any logic
 bool wxWindow::Show(bool show)
 {
     wxAutoNSAutoreleasePool pool;
     // If the window is marked as visible, then it shouldn't have a dummy view
     // If the window is marked hidden, then it should have a dummy view
     // wxSpinCtrl (generic) abuses m_isShown, don't use it for any logic

-//    wxASSERT_MSG( (m_isShown && !m_dummyNSView) || (!m_isShown && m_dummyNSView),"wxWindow: m_isShown does not agree with m_dummyNSView");
+//    wxASSERT_MSG( (m_isShown && !m_dummyNSView) || (!m_isShown && m_dummyNSView),wxT("wxWindow: m_isShown does not agree with m_dummyNSView"));

     // Return false if there isn't a window to show or hide
     NSView *cocoaView = GetNSViewForHiding();
     if(!cocoaView)
     // Return false if there isn't a window to show or hide
     NSView *cocoaView = GetNSViewForHiding();
     if(!cocoaView)


@@ -335,7 +545,7 @@ bool wxWindow::Show(bool show)
 
 void wxWindowCocoa::DoSetSize(int x, int y, int width, int height, int sizeFlags)
 {
 
 void wxWindowCocoa::DoSetSize(int x, int y, int width, int height, int sizeFlags)
 {

-//    wxLogDebug("wxWindow=%p::DoSetSizeWindow(%d,%d,%d,%d,Auto: %s%s)",this,x,y,width,height,(sizeFlags&wxSIZE_AUTO_WIDTH)?"W":".",sizeFlags&wxSIZE_AUTO_HEIGHT?"H":".");
+    wxLogTrace(wxTRACE_COCOA_Window_Size,"wxWindow=%p::DoSetSizeWindow(%d,%d,%d,%d,Auto: %s%s)",this,x,y,width,height,(sizeFlags&wxSIZE_AUTO_WIDTH)?"W":".",sizeFlags&wxSIZE_AUTO_HEIGHT?"H":".");

     int currentX, currentY;
     int currentW, currentH;
     DoGetPosition(&currentX, &currentY);
     int currentX, currentY;
     int currentW, currentH;
     DoGetPosition(&currentX, &currentY);


@@ -375,15 +585,42 @@ void wxWindowCocoa::DoSetSize(int x, int y, int width, int height, int sizeFlags
 
 void wxWindowCocoa::DoMoveWindow(int x, int y, int width, int height)
 {
 
 void wxWindowCocoa::DoMoveWindow(int x, int y, int width, int height)
 {

-//    wxLogDebug("wxWindow=%p::DoMoveWindow(%d,%d,%d,%d)",this,x,y,width,height);
+    wxAutoNSAutoreleasePool pool;
+    wxLogTrace(wxTRACE_COCOA_Window_Size,"wxWindow=%p::DoMoveWindow(%d,%d,%d,%d)",this,x,y,width,height);

 
     NSView *nsview = GetNSViewForSuperview();
     NSView *superview = [nsview superview];
 
     NSView *nsview = GetNSViewForSuperview();
     NSView *superview = [nsview superview];

-    wxCHECK_RET(superview,"NSView does not have a superview");
-    NSRect parentRect = [superview frame];
+    wxCHECK_RET(superview,wxT("NSView does not have a superview"));
+    NSRect parentRect = [superview bounds];

 
     NSRect cocoaRect = NSMakeRect(x,parentRect.size.height-(y+height),width,height);
     [nsview setFrame: cocoaRect];
 
     NSRect cocoaRect = NSMakeRect(x,parentRect.size.height-(y+height),width,height);
     [nsview setFrame: cocoaRect];

+    // Be sure to redraw the parent to reflect the changed position
+    [superview setNeedsDisplay:YES];
+}
+
+void wxWindowCocoa::SetInitialFrameRect(const wxPoint& pos, const wxSize& size)
+{
+    NSView *nsview = GetNSViewForSuperview();
+    NSView *superview = [nsview superview];
+    wxCHECK_RET(superview,wxT("NSView does not have a superview"));
+    NSRect parentRect = [superview bounds];
+    NSRect frameRect = [nsview frame];
+    if(size.x!=-1)
+        frameRect.size.width = size.x;
+    if(size.y!=-1)
+        frameRect.size.height = size.y;
+    frameRect.origin.x = pos.x;
+    frameRect.origin.y = parentRect.size.height-(pos.y+frameRect.size.height);
+    // Tell Cocoa to change the margin between the bottom of the superview
+    // and the bottom of the control.  Keeps the control pinned to the top
+    // of its superview so that its position in the wxWindows coordinate
+    // system doesn't change.
+    if(![superview isFlipped])
+        [nsview setAutoresizingMask: NSViewMinYMargin];
+    // MUST set the mask before setFrame: which can generate a size event
+    // and cause a scroller to be added!
+    [nsview setFrame: frameRect];

 }
 
 // Get total size
 }
 
 // Get total size


@@ -394,22 +631,22 @@ void wxWindow::DoGetSize(int *w, int *h) const
         *w=(int)cocoaRect.size.width;
     if(h)
         *h=(int)cocoaRect.size.height;
         *w=(int)cocoaRect.size.width;
     if(h)
         *h=(int)cocoaRect.size.height;

-//    wxLogDebug("wxWindow=%p::DoGetSize = (%d,%d)",this,(int)cocoaRect.size.width,(int)cocoaRect.size.height);
+    wxLogTrace(wxTRACE_COCOA_Window_Size,"wxWindow=%p::DoGetSize = (%d,%d)",this,(int)cocoaRect.size.width,(int)cocoaRect.size.height);

 }
 
 void wxWindow::DoGetPosition(int *x, int *y) const
 {
     NSView *nsview = GetNSViewForSuperview();
     NSView *superview = [nsview superview];
 }
 
 void wxWindow::DoGetPosition(int *x, int *y) const
 {
     NSView *nsview = GetNSViewForSuperview();
     NSView *superview = [nsview superview];

-    wxCHECK_RET(superview,"NSView does not have a superview");
-    NSRect parentRect = [superview frame];
+    wxCHECK_RET(superview,wxT("NSView does not have a superview"));
+    NSRect parentRect = [superview bounds];

 
     NSRect cocoaRect = [nsview frame];
     if(x)
         *x=(int)cocoaRect.origin.x;
     if(y)
         *y=(int)(parentRect.size.height-(cocoaRect.origin.y+cocoaRect.size.height));
 
     NSRect cocoaRect = [nsview frame];
     if(x)
         *x=(int)cocoaRect.origin.x;
     if(y)
         *y=(int)(parentRect.size.height-(cocoaRect.origin.y+cocoaRect.size.height));

-//    wxLogDebug("wxWindow=%p::DoGetPosition = (%d,%d)",this,(int)cocoaRect.origin.x,(int)cocoaRect.origin.y);
+    wxLogTrace(wxTRACE_COCOA_Window_Size,"wxWindow=%p::DoGetPosition = (%d,%d)",this,(int)cocoaRect.origin.x,(int)cocoaRect.origin.y);

 }
 
 WXWidget wxWindow::GetHandle() const
 }
 
 WXWidget wxWindow::GetHandle() const


@@ -452,15 +689,24 @@ void wxWindow::DoClientToScreen(int *x, int *y) const
 // Get size *available for subwindows* i.e. excluding menu bar etc.
 void wxWindow::DoGetClientSize(int *x, int *y) const
 {
 // Get size *available for subwindows* i.e. excluding menu bar etc.
 void wxWindow::DoGetClientSize(int *x, int *y) const
 {

-    wxLogDebug("DoGetClientSize:");
-    wxWindowCocoa::DoGetSize(x,y);
-    // TODO: Actually account for menubar, borders, etc...
+    wxLogTrace(wxTRACE_COCOA,wxT("DoGetClientSize:"));
+    if(m_cocoaScroller)
+        m_cocoaScroller->DoGetClientSize(x,y);
+    else
+        wxWindowCocoa::DoGetSize(x,y);

 }
 
 void wxWindow::DoSetClientSize(int width, int height)
 {
 }
 
 void wxWindow::DoSetClientSize(int width, int height)
 {

-    wxLogDebug("DoSetClientSize=(%d,%d)",width,height);
-    // TODO
+    wxLogTrace(wxTRACE_COCOA_Window_Size,wxT("DoSetClientSize=(%d,%d)"),width,height);
+    if(m_cocoaScroller)
+        m_cocoaScroller->ClientSizeToSize(width,height);
+    CocoaSetWxWindowSize(width,height);
+}
+
+void wxWindow::CocoaSetWxWindowSize(int width, int height)
+{
+    wxWindowCocoa::DoSetSize(-1,-1,width,height,wxSIZE_USE_EXISTING);

 }
 
 int wxWindow::GetCharHeight() const
 }
 
 int wxWindow::GetCharHeight() const


@@ -512,10 +758,19 @@ void wxWindow::SetScrollPos(int orient, int pos, bool refresh)
     // TODO
 }
 
     // TODO
 }
 

+void wxWindow::CocoaCreateNSScrollView()
+{
+    if(!m_cocoaScroller)
+    {
+        m_cocoaScroller = new wxWindowCocoaScroller(this);
+    }
+}
+

 // New function that will replace some of the above.
 void wxWindow::SetScrollbar(int orient, int pos, int thumbVisible,
     int range, bool refresh)
 {
 // New function that will replace some of the above.
 void wxWindow::SetScrollbar(int orient, int pos, int thumbVisible,
     int range, bool refresh)
 {

+    CocoaCreateNSScrollView();

     // TODO
 }
 
     // TODO
 }
 


@@ -525,6 +780,13 @@ void wxWindow::ScrollWindow(int dx, int dy, const wxRect *rect)
     // TODO
 }
 
     // TODO
 }
 

+void wxWindow::DoSetVirtualSize( int x, int y )
+{
+    wxWindowBase::DoSetVirtualSize(x,y);
+    CocoaCreateNSScrollView();
+    [m_cocoaNSView setFrameSize:NSMakeSize(m_virtualSize.x,m_virtualSize.y)];
+}
+

 bool wxWindow::SetFont(const wxFont& font)
 {
     // TODO
 bool wxWindow::SetFont(const wxFont& font)
 {
     // TODO