Reverted to my own solution; I think it's better because it distinguishes

between Universal and non-Universal solutions where menubars are
built in and don't affect client origin


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@14397 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/univ/framuniv.cpp

@@ -195,8 +195,7 @@ wxPoint wxFrame::GetClientAreaOrigin() const
     }
 #endif // wxUSE_MENUS
 
-   // This is done in wxFrameBase already
-#if 0 // wxUSE_TOOLBAR
+#if wxUSE_TOOLBAR
     if ( m_frameToolBar )
     {
         if ( m_frameToolBar->GetWindowStyleFlag() & wxTB_VERTICAL )

src/univ/toolbar.cpp

@@ -264,8 +264,10 @@ bool wxToolBar::Realize()
     {
         SetSize(sz.x, m_maxHeight);
     }
-    
-    SetSize( x+16, m_defaultHeight + 14 );
+
+    // Commenting out Robert's fix since the above should be
+    // more general
+    //    SetSize( x+16, m_defaultHeight + 14 );
     
     return TRUE;
 }