when passing `crossOrigin: ''` (empty string) option, the `crossorigin` attribute should also be set an empty string value (valid keyword, with same effect as `"anonymous"` value). Need to add this test, because in JavaScript the empty string is a falsy value that could have been caught as `false` and led to no `crossorigin` attribute set.
* Test(ImageOverlay): crossOrigin option to IMG crossorigin attribute
typical values: false (default), true => '', <string> => <string>
now that a string value can be passed to this option.
* Test(TileLayer): crossOrigin option to IMG crossorigin attribute
typical values: false (default), true => '', <string> => <string>
now that a string value can be passed to this option.
* Style(TileLayerSpec): remove commented code
* Style(TileLayerSpec): remove more commented code
following issue #6004 (fixed by PR #6006).
Added 2 dedicated tests.
Unfortunately I could not add a test for detectRetina option that easily, because since the move to Rollup, Browser values became internal variables, and overriding them later on does not have any effect on Leaflet internal references… Not sure how to workaround this for the sake of testing.
* Load CSS in PhantomJS
* Allow to take screenshot in PhantomJS while running tests
For that, one need to run this call:
window.top.callPhantom({'render': 'screenshot.png'});
* Make PhantomJS serve images
* Bonus: those tests now passe in Phantom too
* Add helper to make screenshot in tests
takeScreenshot();
or
takeScreenshot('path/to/screenshot.png');
* Fix#4039: Use different L.Bounds for "marking as prunable" and loading tiles.
* configurable tile pruning tests
* using spy for testing load did not happen
Map size does not change from one zoom to another, so we dont need
to scale it.
Without this change, we are creating more or less 4x tiles when
zooming in.
There are still cases where we are requesting a bit more tiles than
what I would expect, but I see it also happens on 0.7.3
