Fix worldCopyJump set Center and Zoom first

[fastrde]

Hello,

here is the chain that produces the error in Issue #1904.

_onViewReset() 
-> L.Map.latLngToLayerPoint() 
-> L.Map.getPixelOrigin() 
-> L.Map._checkIfLoaded() 
-> BOOM

now the call of _onViewReset() is event-based and waits for the map to get ready ('load' event). That fixes Issue #1904 and should work as well for Issue #1831

jsfiddle with (manuelly) patched src.
http://jsfiddle.net/fastrde/2AuE7/8/

lg

fastrde

[mourner]

Thanks Fabian!
It seems that this would fail if map.dragging.enable() is called after the map is initialized.
Perhaps replace the changed line with a call to map.whenReady(this._onViewReset, this)?

[mourner]

It would also be great to cover this race condition with a test so that we don't run into regression in fugure.

[fastrde]

you're right. I doesn't think about that case. fixed and test added.

[mourner]

I meant automatic test in the spec folder :)

[fastrde]

Hi Vladimir,

please have a close look at the test, it's my first mocha testcase.

lg

fastrde

[mourner]

Looks good, congrats on your first test. :)