feat(ktx): GeoJson and Kml KTX

[arriolac]

Took a pass in the geojson and kml packages to see what we can make Kotlin friendly. The only method I could find that we can improve is the constructor for GeoJsonLayer and KmlLayer.

Before:

GeoJsonLayer layer = new GeoJsonLayer(map, geoJsonFile, null, polygonManager, null, groundOverlayManager);

After:

val layer = GeoJsonLayer(
    map = map,
    geoJsonFile = geoJsonFile,
    polygonManager = polygonManager,
    groundOverlayManager = groundOverlayManager
)

Other changes:

• making all extension functions and properties in the kx library inline to avoid overhead of new memory allocations when using the ktx aliases vs. using the method/properties directly
• Added @ColorInt annotations in geojson classes where I found it to be helpful
• Added @RawRes annotation in kml class
• Noticed an improvement that we can make in KmlLayer. Issue filed here Allow passing in a Context instead of a FragmentActivity when constructing KmlLayer #629

[codecov]

Codecov Report

Merging #628 into android-maps-utils-ktx will not change coverage by %.
The diff coverage is n/a.

@@                   Coverage Diff                   @@
##           android-maps-utils-ktx     #628   +/-   ##
=======================================================
  Coverage                   18.99%   18.99%           
=======================================================
  Files                          71       71           
  Lines                        4037     4037           
  Branches                      606      606           
=======================================================
  Hits                          767      767           
  Misses                       3238     3238           
  Partials                       32       32           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5c3b89...fa1f839. Read the comment docs.

[barbeau]

Generally LGTM, but I think it would make sense to merge #631 first and then update this to match the new KmlLayer constructor signature before merging this PR.

[barbeau]

I'd suggest merging #631 and then update this to take in a Context and adding the cache parameter.

[arriolac]

Yep, agreed

[arriolac]

Updated fa1f839