Use f=geojson if possible for FeatureLayer

[patrickarlt]

This updates L.esri.Tasks.Query (and subsequently anything that uses it) to request GeoJSON by default if the service is hosted in ArcGIS Online (URL has .arcgis.com).

This has a few advantages over the current methods of converting geometries client side

• Although largely mitigated edge cases discovered in Simplifying Complex Polygons #320 with simplifying geometries can no longer occur since geometries will be simplified server side BEFORE being converted to GeoJSON. These issues were a result of trying to convert simplified geometries on the client.
• Shaves a few milliseconds off every set of requests, not a huge deal but still good.

Unfortunately there is a bug in the f=geojson implementation where multipolygon features are not being returned properly.

As you can see holes in polygons are not being returned inside the polygon but rather as another polygon which makes some things appear twice.

Once this is fixed I'll merge this in so it can get released.

[jgravois]

👯

[patrickarlt]

This should be patched tonight and once tests pass this can be merged.

[patrickarlt]

This is fixed in production, now the tests just have to pass.