olImageLoadError class name added but never removed

[tschaub]

I'm seeing cases where an image loading error causes the olImageLoadError class to be applied, and then later when the same image has successfully loaded (zooming out) the style declarations from the error block are still applied. Though it seems like this would be a cross browser problem, I'm only seeing it on IE, where the opacity filter still applies after the image successfully loads.

The problem goes away when I remove the olImageLoadError in the "success" handler. This certainly seems like good practice even if it is not completely clear why the problem doesn't manifest itself in all browsers.

[tschaub]

remove image error class name

[tschaub]

Tests pass on FF3.5 and IE8. Thanks for any review.

[ahocevar]

Exactly the right thing to do -- good catch! Please commit.

[tschaub]

(In [9940]) Removing any onImageLoadError class name from images when they successfully load. r=ahocevar (closes #2407)