exception calling getIeDocumentElementOffset() on IE11

[dylan]

I've received a report that ​https://github.com/dojo/dojo/blob/db66e918cd5a736fa4023741ace82e1bdc0fb612/dom-geometry.js#L434 is causing an issue within IE11, with the vague details of:

Error: Object doesn't support property or method 'getIeDocumentElementOffset'

The likely cause is that we need a feature test here, rather than a UA sniff. We should check this for IE11 and Edge.

[bill]

Hmm, that is vague, and mysterious. There's no point to adding any feature tests here though. You could simply change

if(has("ie")){
        geom.getIeDocumentElementOffset = function getIeDocumentElementOffset(/*Document?*/ doc){
              ...
        };
}

to be:

geom.getIeDocumentElementOffset = function getIeDocumentElementOffset(/*Document?*/ doc){
      ...
};

Better yet, since we desupported IE6 and IE7 years ago, you can change that whole code block to be:

geom.getIeDocumentElementOffset = function(/*Document?*/ doc){
        return {
                x: 0,
                y: 0
        };
};

... and then stop calling that method internally, since it doesn't do anything.

BTW, the only way the user could be hitting that error is if both has("ie") and has("dom-addeventlistener") are falsy. So I doubt the problem is happening on IE11 as they claim. Probably it's something weird with an embedded browser based on IE code.

[bill]

PS: The call to has("dom-addeventlistener") in normalizeEvent() is goofy, since it's really trying to test whether events define pageX/pageY. But I guess it's good enough.

[bill]

I'll fix this one; I'm doing some other fixes in core now.

[bill]

Fixed in a7a5646855dfe5b25f6b0c1e1c447a9b7342c110 and more cleanup in 779e4e2315418ce406262540891ee8db3017f65a.

Backported in bde5d981f570a1c95b3fbb038f7edc77d97c3757 and 0aa20f7df1048587eae5b316d8861a33900f637b.