add/remove/modified events for annotations should not include viewportId and renderingEngineId

[sedghi]

add/remove/modified events should not have viewportId and renderingEngineId as their event details. There are only one place that we depend on these Ids (at the onAnnotationModifiedListener where we trigger one last render for the viewports to update their text box since the cachedStats has been updated in a throttle).

I went ahead and removed all the unnecessary Ids but the rendering of annotations fails on the non acquisition planes. It appears
that triggerAnnotationForViewportIds is run for each viewports separately (assuming pt/ct layout, and annotating on ctAxial, it runs for ctAxial, ptAxial, fusionAxial separately), but the implemented approach was to trigger all in one, which failed.

Needs more investigation as why.

Potential reasons:

• async throttling doesn't invalidate other viewports from not running their cachedStats and becomes an infinite loop? @JamesAPetts

Branch to look is here

[sedghi]

@jbocce ditto

[sedghi]

Although it would have been ideal not to begin with this, I am confident that no bugs will occur as a result. Therefore, it was not planned.