sfx2: fix sfx2::sidebar::Theme lifecycle

It's a horrible idea to have global VCL Images, since that will
inevitably crash on shutdown when the static dtor runs after
DeInitVCL, which breaks ~every JunitTest now.

0x00002af6750d2d51 in rtl::Reference<sfx2::sidebar::Theme>::~Reference (this=0x2af6756ceb28 <sfx2::sidebar::Theme::mpInstance>, __in_chrg=<optimized out>)

Try to fix that by moving the global sidebar::Theme instance to
SfxApplication where it can hopefully be deleted before shutdown.

Change-Id: Ia78f1e458699335b53a741b6463ce48af69584a7

4 files changed, 21 insertions, 16 deletions

diff --git a/sfx2/source/appl/app.cxx b/sfx2/source/appl/app.cxx
index c66884246d9c..fb84391c9841 100644
--- a/sfx2/source/appl/app.cxx
+++ b/sfx2/source/appl/app.cxx
@@ -88,6 +88,7 @@
 #include <sfx2/event.hxx>
 #include "imestatuswindow.hxx"
 #include "workwin.hxx"
+#include <sfx2/sidebar/Theme.hxx>
 #include <sfx2/tbxctrl.hxx>
 #include <sfx2/sfxdlg.hxx>
 #include "sfx2/stbitem.hxx"
@@ -636,4 +637,14 @@ ErrCode SfxApplication::CallBasic( const OUString& rCode, BasicManager* pMgr, Sb
 #endif
 }
 
+sfx2::sidebar::Theme & SfxApplication::GetSidebarTheme()
+{
+    if (!pAppData_Impl->m_pSidebarTheme.is())
+    {
+        pAppData_Impl->m_pSidebarTheme.set(new sfx2::sidebar::Theme);
+        pAppData_Impl->m_pSidebarTheme->InitializeTheme();
+    }
+    return *pAppData_Impl->m_pSidebarTheme;
+}
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sfx2/source/appl/appdata.cxx b/sfx2/source/appl/appdata.cxx
index a2c79ca4e5c5..d031fbcfe70c 100644
--- a/sfx2/source/appl/appdata.cxx
+++ b/sfx2/source/appl/appdata.cxx
@@ -38,6 +38,7 @@
 #include <sfx2/docfac.hxx>
 #include <sfx2/docfile.hxx>
 #include <sfx2/request.hxx>
+#include <sfx2/sidebar/Theme.hxx>
 #include "referers.hxx"
 #include "app.hrc"
 #include "sfx2/sfxresid.hxx"
diff --git a/sfx2/source/inc/appdata.hxx b/sfx2/source/inc/appdata.hxx
index 00bf7b44e9dd..cdd66809ae18 100644
--- a/sfx2/source/inc/appdata.hxx
+++ b/sfx2/source/inc/appdata.hxx
@@ -62,7 +62,10 @@ class BasicManager;
 class SfxBasicManagerHolder;
 class SfxBasicManagerCreationListener;
 
-namespace sfx2 { namespace appl { class ImeStatusWindow; } }
+namespace sfx2 {
+    namespace appl { class ImeStatusWindow; }
+    namespace sidebar { class Theme; }
+}
 
 typedef Link* LinkPtr;
 
@@ -130,6 +133,7 @@ public:
     SfxSlotPool*                pSlotPool;
     SfxDispatcher*              pAppDispat;     // Dispatcher if no document
     SfxInterface**              pInterfaces;
+    ::rtl::Reference<sfx2::sidebar::Theme> m_pSidebarTheme;
 
     sal_uInt16                      nDocNo;             // current Doc-Number (AutoName)
     sal_uInt16                      nInterfaces;
diff --git a/sfx2/source/sidebar/Theme.cxx b/sfx2/source/sidebar/Theme.cxx
index d6d95d78833e..bd191bbf4db9 100644
--- a/sfx2/source/sidebar/Theme.cxx
+++ b/sfx2/source/sidebar/Theme.cxx
@@ -20,6 +20,7 @@
 #include "Paint.hxx"
 #include "SidebarResource.hxx"
 #include "sfx2/sidebar/Tools.hxx"
+#include <sfx2/app.hxx>
 
 #include <tools/svborder.hxx>
 #include <tools/rc.hxx>
@@ -31,25 +32,13 @@ using namespace cssu;
 
 namespace sfx2 { namespace sidebar {
 
-::rtl::Reference<Theme> Theme::mpInstance;
 
-
-
-
-Theme& Theme::GetCurrentTheme (void)
+Theme& Theme::GetCurrentTheme()
 {
-    if ( ! mpInstance.is())
-    {
-        mpInstance.set(new Theme());
-        mpInstance->InitializeTheme();
-    }
-    return *mpInstance;
+    return SFX_APP()->GetSidebarTheme();
 }
 
-
-
-
-Theme::Theme (void)
+Theme::Theme()
     : ThemeInterfaceBase(m_aMutex),
       maImages(),
       maColors(),