At least a partial list: http://lxr.instantbird.org/instantbird/search?string=prototype.__proto__

Apparently Florian had provided a complete list and I missed it: chat/modules/imContentSink.jsm:194: aResult.tags.__proto__ = aBase.tags; chat/modules/imContentSink.jsm:195: aResult.attrs.__proto__ = aBase.attrs; chat/modules/imContentSink.jsm:196: aResult.styles.__proto__ = aBase.styles; chat/modules/imThemes.jsm:92: this.__proto__ = HTMLTheme_prototype; chat/protocols/jsTest/jsTestProtocol.js:34:Conversation.prototype.__proto__ = GenericConvIMPrototype; chat/protocols/jsTest/jsTestProtocol.js:72:Account.prototype.__proto__ = GenericAccountPrototype; chat/protocols/jsTest/jsTestProtocol.js:93:jsTestProtocol.prototype.__proto__ = GenericProtocolPrototype; chat/protocols/yahoo/yahoo.js:75:YahooConversation.prototype.__proto__ = GenericConvIMPrototype; chat/protocols/yahoo/yahoo.js:143:YahooConference.prototype.__proto__ = GenericConvChatPrototype; chat/protocols/yahoo/yahoo.js:155:YahooConferenceBuddy.prototype.__proto__ = GenericConvChatBuddyPrototype; chat/protocols/yahoo/yahoo.js:170:YahooAccountBuddy.prototype.__proto__ = GenericAccountBuddyPrototype; chat/protocols/yahoo/yahoo.js:522:YahooAccount.prototype.__proto__ = GenericAccountPrototype; im/components/ibConvStatsService.js:249: stats.__proto__ = ConversationStats.prototype; im/content/debug/fake/fake.js:376: this.__proto__ = aChatNick ? GenericConvChatPrototype : GenericConvIMPrototype;

Looks like the following are left: > chat/modules/imContentSink.jsm:194: aResult.tags.__proto__ = aBase.tags; > chat/modules/imContentSink.jsm:195: aResult.attrs.__proto__ = aBase.attrs; > chat/modules/imContentSink.jsm:196: aResult.styles.__proto__ = aBase.styles; > im/components/ibConvStatsService.js:249: stats.__proto__ = ConversationStats.prototype; > im/content/debug/fake/fake.js:376: this.__proto__ = aChatNick ? GenericConvChatPrototype : GenericConvIMPrototype;

(In reply to Patrick Cloke [:clokep] from comment #4) > > chat/modules/imContentSink.jsm:194: aResult.tags.__proto__ = aBase.tags; > > chat/modules/imContentSink.jsm:195: aResult.attrs.__proto__ = aBase.attrs; > > chat/modules/imContentSink.jsm:196: aResult.styles.__proto__ = aBase.styles; I'll try to have a look at these. > > im/components/ibConvStatsService.js:249: stats.__proto__ = ConversationStats.prototype; nhnt11, can you look into this one? > > im/content/debug/fake/fake.js:376: this.__proto__ = aChatNick ? GenericConvChatPrototype : GenericConvIMPrototype; This one seems relatively easy. I think we can use Object.create, like the warning suggests.

Anyone will to take a look at Fake or do we just not care enough since it isn't in the released product?

Note: setting the review flag on clokep as he asked about Fake, but anybody should feel free to steal that trivial review. (In reply to Patrick Cloke [:clokep] from comment #8) > Anyone will to take a look at Fake or do we just not care enough since it > isn't in the released product? I'm hoping that Fake could become a basis for some future UI tests, so I would prefer that we don't break it. I've just had a quick look. Untested (but I don't think it's worth testing right now).

Uploading an unbitrotted version of attachment 8365019 [details] [diff] [review]. These should all apply cleanly for checkin now. (Although the tree is closed.)

https://hg.mozilla.org/comm-central/rev/4b91da005116 https://hg.mozilla.org/comm-central/rev/927179691f40 https://hg.mozilla.org/comm-central/rev/c94223070d13 https://hg.mozilla.org/comm-central/rev/2a38a2ea7e97 https://hg.mozilla.org/comm-central/rev/25d3c13a55b9