BEGIN_EVENT_TABLE(wxTopLevelWindowBase, wxWindow)
EVT_CLOSE(wxTopLevelWindowBase::OnCloseWindow)
EVT_SIZE(wxTopLevelWindowBase::OnSize)
- EVT_WINDOW_DESTROY(wxTopLevelWindowBase::OnChildDestroy)
- WX_EVENT_TABLE_CONTROL_CONTAINER(wxTopLevelWindowBase)
END_EVENT_TABLE()
-WX_DELEGATE_TO_CONTROL_CONTAINER(wxTopLevelWindowBase, wxWindow)
-
// ============================================================================
// implementation
// ============================================================================
{
// Unlike windows, top level windows are created hidden by default.
m_isShown = false;
- m_winDefault = NULL;
- m_winTmpDefault = NULL;
-
- WX_INIT_CONTROL_CONTAINER();
}
wxTopLevelWindowBase::~wxTopLevelWindowBase()
{
- m_winDefault = m_winTmpDefault = NULL;
-
// don't let wxTheApp keep any stale pointers to us
if ( wxTheApp && wxTheApp->GetTopWindow() == this )
wxTheApp->SetTopWindow(NULL);
wxTopLevelWindows.DeleteObject(this);
+ // delete any our top level children which are still pending for deletion
+ // immediately: this could happen if a child (e.g. a temporary dialog
+ // created with this window as parent) was Destroy()'d) while this window
+ // was deleted directly (with delete, or maybe just because it was created
+ // on the stack) immediately afterwards and before the child TLW was really
+ // destroyed -- not destroying it now would leave it alive with a dangling
+ // parent pointer and result in a crash later
+ for ( wxObjectList::iterator i = wxPendingDelete.begin();
+ i != wxPendingDelete.end();
+ )
+ {
+ wxWindow * const win = wxDynamicCast(*i, wxWindow);
+ if ( win && win->GetParent() == this )
+ {
+ wxPendingDelete.erase(i);
+
+ delete win;
+
+ // deleting it invalidated the list (and not only one node because
+ // it could have resulted in deletion of other objects to)
+ i = wxPendingDelete.begin();
+ }
+ else
+ {
+ ++i;
+ }
+ }
+
if ( IsLastBeforeExit() )
{
// no other (important) windows left, quit the app
if ( !wxPendingDelete.Member(this) )
wxPendingDelete.Append(this);
- if (wxTopLevelWindows.GetCount() > 1)
+ // normally we want to hide the window immediately so that it doesn't get
+ // stuck on the screen while it's being destroyed, however we shouldn't
+ // hide the last visible window as then we might not get any idle events
+ // any more as no events will be sent to the hidden window and without idle
+ // events we won't prune wxPendingDelete list and the application won't
+ // terminate
+ for ( wxWindowList::const_iterator i = wxTopLevelWindows.begin(),
+ end = wxTopLevelWindows.end();
+ i != end;
+ ++i )
{
- // Hide it immediately. This should
- // not be done if this TLW is the
- // only one left since we then would
- // risk not to get any idle events
- // at all anymore during which we
- // could delete any pending events.
- Hide();
+ wxTopLevelWindow * const win = static_cast<wxTopLevelWindow *>(*i);
+ if ( win != this && win->IsShown() )
+ {
+ // there remains at least one other visible TLW, we can hide this
+ // one
+ Hide();
+
+ break;
+ }
}
return true;
// then decide whether we should exit at all
for ( i = wxTopLevelWindows.begin(); i != end; ++i )
{
- wxTopLevelWindow * const win = wx_static_cast(wxTopLevelWindow *, *i);
+ wxTopLevelWindow * const win = static_cast<wxTopLevelWindow *>(*i);
if ( win->ShouldPreventAppExit() )
{
// there remains at least one important TLW, don't exit
for ( i = wxTopLevelWindows.begin(); i != end; ++i )
{
// don't close twice the windows which are already marked for deletion
- wxTopLevelWindow * const win = wx_static_cast(wxTopLevelWindow *, *i);
+ wxTopLevelWindow * const win = static_cast<wxTopLevelWindow *>(*i);
if ( !wxPendingDelete.Member(win) && !win->Close() )
{
// one of the windows refused to close, don't exit
wxSize wxTopLevelWindowBase::GetDefaultSize()
{
wxSize size = wxGetClientDisplayRect().GetSize();
-
+#ifndef __WXOSX_IPHONE__
// create proportionally bigger windows on small screens
if ( size.x >= 1024 )
size.x = 400;
size.y *= 2;
size.y /= 3;
}
-
+#endif
return size;
}
{
// on some platforms centering top level windows is impossible
// because they are always maximized by guidelines or limitations
- if(IsAlwaysMaximized())
+ //
+ // and centering a maximized window doesn't make sense as its position
+ // can't change
+ if ( IsAlwaysMaximized() || IsMaximized() )
return;
// we need the display rect anyhow so store it first: notice that we should
// parent frame under Mac but could happen elsewhere too if the frame
// was hidden/moved away for some reason), don't use it as otherwise
// this window wouldn't be visible at all
- if ( !rectDisplay.Contains(rectParent.GetTopLeft()) &&
- !rectParent.Contains(rectParent.GetBottomRight()) )
+ if ( !rectParent.Intersects(rectDisplay) )
{
- // this is enough to make IsEmpty() test below pass
- rectParent.width = 0;
+ // just centre on screen then
+ rectParent = rectDisplay;
}
}
-
- if ( rectParent.IsEmpty() )
+ else
{
// we were explicitly asked to centre this window on the entire screen
// or if we have no parent anyhow and so can't centre on it
rectParent = rectDisplay;
}
- // centering maximized window on screen is no-op
- if((rectParent == rectDisplay) && IsMaximized())
- return;
+ if ( !(dir & wxBOTH) )
+ dir |= wxBOTH; // if neither is specified, center in both directions
// the new window rect candidate
- wxRect rect = GetRect().CentreIn(rectParent, dir);
+ wxRect rect = GetRect().CentreIn(rectParent, dir & ~wxCENTRE_ON_SCREEN);
// we don't want to place the window off screen if Centre() is called as
// this is (almost?) never wanted and it would be very difficult to prevent
// passing wxNullIcon to SetIcon() is possible (it means that we shouldn't
// have any icon), but adding an invalid icon to wxIconBundle is not
wxIconBundle icons;
- if ( icon.Ok() )
+ if ( icon.IsOk() )
icons.AddIcon(icon);
SetIcons(icons);
// whole client area
void wxTopLevelWindowBase::DoLayout()
{
+ // We are called during the window destruction several times, e.g. as
+ // wxFrame tries to adjust to its tool/status bars disappearing. But
+ // actually doing the layout is pretty useless in this case as the window
+ // will disappear anyhow -- so just don't bother.
+ if ( IsBeingDeleted() )
+ return;
+
+
// if we're using constraints or sizers - do use them
if ( GetAutoLayout() )
{
else
{
// do we have _exactly_ one child?
- wxWindow *child = (wxWindow *)NULL;
+ wxWindow *child = NULL;
for ( wxWindowList::compatibility_iterator node = GetChildren().GetFirst();
node;
node = node->GetNext() )
Destroy();
}
-void wxTopLevelWindowBase::OnChildDestroy(wxWindowDestroyEvent& event)
-{
- event.Skip();
-
- wxWindow * const win = event.GetWindow();
- if ( win == m_winDefault )
- m_winDefault = NULL;
- if ( win == m_winTmpDefault )
- m_winTmpDefault = NULL;
-}
-
bool wxTopLevelWindowBase::SendIconizeEvent(bool iconized)
{
wxIconizeEvent event(GetId(), iconized);
projects / wxWidgets.git / blobdiff