Closed Bug 351215 Opened 18 years ago Closed 18 years ago

instanceof now failing for subscript loader generated objects

Categories

(Other Applications :: ChatZilla, defect)

defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bugzilla-mozilla-20000923, Assigned: bugzilla-mozilla-20000923)

Details

(Whiteboard: [cz-0.9.76])

Attachments

(1 file)

Woohoo. Objects (especially arrays) created by subscript loader code (like, say, every single plugin) now fails instanceof in the main window code. This utterly screws over hooks, and ends up with plugin-set hooks matching pretty much every event. Global search and replace time.
Assignee: rginda → silver
Status: NEW → ASSIGNED
Attachment #236602 - Flags: review?(samuel)
Is this caused by a JS engine and/or XPConnect regression that should be filed separately, or was the change intentional?
Don't know, don't care, right now. If you want want to do the work to find out, do it, but I sure don't. I just want ChatZilla to actually work. :(
Comment on attachment 236602 [details] [diff] [review]
Use isinstance everywhere

How about changing isinstance to check instanceof first?
Attachment #236602 - Flags: review?(samuel) → review+
Sorry, I did look, but I was expecting it as the first test... oops...
(In reply to comment #3)
> Don't know, don't care, right now. If you want want to do the work to find out,
> do it, but I sure don't. I just want ChatZilla to actually work. :(

Do you have an approximate regression range?
No. It could easily be as far back as bug 336040 comment 5.
Checking in mozilla/extensions/irc/js/lib/file-utils.js;
/cvsroot/mozilla/extensions/irc/js/lib/file-utils.js,v  <--  file-utils.js
new revision: 1.10; previous revision: 1.9
done
Checking in mozilla/extensions/irc/js/lib/menu-manager.js;
/cvsroot/mozilla/extensions/irc/js/lib/menu-manager.js,v  <--  menu-manager.js
new revision: 1.8; previous revision: 1.7
done
Checking in mozilla/extensions/irc/js/lib/pref-manager.js;
/cvsroot/mozilla/extensions/irc/js/lib/pref-manager.js,v  <--  pref-manager.js
new revision: 1.15; previous revision: 1.14
done
Checking in mozilla/extensions/irc/js/lib/text-serializer.js;
/cvsroot/mozilla/extensions/irc/js/lib/text-serializer.js,v  <--  text-serializer.js
new revision: 1.5; previous revision: 1.4
done
Checking in mozilla/extensions/irc/js/lib/utils.js;
/cvsroot/mozilla/extensions/irc/js/lib/utils.js,v  <--  utils.js
new revision: 1.65; previous revision: 1.64
done
Checking in mozilla/extensions/irc/xul/content/config.js;
/cvsroot/mozilla/extensions/irc/xul/content/config.js,v  <--  config.js
new revision: 1.11; previous revision: 1.10
done
Checking in mozilla/extensions/irc/xul/content/handlers.js;
/cvsroot/mozilla/extensions/irc/xul/content/handlers.js,v  <--  handlers.js
new revision: 1.137; previous revision: 1.136
done
Checking in mozilla/extensions/irc/xul/content/networks.js;
/cvsroot/mozilla/extensions/irc/xul/content/networks.js,v  <--  networks.js
new revision: 1.2; previous revision: 1.1
done
Checking in mozilla/extensions/irc/xul/content/static.js;
/cvsroot/mozilla/extensions/irc/xul/content/static.js,v  <--  static.js
new revision: 1.210; previous revision: 1.209
done
Checking in mozilla/extensions/irc/xul/lib/munger.js;
/cvsroot/mozilla/extensions/irc/xul/lib/munger.js,v  <--  munger.js
new revision: 1.21; previous revision: 1.20
done
Checking in mozilla/extensions/irc/xul/lib/tree-utils.js;
/cvsroot/mozilla/extensions/irc/xul/lib/tree-utils.js,v  <--  tree-utils.js
new revision: 1.3; previous revision: 1.2
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.76]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: