Don't (indirectly) call virtual funcs in ctor.

We're not fully constructed at that point.

Change-Id: Ie80874c57000481c4eaa8d71118c8be6f0059164

sd/source/ui/framework/factories/BasicViewFactory.cxx

@@ -368,10 +368,22 @@ void SAL_CALL BasicViewFactory::initialize (const Sequence<Any>& aArguments)
         pDescriptor->mpViewShell->Init(bIsCenterPane);
         mpBase->GetViewShellManager()->ActivateViewShell(pDescriptor->mpViewShell.get());
 
+        Reference<awt::XWindow> xWindow(rxPane->getWindow());
         pDescriptor->mpWrapper = new ViewShellWrapper(
             pDescriptor->mpViewShell,
             rxViewId,
-            rxPane->getWindow());
+            xWindow);
+
+        // register ViewShellWrapper on pane window
+        if (xWindow.is())
+        {
+            xWindow->addWindowListener(pDescriptor->mpWrapper);
+            if (pDescriptor->mpViewShell != NULL)
+            {
+                pDescriptor->mpViewShell->Resize();
+            }
+        }
+
         pDescriptor->mxView.set( pDescriptor->mpWrapper->queryInterface( XResource::static_type() ), UNO_QUERY_THROW );
     }
 

sd/source/ui/framework/factories/ViewShellWrapper.cxx

@@ -67,14 +67,6 @@ ViewShellWrapper::ViewShellWrapper (
       mxViewId(rxViewId),
       mxWindow(rxWindow)
 {
-    if (rxWindow.is())
-    {
-        rxWindow->addWindowListener(this);
-        if (pViewShell != NULL)
-        {
-            pViewShell->Resize();
-        }
-    }
 }