Do not accidentally instantiate other exts when removing exts at startup

desktop/source/deployment/manager/dp_extensionmanager.cxx

@@ -464,7 +464,7 @@ void ExtensionManager::activateExtension(
             //Check if this is a disabled user extension,
             if (i == 0 && bUserDisabled)
             {
-                   aExt->revokePackage(xAbortChannel, xCmdEnv);
+                   aExt->revokePackage(bStartup, xAbortChannel, xCmdEnv);
                    continue;
             }
 
@@ -473,7 +473,7 @@ void ExtensionManager::activateExtension(
             //repositories with a lower priority
             if (bActive)
             {
-                aExt->revokePackage(xAbortChannel, xCmdEnv);
+                aExt->revokePackage(bStartup, xAbortChannel, xCmdEnv);
             }
             else
             {
@@ -694,7 +694,7 @@ Reference<deploy::XPackage> ExtensionManager::addExtension(
                     try
                     {
                         xOldExtension->revokePackage(
-                            xAbortChannel, Reference<ucb::XCommandEnvironment>());
+                            false, xAbortChannel, Reference<ucb::XCommandEnvironment>());
                         //save the old user extension in case the user aborts
                         //store the extension in the tmp repository, this will overwrite
                         //xTmpPackage (same identifier). Do not let the user abort or
@@ -873,7 +873,7 @@ void ExtensionManager::removeExtension(
         Reference<deploy::XPackage> xOldExtension =
             xPackageManager->getDeployedPackage(
                 identifier, fileName, xCmdEnv);
-        xOldExtension->revokePackage(xAbortChannel, xCmdEnv);
+        xOldExtension->revokePackage(false, xAbortChannel, xCmdEnv);
 
         xPackageManager->removePackage(
             identifier, fileName, xAbortChannel, xCmdEnv);
@@ -1024,7 +1024,7 @@ sal_Int32 ExtensionManager::checkPrerequisitesAndEnable(
         if (ret)
         {
             //There are some unfulfilled prerequisites, try to revoke
-            extension->revokePackage(xAbortChannel, xCmdEnv);
+            extension->revokePackage(false, xAbortChannel, xCmdEnv);
         }
         const OUString id(dp_misc::getIdentifier(extension));
         activateExtension(id, extension->getName(),

desktop/source/deployment/manager/dp_manager.cxx

@@ -1322,7 +1322,7 @@ bool PackageManagerImpl::synchronizeRemovedExtensions(
                 Reference<deployment::XPackage> xPackage = m_xRegistry->bindPackage(
                     url, i->second.mediaType, true, i->first, xCmdEnv );
                 OSL_ASSERT(xPackage.is()); //Even if the files are removed, we must get the object.
-                xPackage->revokePackage(xAbortChannel, xCmdEnv);
+                xPackage->revokePackage(true, xAbortChannel, xCmdEnv);
                 removePackage(xPackage->getIdentifier().Value, xPackage->getName(),
                               xAbortChannel, xCmdEnv);
                 bModified |= true;

desktop/source/deployment/registry/dp_backend.cxx

@@ -726,13 +726,14 @@ void Package::registerPackage(
 
 //______________________________________________________________________________
 void Package::revokePackage(
+    sal_Bool startup,
     Reference<task::XAbortChannel> const & xAbortChannel,
     Reference<XCommandEnvironment> const & xCmdEnv )
     throw (deployment::DeploymentException,
            CommandFailedException, CommandAbortedException,
            lang::IllegalArgumentException, RuntimeException)
 {
-    processPackage_impl( false /* revoke */, false, xAbortChannel, xCmdEnv );
+    processPackage_impl( false /* revoke */, startup, xAbortChannel, xCmdEnv );
 
 }
 

desktop/source/deployment/registry/inc/dp_backend.h

@@ -204,6 +204,7 @@ public:
                css::ucb::CommandAbortedException,
                css::lang::IllegalArgumentException, css::uno::RuntimeException);
     virtual void SAL_CALL revokePackage(
+        sal_Bool startup,
         css::uno::Reference<css::task::XAbortChannel> const & xAbortChannel,
         css::uno::Reference<css::ucb::XCommandEnvironment> const & xCmdEnv )
         throw (css::deployment::DeploymentException,

desktop/source/deployment/registry/package/dp_package.cxx

@@ -909,7 +909,7 @@ void BackendImpl::PackageImpl::processPackage_(
                     {
                         try {
                             bundle[ pos ]->revokePackage(
-                                xSubAbortChannel, xCmdEnv );
+                                startup, xSubAbortChannel, xCmdEnv );
                         }
                         catch (const Exception &)
                         {
@@ -950,7 +950,8 @@ void BackendImpl::PackageImpl::processPackage_(
                 xPackage->createAbortChannel() );
             AbortChannel::Chain chain( abortChannel, xSubAbortChannel );
             try {
-                bundle[ pos ]->revokePackage( xSubAbortChannel, xCmdEnv );
+                bundle[ pos ]->revokePackage(
+                    startup, xSubAbortChannel, xCmdEnv );
             }
             catch (const RuntimeException &) {
                 throw;

desktop/source/deployment/registry/script/dp_script.cxx

@@ -398,7 +398,7 @@ void BackendImpl::PackageImpl::processPackage_(
     bool bDialog = !m_dialogURL.isEmpty();
     Reference<css::script::XLibraryContainer3> xDialogLibs;
 
-    bool bRunning = office_is_running();
+    bool bRunning = !startup && office_is_running();
     if( bRunning )
     {
         if( bScript )
@@ -457,7 +457,7 @@ void BackendImpl::PackageImpl::processPackage_(
         bDialogSuccess = lcl_maybeAddScript(bDialog, m_dialogName, m_dialogURL, xDialogLibs);
     }
     bool bSuccess = bScript || bDialog;     // Something must have happened
-    if( bRunning && !startup)
+    if( bRunning )
         if( (bScript && !bScriptSuccess) || (bDialog && !bDialogSuccess) )
             bSuccess = false;
 

offapi/com/sun/star/deployment/XPackage.idl

@@ -166,6 +166,10 @@ interface XPackage
 
         NEVER call this directly. This is done by the extension manager if necessary.
 
+        @param startup
+               indicates that registration is adapted to the particular
+               startup scenario. That is, it is set to <TRUE/>, when called
+               from <member>XExtensionManager::synchronize</member>
         @param xAbortChannel
                abort channel to asynchronously abort the registration process,
                or <NULL/>
@@ -173,6 +177,7 @@ interface XPackage
                command environment for error and progress handling
     */
     void revokePackage(
+        [in] boolean startup,
         [in] com::sun::star::task::XAbortChannel xAbortChannel,
         [in] com::sun::star::ucb::XCommandEnvironment xCmdEnv )
         raises (DeploymentException,