[442677] Improve comments and avoid to delete an already deleted element

Bug: 442677
Change-Id: I76f3826c42361c5890311a27b7e724daa8bf9f0c
Signed-off-by: Maxime PORHEL <maxime.porhel@obeo.fr>
[442701] Avoid leak with the tabbar with copy appearance action

Use a TabbarActionContributionItem to dispose the GMF action when the
tabbar is disposed.
 
Bug: 442701
Change-Id: Id94ef3407f550cb2c1698e2648955bc2a1e5bcac
Signed-off-by: Maxime PORHEL <maxime.porhel@obeo.fr>
[441262] Update the release note document

Bug: 441262
Change-Id: I00e989ff9d3a5b011504592e8acabbc413507bfc
Signed-off-by: Belqassim Djafer <belqassim.djafer@obeo.fr>
[431035] Snap to shape is now enabled by default, disable it for tests

Indeed, the existing tests expect a disabled snap to shape, so the
activation of it could have unexpected result.

Bug: 431035
Change-Id: I0c4b9169ddb66897479d58be83a3a6d1198045bb
Signed-off-by: Laurent Redor <laurent.redor@obeo.fr>
[442176] Move WorkspaceEPackageRegistry init code out of plugin.start()



Bug: 442176
Change-Id: I8465a29b29bee03474d91d40e2007b11554fd918
Signed-off-by: Cedric Brun <cedric.brun@obeo.fr>
[442176] Stop using WorkspaceEPackageRegistry to detect models

XMIModelFileHandler does not check the namespaces present in the file
any more but just that the XMI namespace is present. Then we let the
resource load fail if an unknwon metamodel is found. 

We could have continued to check if the found namespace are present in
the EcorePackageRegistry (global or resource set one's) but with the
current behavior, the resource load will throws the expected errors.

Bug: 442176
Change-Id: Iac8de595dd13af4a63735699907b7de74aa98790
Signed-off-by: Cedric Brun <cedric.brun@obeo.fr>
Signed-off-by: Maxime Porhel <maxime.porhel@obeo.fr>
[442176] Move WorkspaceEPackageRegistry in o.e.s.editor plugin

Bug: 442176  
Change-Id: I78d54eef34454c919e6c39c0172f115042608cda
Signed-off-by: Cedric Brun <cedric.brun@obeo.fr>
[441770] Cache the previous invocation of StyleConfigurationRegistry.getStyleConfiguration()

StyleConfigurationRegistry.getStyleConfiguration() can be called a lot
in sensitive loops. It used to create a new StyleWrapper instance almost
every time, but StyleWrapper.hashCode(), which is used to find a match
in the registry, can be very costly (it uses EcoreUtil.getURI() to match
elements accross sessions and ResourceSets).

In practice, at least in the case of this bug, successive calls to
getStyleConfiguration() will often use the same mapping instance and
styles with the same StyleDescription many times in a row. Simply
caching the parameters and result of the previous invocation, and
matching the parameters using == gives a performance boost in these
scenario, with very little overhead in time and memory in cases where we
revert to the previous behavior.

Bug: 441770
Change-Id: I991f0ab59c88ca63bd5ea4eefc3a605b0004192c
Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
[442654] Change title for the rename representation dialog

Bug: 442654
Change-Id: I269911a9d2026cb42d7453995d7d504d33cad114
Signed-off-by: Jessy Mallet <jessy.mallet@obeo.fr>