IsValid for Bounds and LatLngBounds

[domoritz]

Implemented isValid() function for Bounds and LatLngBounds. Fixes issue #966. Added specification as well.

Example usage:

data = L.geoJson([]); // data may come from user input/ ajax
bounds = data.getBounds()
if (bounds.isValid()) {
  map.fitBounds(bounds);
}

[danzel]

Can you please revert/remove your rebuilt leaflet from the pull, it ruins trying to merging multiple pulls.
Thanks!

Could isValid just be
return this._southWest && this._northEast;
unless there is a case where they could be truthy but not valid?

[domoritz]

Of course it can simply be this._southWest && this._northEast; but then it doesn't return true or false but object and undefined. However, in the end it has the same results.

[domoritz]

Any comments?

[danzel]

Yep looks great to me!
@mourner will be the one to pull it, he'll be back in the next week I believe.

[domoritz]

@danzel @mourner Is there anything left to do for me before this gets merged in?

[mourner]

You forgot to remove typeof. Also, to cast everything to boolean, you can just do !!(this.min && this.max)

[domoritz]

Stupid me.

[mourner]

Looks good to me!