Use ol.LEGACY_IE_SUPPORT and ol.IS_LEGACY_IE define

[fredj]

See #1605

[fredj]

http://www.w3.org/TR/html5/embedded-content-0.html#dom-img-complete may also be used

[elemoine]

I know this is not ready for review yet, but I'm curious to know what your goal is. Remove useless code at compile time when "legacy ie" is not defined?

[fredj]

I know this is not ready for review yet, but I'm curious to know what your goal is. Remove useless code at compile time when "legacy ie" is not defined?

yes, that's it

[fredj]

Tested on IE7 and IE8 (browserstack): it's working as expected.

[elemoine]

That looks good to me. Out of curiosity have you actually checked that the compiler actually optimizes this away when these defines are not set?

[fredj]

The function is compiled as:

function() {
  this.state = this.d.naturalWidth && this.d.naturalHeight ? 2 : 4;
  Ta(this.e, Yc);
  this.e = null;
  this.dispatchEvent("change")
}

vs:

function() {
  l(this.c.naturalWidth) || (this.c.naturalWidth = this.c.width, this.c.naturalHeight = this.c.height);
  this.state = this.c.naturalWidth && this.c.naturalHeight ? 2 : 4;
  Ta(this.e,Yc);
  this.e = null;
  this.dispatchEvent("change")
};

[elemoine]

Thanks.