Not setting a style results in blank Map

[tobrun]

We are not informing the end developer that setting a style is required.

• provide a default style?
• implement logging/exception to inform end developer?

[tobrun]

Same case for entering a not valid url

[ljbade]

There should be some stuff in the log from the C++ side.

Problem we have is setting style is async, we don't know URL is bad until sometime after we return from setStyleURL.

I used to have a basic URL validity check in the Java side that threw an exception, but caused more issues than it fixed.

[ljbade]

I'm documenting that null will load the default style in #2002 so going to have to implement this for v2.0.0

[ljbade]

Also the failed loading onMapChange event should be fired.

[tobrun]

@bleege
is this issue actionable?
Should we load a default style in those cases?

I noticed this use-case on current master

• open Activity with a map but not calling setStyle
• blank Map
• rotate device
• Mapbox Streets style is loaded

[bleege]

Setting a style is part of the setup process (along with an Access Token) so seeing a blank map seems like a good "error message". I'm open to be convinced otherwise though. ;-)

The reason for the Mapbox Streets in the TestApp after rotation is due to it being set as the default value in the Activity for state tracking. This then gets set in onStateSave() and then loaded in onCreate() during rotation. This won't happen in the SDK on it's own.