From e56334255291a0ccb5165d7ceaa1391b906bb4de Mon Sep 17 00:00:00 2001
From: Armin Le Grand <Armin.Le.Grand@cib.de>
Date: Thu, 21 Jul 2016 17:05:32 +0200
Subject: screenshots: fix fallback cases for sd and sc

Change-Id: I27dab5552e3924df30c8b09c46994b2d645bcde9
---
 sc/qa/unit/screenshots/data/screenshots.txt |  5 +++--
 sd/qa/unit/data/dialogs-test.txt            | 19 ++++++++++---------
 test/source/screenshot_test.cxx             | 24 ++++++++++++++++--------
 3 files changed, 29 insertions(+), 19 deletions(-)

diff --git a/sc/qa/unit/screenshots/data/screenshots.txt b/sc/qa/unit/screenshots/data/screenshots.txt
index 1fa34d7952d1..40a362915d27 100644
--- a/sc/qa/unit/screenshots/data/screenshots.txt
+++ b/sc/qa/unit/screenshots/data/screenshots.txt
@@ -41,6 +41,9 @@ modules/scalc/ui/formatcellsdialog.ui
 # be visualized using a fallback based on VclBuilder
 #
 
+# modules/scalc/ui/headerfootercontent.ui <- deactivated, leads to problems and the test to not work
+# modules/scalc/ui/pivottablelayoutdialog.ui <- deactivated, leads to problems and the test to not work
+
 modules/scalc/ui/autoformattable.ui
 modules/scalc/ui/sortwarning.ui
 modules/scalc/ui/condformatmanager.ui
@@ -60,7 +63,6 @@ modules/scalc/ui/groupbynumber.ui
 modules/scalc/ui/groupbydate.ui
 modules/scalc/ui/sortdialog.ui
 modules/scalc/ui/standardfilterdialog.ui
-modules/scalc/ui/pivottablelayoutdialog.ui
 modules/scalc/ui/validationdialog.ui
 modules/scalc/ui/subtotaldialog.ui
 modules/scalc/ui/scenariodialog.ui
@@ -144,7 +146,6 @@ modules/scalc/ui/rightfooterdialog.ui
 modules/scalc/ui/sharedheaderdialog.ui
 modules/scalc/ui/sharedfooterdialog.ui
 modules/scalc/ui/allheaderfooterdialog.ui
-modules/scalc/ui/headerfootercontent.ui
 modules/scalc/ui/sheetprintpage.ui
 modules/scalc/ui/sidebaralignment.ui
 modules/scalc/ui/sidebarcellappearance.ui
diff --git a/sd/qa/unit/data/dialogs-test.txt b/sd/qa/unit/data/dialogs-test.txt
index 2b7ad1acf71f..81fc5f08c07c 100644
--- a/sd/qa/unit/data/dialogs-test.txt
+++ b/sd/qa/unit/data/dialogs-test.txt
@@ -20,26 +20,27 @@
 # The known dialogs which have a hard-coded representation
 #
 
-modules/simpress/ui/publishingdialog.ui
 modules/sdraw/ui/breakdialog.ui
 modules/sdraw/ui/copydlg.ui
-modules/simpress/ui/customslideshows.ui
 modules/sdraw/ui/drawchardialog.ui
 modules/sdraw/ui/drawpagedialog.ui
-modules/simpress/ui/dlgfield.ui
 modules/sdraw/ui/dlgsnap.ui
 modules/sdraw/ui/insertlayer.ui
 modules/sdraw/ui/insertslidesdialog.ui
 modules/sdraw/ui/crossfadedialog.ui
 modules/sdraw/ui/bulletsandnumbering.ui
 modules/sdraw/ui/drawparadialog.ui
+modules/sdraw/ui/drawprtldialog.ui
+modules/sdraw/ui/vectorize.ui
+
+modules/simpress/ui/publishingdialog.ui
+modules/simpress/ui/customslideshows.ui
+modules/simpress/ui/dlgfield.ui
 modules/simpress/ui/presentationdialog.ui
 modules/simpress/ui/remotedialog.ui
 modules/simpress/ui/templatedialog.ui
 modules/simpress/ui/slidedesigndialog.ui
-modules/sdraw/ui/drawprtldialog.ui
 modules/simpress/ui/interactiondialog.ui
-modules/sdraw/ui/vectorize.ui
 modules/simpress/ui/photoalbum.ui
 modules/simpress/ui/masterlayoutdlg.ui
 modules/simpress/ui/headerfooterdialog.ui
@@ -49,18 +50,18 @@ modules/simpress/ui/headerfooterdialog.ui
 # be visualized using a fallback based on VclBuilder
 #
 
+modules/sdraw/ui/paranumberingtab.ui
+modules/sdraw/ui/namedesign.ui
+modules/sdraw/ui/tabledesigndialog.ui
+# modules/simpress/ui/assistentdialog.ui <- deactivated, leads to problems and the test to not work
 modules/simpress/ui/headerfootertab.ui
 modules/simpress/ui/customanimationproperties.ui
 modules/simpress/ui/customanimationeffecttab.ui
 modules/simpress/ui/customanimationtimingtab.ui
 modules/simpress/ui/customanimationtexttab.ui
-modules/sdraw/ui/paranumberingtab.ui
 modules/simpress/ui/sdviewpage.ui
 modules/simpress/ui/prntopts.ui
 modules/simpress/ui/optimpressgeneralpage.ui
 modules/simpress/ui/interactionpage.ui
-modules/simpress/ui/assistentdialog.ui
-modules/sdraw/ui/namedesign.ui
 modules/simpress/ui/definecustomslideshow.ui
 modules/simpress/ui/insertslides.ui
-modules/sdraw/ui/tabledesigndialog.ui
diff --git a/test/source/screenshot_test.cxx b/test/source/screenshot_test.cxx
index 92f48ee28823..dfbd2cee257c 100644
--- a/test/source/screenshot_test.cxx
+++ b/test/source/screenshot_test.cxx
@@ -178,16 +178,24 @@ void ScreenshotTest::dumpDialogToPath(const OString& rUIXMLDescription)
     if (!rUIXMLDescription.isEmpty())
     {
         VclPtrInstance<Dialog> pDialog(Application::GetDefDialogParent(), WB_STDDIALOG | WB_SIZEABLE, Dialog::InitFlag::NoParent);
-        VclBuilder aBuilder(pDialog, VclBuilderContainer::getUIRootDir(), OStringToOUString(rUIXMLDescription, RTL_TEXTENCODING_UTF8));
-        vcl::Window *pRoot = aBuilder.get_widget_root();
-        Dialog *pRealDialog = dynamic_cast<Dialog*>(pRoot);
 
-        if (!pRealDialog)
-            pRealDialog = pDialog;
+        {
+            VclBuilder aBuilder(pDialog, VclBuilderContainer::getUIRootDir(), OStringToOUString(rUIXMLDescription, RTL_TEXTENCODING_UTF8));
+            vcl::Window *pRoot = aBuilder.get_widget_root();
+            Dialog *pRealDialog = dynamic_cast<Dialog*>(pRoot);
+
+            if (!pRealDialog)
+            {
+                pRealDialog = pDialog;
+            }
+
+            pRealDialog->SetText("LibreOffice DialogScreenshot");
+            pRealDialog->SetStyle(pDialog->GetStyle() | WB_CLOSEABLE);
+
+            dumpDialogToPath(*pRealDialog);
+        }
 
-        pRealDialog->SetText("LibreOffice DialogScreenshot");
-        pRealDialog->SetStyle(pDialog->GetStyle() | WB_CLOSEABLE);
-        dumpDialogToPath(*pRealDialog);
+        pDialog.disposeAndClear();
     }
 }
 
-- 
cgit