Race conditions with creation and destruction of OverviewMap

[ahocevar]

Currently, the update() method of Control.OverviewMap is bound to the moveend event before the initial call of update(), which is there to create the overview map. In some occasions, if the moveend event is triggered before the initial update is finished, the overview map will break because createMap() will run twice at the same time.

The attached patch fixes this by registering update() for the moveend event after calling update() manually.

Note that this patch also removes the clickHandler deactivate/activate cycle, which is obsolete now that #1247 is fixed.

All tests pass in FF2 and IE6.

[ahocevar]

In addition to the problems on creation of the OverviewMap controls, IE will also choke when the page unloads. This is due to the fact that an OL.Map object registers its destroy method for window.unload. We do not want to do this, because the ovmap might already be destroyed before OverviewMap.destroy() is run. See examples/overviewmap.html to see that.

The second patch (1320-r5985-A1.patch) includes the changes of the previous patch, plus unregisters the window.unload event for the ovmap.

All tests pass in FF2 and IE6.

[tschaub]

Nice catches. Please commit.

[ahocevar]

(In [6010]) Race conditions with creation and destruction of OverviewMap. r=tschaub (closes #1320)