deregister marker mouseup and touchend when removing marker

[vbud]

We have run into scenarios where we remove a draggable marker before mouseup is ever fired and, because mouseup/touchend never fired once, _onUp is called but the marker has already been torn down, leading to errors.

Ensuring these listeners are deregistered when tearing down the marker will ensure that doesn't happen.

I didn't see any tests in the marker unit test file that ensure that listeners are being deregistered, so I did not add any tests with this. Let me know if I missed anything. Tests added.

[mollymerp]

Thank you for this contribution! This test would have caught the error you're seeing. Would you mind adding it to the marker.test.js file

test('Marker with draggable:true does not error if removed on mousedown', (t) => {
    const map = createMap(t);
    const marker = new Marker({draggable: true})
        .setLngLat([0, 0])
        .addTo(map);
    const el = marker.getElement();
    simulate.mousedown(el);
    simulate.mousemove(el, {clientX: 10, clientY: 10});

    marker.remove();
    t.ok(map.fire('mouseup'));
    t.end();
});

[vbud]

@mollymerp tests added!

[mollymerp]

✨thanks again @vbud !