fix: Avoid completed event on outside boundary element (#1256)

When drawing an annotation, if it's outside the boundary of the image data, it is removed (depending on your settings).
Because of the order of our callbacks, this would allow for the following scenario:

- Begin drawing annotation
- Drag end handle off image data
- Mouseup with end handle off image data
- Measurement removed from toolstate
- Measurement completed

3rd party applications listening to events might think the measurement was sucessfully created. This change ensures measurementData marked as "canceled" will not emit a "completed" event (at least via manipulator default callbacks)

src/eventDispatchers/mouseEventHandlers/addNewMeasurement.js

@@ -37,6 +37,10 @@ export default function(evt, tool) {
     tool.options,
     'mouse',
     () => {
+      if (measurementData.cancelled) {
+        return;
+      }
+
       const eventType = EVENTS.MEASUREMENT_COMPLETED;
       const eventData = {
         toolName: tool.name,

src/manipulators/moveNewHandle.js

@@ -286,6 +286,7 @@ function _moveEndHandler(
     options.deleteIfHandleOutsideImage &&
     anyHandlesOutsideImage(evt.detail, annotation.handles)
   ) {
+    annotation.cancelled = true;
     removeToolState(element, toolName, annotation);
   }