[Libreoffice-commits] core.git: Branch 'libreoffice-4-3' - sd/qa sd/source xmloff/CppunitTest_xmloff_uxmloff.mk xmloff/Library_xo.mk xmloff/source
Zolnai Tamás
tamas.zolnai at collabora.com
Mon Jun 16 09:50:22 PDT 2014
sd/qa/unit/import-tests.cxx | 15 ++-------------
sd/source/ui/view/drviews2.cxx | 5 -----
sd/source/ui/view/drviews7.cxx | 11 ++---------
xmloff/CppunitTest_xmloff_uxmloff.mk | 1 -
xmloff/Library_xo.mk | 1 -
xmloff/source/draw/ximpshap.cxx | 13 ++++---------
6 files changed, 8 insertions(+), 38 deletions(-)
New commits:
commit 8f990753138ef4d25a54652cf7cf0f257b35dc0b
Author: Zolnai Tamás <tamas.zolnai at collabora.com>
Date: Mon Jun 16 18:47:03 2014 +0200
Revert "Make glTF support an experimental feature"
There are others who think this will work better before the
release.
This reverts commit 653bac184f579524bc25a9f40281763a736ae6fd.
Conflicts:
sd/qa/unit/import-tests.cxx
xmloff/source/draw/ximpshap.cxx
Change-Id: Ie8747745a9a89ce4d6500a35d15a98e28bda840a
diff --git a/sd/qa/unit/import-tests.cxx b/sd/qa/unit/import-tests.cxx
index d591732..8d43ea0 100644
--- a/sd/qa/unit/import-tests.cxx
+++ b/sd/qa/unit/import-tests.cxx
@@ -30,7 +30,6 @@
#include <svx/svdomedia.hxx>
#include <svx/svdoole2.hxx>
#include <svx/xflclit.hxx>
-#include <svtools/miscopt.hxx>
#include <animations/animationnodehelper.hxx>
#include <com/sun/star/drawing/XDrawPage.hpp>
@@ -711,7 +710,6 @@ void SdFiltersTest::testFdo71961()
void SdFiltersTest::testMediaEmbedding()
{
- SvtMiscOptions().SetExperimentalMode(true);
::sd::DrawDocShellRef xDocShRef = loadURL(getURLFromSrc("/sd/qa/unit/data/media_embedding.odp"));
#if HAVE_FEATURE_GLTF
@@ -729,21 +727,12 @@ void SdFiltersTest::testMediaEmbedding()
CPPUNIT_ASSERT_MESSAGE( "missing model", pModelObj != NULL);
CPPUNIT_ASSERT_EQUAL( OUString( "vnd.sun.star.Package:Models/duck/duck.json" ), pModelObj->getMediaProperties().getURL());
CPPUNIT_ASSERT_EQUAL( OUString( "model/vnd.gltf+json" ), pModelObj->getMediaProperties().getMimeType());
-
- // Check the case when experimental mode is disabled
- xDocShRef->DoClose();
- SvtMiscOptions().SetExperimentalMode(false);
- xDocShRef = loadURL(getURLFromSrc("/sd/qa/unit/data/media_embedding.odp"));
- pDoc = xDocShRef->GetDoc();
- CPPUNIT_ASSERT_MESSAGE( "no document", pDoc != NULL );
- pPage = pDoc->GetPage (1);
- CPPUNIT_ASSERT_MESSAGE( "no page", pPage != NULL );
-#endif
-
+#else
// If glTF is not supported, then the fallback image is imported
SdrGrafObj *pGrafic = dynamic_cast<SdrGrafObj*>( pPage->GetObj( 2 ));
CPPUNIT_ASSERT_MESSAGE( "Could not load glTF fallback image", pGrafic != NULL);
CPPUNIT_ASSERT_EQUAL( OUString( "vnd.sun.star.Package:Models/Fallbacks/duck.png" ), pGrafic->GetGrafStreamURL());
+#endif
// Second object is a sound
SdrMediaObj *pMediaObj = dynamic_cast<SdrMediaObj*>( pPage->GetObj( 3 ));
diff --git a/sd/source/ui/view/drviews2.cxx b/sd/source/ui/view/drviews2.cxx
index 49a6597..384a6c7 100644
--- a/sd/source/ui/view/drviews2.cxx
+++ b/sd/source/ui/view/drviews2.cxx
@@ -79,8 +79,6 @@
#include <svx/xlnstwit.hxx>
#include <svx/xlnwtit.hxx>
-#include <svtools/miscopt.hxx>
-
#include <tools/diagnose_ex.h>
#include <unotools/useroptions.hxx>
@@ -1173,9 +1171,6 @@ void DrawViewShell::FuTemporary(SfxRequest& rReq)
#if HAVE_FEATURE_GLTF
case SID_INSERT_3DMODEL:
{
- if( !SvtMiscOptions().IsExperimentalMode() )
- break;
-
SetCurrentFunction( FuInsert3DModel::Create( this, GetActiveWindow(), mpDrawView, GetDoc(), rReq ) );
Cancel();
diff --git a/sd/source/ui/view/drviews7.cxx b/sd/source/ui/view/drviews7.cxx
index 10d453c..81e2e00 100644
--- a/sd/source/ui/view/drviews7.cxx
+++ b/sd/source/ui/view/drviews7.cxx
@@ -43,8 +43,6 @@
#include <svx/extrusionbar.hxx>
#include <svx/fontworkbar.hxx>
-#include <svtools/miscopt.hxx>
-
// #UndoRedo#
#include <svl/slstitm.hxx>
#include <sfx2/app.hxx>
@@ -1596,18 +1594,13 @@ void DrawViewShell::GetMenuState( SfxItemSet &rSet )
}
}
- bool bDisableInsert3DModel = false;
#if !HAVE_FEATURE_GLTF
- bDisableInsert3DModel = true;
-#else
- bDisableInsert3DModel = !SvtMiscOptions().IsExperimentalMode();
-#endif
-
- if (bDisableInsert3DModel && SFX_ITEM_AVAILABLE == rSet.GetItemState(SID_INSERT_3DMODEL))
+ if (SFX_ITEM_AVAILABLE == rSet.GetItemState(SID_INSERT_3DMODEL))
{
rSet.DisableItem(SID_INSERT_3DMODEL);
rSet.Put(SfxVisibilityItem(SID_INSERT_3DMODEL, false));
}
+#endif
GetModeSwitchingMenuState (rSet);
}
diff --git a/xmloff/CppunitTest_xmloff_uxmloff.mk b/xmloff/CppunitTest_xmloff_uxmloff.mk
index 4f793a9..41fed90 100644
--- a/xmloff/CppunitTest_xmloff_uxmloff.mk
+++ b/xmloff/CppunitTest_xmloff_uxmloff.mk
@@ -44,7 +44,6 @@ $(eval $(call gb_CppunitTest_use_libraries,xmloff_uxmloff, \
salhelper \
sax \
svl \
- svt \
test \
tl \
utl \
diff --git a/xmloff/Library_xo.mk b/xmloff/Library_xo.mk
index 1bf512c..553fcc1 100644
--- a/xmloff/Library_xo.mk
+++ b/xmloff/Library_xo.mk
@@ -50,7 +50,6 @@ $(eval $(call gb_Library_use_libraries,xo,\
salhelper \
sax \
svl \
- svt \
tl \
utl \
vcl \
diff --git a/xmloff/source/draw/ximpshap.cxx b/xmloff/source/draw/ximpshap.cxx
index ccb95f9..2e573ab 100644
--- a/xmloff/source/draw/ximpshap.cxx
+++ b/xmloff/source/draw/ximpshap.cxx
@@ -81,7 +81,6 @@
#include <basegfx/polygon/b2dpolygontools.hxx>
#include <basegfx/polygon/b2dpolypolygontools.hxx>
#include <basegfx/vector/b2dvector.hxx>
-#include <svtools/miscopt.hxx>
#include <config_features.h>
@@ -3071,7 +3070,7 @@ void SdXMLPluginShapeContext::StartElement( const ::com::sun::star::uno::Referen
if( xAttrList->getValueByIndex( n ).equalsAscii( "application/vnd.sun.star.media" ) )
mbMedia = true;
#if HAVE_FEATURE_GLTF
- if( xAttrList->getValueByIndex( n ).equalsAscii( "model/vnd.gltf+json" ) && SvtMiscOptions().IsExperimentalMode() )
+ if( xAttrList->getValueByIndex( n ).equalsAscii( "model/vnd.gltf+json" ) )
mbMedia = true;
#endif
// leave this loop
@@ -3524,14 +3523,9 @@ SvXMLImportContext *SdXMLFrameShapeContext::CreateChildContext( sal_uInt16 nPref
if ( !msHyperlink.isEmpty() )
pShapeContext->setHyperlink( msHyperlink );
- // Ignore gltf model if necessary and so the fallback image will be imported
- bool bIngoreGltf;
#if !HAVE_FEATURE_GLTF
- bIngoreGltf = true;
-#else
- bIngoreGltf = !SvtMiscOptions().IsExperimentalMode();
-#endif
- if( bIngoreGltf && IsXMLToken(rLocalName, XML_PLUGIN ) )
+ // Ignore gltf model if necessary and so the fallback image will be imported
+ if( IsXMLToken(rLocalName, XML_PLUGIN ) )
{
SdXMLPluginShapeContext* pPluginContext = dynamic_cast<SdXMLPluginShapeContext*>(pShapeContext);
if( pPluginContext && pPluginContext->getMimeType() == "model/vnd.gltf+json" )
@@ -3540,6 +3534,7 @@ SvXMLImportContext *SdXMLFrameShapeContext::CreateChildContext( sal_uInt16 nPref
return this;
}
}
+#endif
mxImplContext = pContext;
mbSupportsReplacement = IsXMLToken(rLocalName, XML_OBJECT ) || IsXMLToken(rLocalName, XML_OBJECT_OLE);
More information about the Libreoffice-commits
mailing list