XML parser missing bind for IE

[tschaub]

For IE, the XML format read method was not getting the correct 'this'.

Patch to follow.

[tschaub]

patch with tests

[tschaub]

Should be an easy review. All tests pass in IE and FF.

[euzuro]

looks good to me. only thing i'd comment on is that this test:

    if(format.xmldom) { 
        t.ok(true, "format only has xmldom in browsers with ActiveX"); 
    } 

doesn't really seem helpful... does it?

Otherwise good to go. All tests pass.

[tschaub]

That test is run conditionally - only in browsers with ActiveXObject. The planner is looking for 5 tests in such browsers, 4 in others. If format.xmldom is true in non-ActiveXObject browsers, the tests will fail because they will get 5 instead of 4. Not the most instructive sort of error - but we have to make sure that format.xmldom is not true in non-ActiveX browsers.

[tschaub]

in w/ r3875

[euzuro]

Suggested logic modification for tests

[euzuro]

I added a new patch that I think makes that xmldom/ActiveXObject bit more clear?

please review.

[tschaub]

we really want to test (!window.ActiveXObject format.xmldom)

[euzuro]

r3896