Resolves: tdf#44241 highlight the newly installed extension

this was changed by

commit a6efec83
Date:   Mon Nov 28 10:54:55 2011 -0200

    Fix for bug fdo39748, Easy-hack Cleanup extension list.

so restore addPackageToList to not remove the old one before potentially
re-adding it, and instead follow the same model as
TheExtensionManager::modified to refresh the list

Change-Id: I8c3207760361dbd42e400fa9f323ca42971bbcc0
Reviewed-on: https://gerrit.libreoffice.org/28137Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

desktop/source/deployment/gui/dp_gui_dialog2.cxx

@@ -746,16 +746,12 @@ void ExtMgrDialog::setGetExtensionsURL( const OUString &rURL )
     m_pGetExtensions->SetURL( rURL );
 }
 
-
 void ExtMgrDialog::addPackageToList( const uno::Reference< deployment::XPackage > &xPackage,
                                      bool bLicenseMissing )
 {
-
     const SolarMutexGuard aGuard;
     m_pUpdateBtn->Enable();
 
-    m_pExtensionBox->removeEntry(xPackage);
-
     if (m_pBundledCbx->IsChecked() && (xPackage->getRepositoryName() == BUNDLED_PACKAGE_MANAGER) )
     {
         m_pExtensionBox->addEntry( xPackage, bLicenseMissing );
@@ -768,13 +764,8 @@ void ExtMgrDialog::addPackageToList( const uno::Reference< deployment::XPackage
     {
         m_pExtensionBox->addEntry( xPackage, bLicenseMissing );
     }
-    else
-    {
-    //OSL_FAIL("Package will not be displayed");
-    }
 }
 
-
 void ExtMgrDialog::prepareChecking()
 {
     m_pExtensionBox->prepareChecking();
@@ -1055,11 +1046,12 @@ IMPL_LINK_NOARG_TYPED(ExtMgrDialog, HandleAddBtn, Button*, void)
     setBusy( false );
 }
 
-
 IMPL_LINK_NOARG_TYPED(ExtMgrDialog, HandleExtTypeCbx, Button*, void)
 {
     // re-creates the list of packages with addEntry selecting the packages
+    prepareChecking();
     m_pManager->createPackageList();
+    checkEntries();
 }
 
 IMPL_LINK_NOARG_TYPED(ExtMgrDialog, HandleUpdateBtn, Button*, void)

desktop/source/deployment/gui/dp_gui_extlistbox.cxx

@@ -970,7 +970,6 @@ void ExtensionBox_Impl::addEntry( const uno::Reference< deployment::XPackage > &
     m_bNeedsRecalc = true;
 }
 
-
 void ExtensionBox_Impl::updateEntry( const uno::Reference< deployment::XPackage > &xPackage )
 {
     typedef std::vector< TEntry_Impl >::iterator ITER;
@@ -1000,7 +999,6 @@ void ExtensionBox_Impl::updateEntry( const uno::Reference< deployment::XPackage
     }
 }
 
-
 //This function is also called as a result of removing an extension.
 //see PackageManagerImpl::removePackage
 //The gui is a registered as listener on the package. Removing it will cause the