[Libreoffice-commits] .: Branch 'libreoffice-3-6' - reportdesign/source

Michael Stahl mst at kemper.freedesktop.org
Tue Jul 24 09:07:50 PDT 2012


 reportdesign/source/ui/report/propbrw.cxx |    4 ----
 1 file changed, 4 deletions(-)

New commits:
commit e3f087d9ed753be702503220e1f97a3514d74d62
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Jul 24 17:29:20 2012 +0200

    fdo#52399: Do not needlessly set DefaultContext of global ServiceManager
    
    ...the new cppuhelper/source/defaultbootstrap.cxx now forbids this nonsense with
    a PropertyVetoException.  It looks like the code in propbrw.cxx had always been
    confused about what to do exactly (using xOwnContext as delegate in the call to
    createComponentContext should just be enough) and the useless re-setting of the
    global ServiceManager's DefaultContext happened to work by accident.
    
    Change-Id: I80a87fe9b0948d9aa6dc9e2e63497097044a4959
    (cherry picked from commit c2617e960e1ff9c132af34b11f2e099f04fb46a3)
    
    Signed-off-by: Michael Stahl <mstahl at redhat.com>

diff --git a/reportdesign/source/ui/report/propbrw.cxx b/reportdesign/source/ui/report/propbrw.cxx
index 45b4f1a..cb4bd10 100644
--- a/reportdesign/source/ui/report/propbrw.cxx
+++ b/reportdesign/source/ui/report/propbrw.cxx
@@ -173,7 +173,6 @@ PropBrw::PropBrw(const Reference< XMultiServiceFactory >&   _xORB,Window* pParen
                     uno::Reference< uno::XInterface > xDefaultHelpProvider( inspection::DefaultHelpProvider::create( m_xInspectorContext, xInspectorUI ) );
                 }
             }
-            xFactoryProperties->setPropertyValue( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DefaultContext" ) ) ,makeAny(xOwnContext));
         }
         catch (Exception&)
         {
@@ -182,9 +181,6 @@ PropBrw::PropBrw(const Reference< XMultiServiceFactory >&   _xORB,Window* pParen
             {
                 ::comphelper::disposeComponent(m_xBrowserController);
                 ::comphelper::disposeComponent(m_xBrowserComponentWindow);
-                Reference< XPropertySet > xFactoryProperties( m_xORB, UNO_QUERY_THROW );
-                if ( xOwnContext.is() )
-                    xFactoryProperties->setPropertyValue( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DefaultContext" ) ) ,makeAny(xOwnContext));
             }
             catch(Exception&) { }
             m_xBrowserController.clear();


More information about the Libreoffice-commits mailing list