[Libreoffice-commits] core.git: Branch 'libreoffice-5-0' - embeddedobj/source

Caolán McNamara caolanm at redhat.com
Tue Jun 23 01:13:17 PDT 2015


 embeddedobj/source/commonembedding/visobj.cxx |   20 ++++++--------------
 embeddedobj/source/inc/commonembobj.hxx       |    1 -
 2 files changed, 6 insertions(+), 15 deletions(-)

New commits:
commit f2b3519c6b2aceacbe2fd9d53eb52dd36a356ecc
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Tue Jun 23 09:12:15 2015 +0100

    Resolves: tdf#91683 Revert avoid activating chart objects...
    
    just to get their mapunit or set visual size.
    
    Will have to try something different here apparently
    
    This reverts commit 757f461ef12548af0be470f7c05ff67df3dc1314.

diff --git a/embeddedobj/source/commonembedding/visobj.cxx b/embeddedobj/source/commonembedding/visobj.cxx
index cd16ffb..fea7c3a 100644
--- a/embeddedobj/source/commonembedding/visobj.cxx
+++ b/embeddedobj/source/commonembedding/visobj.cxx
@@ -54,14 +54,10 @@ void SAL_CALL OCommonEmbeddedObject::setVisualAreaSize( sal_Int64 nAspect, const
     bool bBackToLoaded = false;
     if ( m_nObjectState == embed::EmbedStates::LOADED )
     {
-        //attempt an optimization for chart components
-        if (!IsChartComponent())
-        {
-            changeState( embed::EmbedStates::RUNNING );
+        changeState( embed::EmbedStates::RUNNING );
 
-            // the links should be switched back to loaded state for now to avoid locking problems
-            bBackToLoaded = m_bIsLink;
-        }
+        // the links should be switched back to loaded state for now to avoid locking problems
+        bBackToLoaded = m_bIsLink;
     }
 
     bool bSuccess = m_pDocHolder->SetExtent( nAspect, aSize );
@@ -137,14 +133,10 @@ sal_Int32 SAL_CALL OCommonEmbeddedObject::getMapUnit( sal_Int64 nAspect )
     bool bBackToLoaded = false;
     if ( m_nObjectState == embed::EmbedStates::LOADED )
     {
-        //attempt an optimization for chart components
-        if (!IsChartComponent())
-        {
-            changeState( embed::EmbedStates::RUNNING );
+        changeState( embed::EmbedStates::RUNNING );
 
-            // the links should be switched back to loaded state for now to avoid locking problems
-            bBackToLoaded = m_bIsLink;
-        }
+        // the links should be switched back to loaded state for now to avoid locking problems
+        bBackToLoaded = m_bIsLink;
     }
 
     sal_Int32 nResult = m_pDocHolder->GetMapUnit( nAspect );
diff --git a/embeddedobj/source/inc/commonembobj.hxx b/embeddedobj/source/inc/commonembobj.hxx
index bc89f9a..410850e 100644
--- a/embeddedobj/source/inc/commonembobj.hxx
+++ b/embeddedobj/source/inc/commonembobj.hxx
@@ -171,7 +171,6 @@ private:
                 const OUString& aNewName );
 
     OUString GetDocumentServiceName() const { return m_aDocServiceName; }
-    bool IsChartComponent() const { return m_aDocServiceName == "com.sun.star.chart2.ChartDocument"; }
     OUString GetPresetFilterName() const { return m_aPresetFilterName; }
 
     ::com::sun::star::uno::Reference< ::com::sun::star::io::XInputStream >


More information about the Libreoffice-commits mailing list