Manually adjusting some more anomalous META-INF/services registrations.

Some META-INF/services/* cleanups.
1. Some references to service implementations which no longer exist (or never existed)?
2. Typos, e.g. trailing space in class name.
3. Duplicate registrations of impl classes already registered in the right place.
4. An interface registered as an impl of itself (??).
5. DataLoader's need no longer be registered in lookup of unit tests; they should be found anyway.
6. No need to construct procedural MIMEResolver's in lookup of unit tests; declarative resolvers run anyway.
7. Some registrations in the wrong module (after impl was moved), in some cases specifying the wrong package name.
...and comments in other files which are dubious and which cannot be trivially converted to @ServiceProvider.
Some cases remain where @ServiceProvider really cannot be used:
1. Some services are being masked but nothing put in their place.
2. An impl which is not normally registered needs to be registered from another module (generally inside unit tests).

META-INF/services cleanups.
1. Fixing syntax of position attribute in some CND modules.
2. Deleting files with no registrations.
3. Removing unnecessary comments.
4. Removing incorrect leading spaces in editor.lib2.

removing unused module dependencies on deprecated editor modules

#150606 Providing logging of the problem and avoiding NPE.

Increasing spec. versions after 6.5.

#148157 Avoiding adding action for nodes it doesn't work for.

#148062 Fixing image reference, to be shown in the VW multiview.

#149015 Missing part of the original fix.

#149015 Applying fix from 6.1 branch to trunk.