Should we rename the method getImageHolder in ImageViewer to getCustomViewerHolder, so that the  function name is not specific to the ImageViewer? Just another small step towards a CustomViewer API that allow CustomViewerProviders to register in EditorManager.