Map undefined in L.PopUp _updatePosition method when popup closed while viewport change

[fnicollet]

Hi,
I just ran into an issue in my application. It was in IE10 but i don't believe that's related.

I can't really reproduce this issue with a clear example, sorry. I had a pop-up opened and changed the viewport bounds. On viewport change, i also update the markers and in that case, it cleared the markers, including the one that had a pop-up attached.
I think the event handlers just got in a bad order

There is a test on "this._map" already in L.PopUp as in "_update". To prevent the bug from happening, the same test should be added to _updatePosition:

_updatePosition: function () {
    if (!this._map) { return; }
    var pos = this._map.latLngToLayerPoint(this._latlng),

...

Just while the pop-up is being closed, I think that's just bad timing :)

Fabien

[danzel]

Confirmed and PR #1098 sent to fix.
If you remove the popup from the map (close it) during a viewreset it can try re-position itself which will fail.