Because the web console creates sandbox objects whose proto is the window, 'this' might not have the InstallTrigger on it directly. This means that the 'delete' will silently do nothing and we'll infinitely recur when trying to return the actual value. This means that we have to walk the prototype chain actually looking for the window.
Created attachment 488376 [details] [diff] [review] Proposed fix
Attachment #488376 - Flags: review?(dtownsend)
Comment on attachment 488376 [details] [diff] [review] Proposed fix Replace the XXX with the right bug number and this looks good. Does it make sense to add a test for this?
Attachment #488376 - Flags: review?(dtownsend) → review+
In trying to add a test, I found and filed bug 610078.
Attachment #488376 - Flags: approval2.0?
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.