[441770] Optimize how list items are sorted

Keep sorting on each
AbstractDNodeListCompartmentEditPart.getModelChildren() invocation to
minimize the amount of change, but remove all the obvious inefficiencies
of the previous sorting method.

Bug: 441770
Change-Id: I8fa82730dc197e5b6389d27113434c87b1a0b813
Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
[441770] Avoid creating useless SimpleStyleConfiguration instances

Use a single SimpleStyleConfiguration instance in the default case
instead of creating a new stateless equivalent instance on each call.

Bug: 441770
Change-Id: I9d652d6e6318a49a873ee102db72df5a2bb4a980
Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
[431035] Enable snap to shape by default

Bug: 431035
Change-Id: I5538b575accffd0f3b75569656d184847f6a2ed7
Signed-off-by: Laurent Redor <laurent.redor@obeo.fr>
[443286] Install the AirdCrossReferenceAdapter after the domain creation

Move the AirdCrossReferenceAdapterImpl after the domain creation.

In Bug 437646, AirdDCrossReferencerAdapter installation has been moved
in the session factory and a common adapter is installed on the resource
set (we add previously on adapter per aird and only on AirdResourceImpl
see Bug 437646). 

It was done just after the resource set creation (by resource set
factory which uses the first found extension) but just after the add of
the adapter, we create the editing domain: we call the
EditingDomainInFactoryService, which looks in the extension the 'first
most overrider' factory and create an editing domain. The creation 
receives the previously created set as parameter, but the created 
editing domain can use its own resource set. 

Our adapter installation was done too soon. This typically occurs with
Capella which does not use one editing domain per session, but has its
own shared editing domain.

Bug: 443286
Change-Id: If27099f4523c1a9c574ccaca7995d20f9c01dcf5
Signed-off-by: Maxime PORHEL <maxime.porhel@obeo.fr>
[442701] Avoid leak in RefreshDDiagramElementAction

There is no need to store the selection and run(IAction) now calls
run(). 

Bug: 442701
Change-Id: Ic8818e88b0a4e82971fdc83a640b735a7143add6
Signed-off-by: Maxime PORHEL <maxime.porhel@obeo.fr>