L.Control.Layers: 'overlayadd' and 'overlayremove' event should also fire on external overlay changes.

[Xelio]

Not only on clicking layer contorl, should also fire on add/remove of the overlay on other place.
jsfiddle

[jeroen]

Will this cause the event to fire 100 times when a geojson layer is added?

[Xelio]

It would only fire if the layer added to layer control and would only fire once per add / remove.

[mourner]

This change seems to only hadle overlay changes, although base layers can be changed programmatically too (by removing one and adding the other, in which case overlay events would be fired).

[Xelio]

External base layer change now fire baselayerchange instead of overlayadd.
The layer object passed by baselayerchange event is now the object in _layers of L.Control.Layers which has name property. Instead of layer only.

[Xelio]

And the change of base layer now only fire baselayerchange without overlayremove.
The jsfiddle is updated to test above changes.

[Xelio]

Or should the layer name pass as another property, layer only pass as layer?

[Xelio]

Any problem?

[mourner]

Nope, just didn't get to reviewing this pull again yet.

[mourner]

Simplified the code a bit and merged. I also changed the events to pass layer and name as separate properties as you suggested in a comment above.