From 058d7af6e9b471dc9b2a60bedd60ee9c8f682936 Mon Sep 17 00:00:00 2001
From: David Elliott <dfe@tgwbd.org>
Date: Sat, 10 Apr 2004 03:18:05 +0000
Subject: [PATCH] 1. Faster DoGetBestSize for single-cell controls. 2. Properly
 handle exceptions so controls which aren't NSControl don't crash.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@26687 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 src/cocoa/control.mm | 44 ++++++++++++++++++++++++++++++++++++++------
 1 file changed, 38 insertions(+), 6 deletions(-)

diff --git a/src/cocoa/control.mm b/src/cocoa/control.mm
index d43d8a1112..fe1ccaa7b5 100644
--- a/src/cocoa/control.mm
+++ b/src/cocoa/control.mm
@@ -19,6 +19,10 @@
 #include "wx/cocoa/autorelease.h"
 
 #import <AppKit/NSControl.h>
+#import <AppKit/NSCell.h>
+#import <Foundation/NSException.h>
+
+#include <math.h>
 
 @interface wxNonControlNSControl : NSControl
 {
@@ -72,14 +76,42 @@ wxControl::~wxControl()
 wxSize wxControl::DoGetBestSize() const
 {
     wxAutoNSAutoreleasePool pool;
-    wxASSERT(m_cocoaNSView);
+    wxASSERT(GetNSControl());
+    /* We can ask single-celled controls for their cell and get its size */
+    NSCell *cell = nil;
+NS_DURING
+    cell = [GetNSControl() cell];
+NS_HANDLER
+    // TODO: if anything other than method not implemented, re-raise
+NS_ENDHANDLER
+    if(cell)
+    {
+        NSSize cellSize = [cell cellSize];
+        wxSize size((int)ceilf(cellSize.width),(int)ceilf(cellSize.height));
+        wxLogTrace(wxTRACE_COCOA_Window_Size,wxT("wxControl=%p::DoGetBestSize()==(%d,%d) from NSCell"),this,size.x,size.y);
+        return size;
+    }
+
+    /* multi-celled control? size to fit, get the size, then set it back */
     NSRect storedRect = [m_cocoaNSView frame];
+    bool didFit = false;
+NS_DURING
     [GetNSControl() sizeToFit];
-    NSRect cocoaRect = [m_cocoaNSView frame];
-    wxSize size((int)cocoaRect.size.width+10,(int)cocoaRect.size.height);
-    [m_cocoaNSView setFrame: storedRect];
-    wxLogTrace(wxTRACE_COCOA_Window_Size,wxT("wxControl=%p::DoGetBestSize()==(%d,%d)"),this,size.x,size.y);
-    return size;
+    didFit = true;
+NS_HANDLER
+    // TODO: if anything other than method not implemented, re-raise
+NS_ENDHANDLER
+    if(didFit)
+    {
+        NSRect cocoaRect = [m_cocoaNSView frame];
+        wxSize size((int)ceilf(cocoaRect.size.width),(int)ceilf(cocoaRect.size.height));
+        [m_cocoaNSView setFrame: storedRect];
+        wxLogTrace(wxTRACE_COCOA_Window_Size,wxT("wxControl=%p::DoGetBestSize()==(%d,%d) from sizeToFit"),this,size.x,size.y);
+        return size;
+    }
+    // Cocoa can't tell us the size, probably not an NSControl.
+    wxLogDebug(wxT("Class %s (or superclass still below wxControl) should implement DoGetBestSize()"),GetClassInfo()->GetClassName());
+    return wxControlBase::DoGetBestSize();
 }
 
 bool wxControl::ProcessCommand(wxCommandEvent& event)
-- 
2.47.2