freehand feature drawing is broken in IE

[tschaub]

This is a regression.

http://openlayers.org/dev/examples/draw-feature.html

The draw feature controls are in freehand mode by default. In IE, you cannot draw unless you hold down shift (toggle out of freehand).

[euzuro]

regressions are bad news. let's be sure to get this fixed for 2.7

[tschaub]

Broke with r7546.

[tschaub]

(In [7802]) Temporary fix for weird vml rendering bug. (see #1690)

[tschaub]

Oddly, even calling feature.geometry.getBounds() in the drawFeature function causes the renderer failure (with no exception). This is all due to a linestring with no components - but we are supporting drawing of these elsewhere.

[ahocevar]

The above patch fixes the issue. When moving the point and changing its coordinates, the bounds of the point also need to be cleared.

This one was hard to find. For whatever reason, IE fails without an exception before applying this patch. First I thought this was because Geometry.Point::clone does not clone the bounds, but that was not the reason. IE only fails when drawing the point, not the line in Handler.Path::drawFeature.

So the issue has nothing to do with a linestring with no components. Although the reason for the strange behaviour of IE is unclear to me, the patch seems obvious and works. All tests still pass.

[elemoine]

Andreas, thanks for the patch, it looks good to me.

[ahocevar]

(In [7859]) Clear point bounds after changing coordinates. Prevents strange behaviour of IE when doing getBounds(). r=elemoine (closes #1690)