Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wire/debug trace breaks IE <= 9 #78

Closed
briancavalier opened this issue Feb 28, 2013 · 3 comments
Closed

wire/debug trace breaks IE <= 9 #78

briancavalier opened this issue Feb 28, 2013 · 3 comments
Assignees
Milestone

Comments

@briancavalier
Copy link
Member

The problem is that trace is very aggressive in trying to advise every possible method of every object. This works fine in sane browsers where builtins, like String, are actually real Javascript objects, but breaks in IE where builtins aren't actually Javascript objects, but rather bizarro IE things.

So, for example, if you have a basic type, like a String, in your wire spec, wire/debug will break as it tries to advise the string's methods.

In limited testing, a reasonable workaround seems to be to limit wire/debug to things that are actually objects via the usual Object.prototype.toString.call(thing) === '[object Object]' test (which avoids arrays as well, obviously).

@ghost ghost assigned briancavalier Feb 28, 2013
@unscriptable
Copy link
Member

it's probably best not to mess with DOM nodes in IE either, so this is probably for the best.

briancavalier added a commit that referenced this issue Feb 28, 2013
@briancavalier
Copy link
Member Author

Yeah, def. We're already filtering out DOM nodes, thankfully. Interestingly, it's possible to AOP DOM nodes using meld in WebKit! Dunno how useful it would be, though, so yeah, best just to stay away from them across the board for now.

@briancavalier
Copy link
Member Author

Fixed by 6595244

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants