1586250 - 'Method not implemented' when calling method: [prplIAccount::unInit]

Status: RESOLVED FIXED

(Chat Core :: Yahoo! Messenger, defect)

Description

[Patrick Cloke [:clokep]]

Attachment: Patch v1

If you have an old Yahoo or Facebook account, an error is thrown to the console when quitting:

JavaScript error: file:///Users/pcloke/mozilla/mozilla-central/obj-x86_64-apple-darwin18.7.0-tb/dist/Thunderbird%20Daily.app/Contents/Resources/components/imAccounts.js, line 786: NS_ERROR_NOT_IMPLEMENTED: Method not implemented'Method not implemented' when calling method: [prplIAccount::unInit]
JavaScript error: file:///Users/pcloke/mozilla/mozilla-central/obj-x86_64-apple-darwin18.7.0-tb/dist/Thunderbird%20Daily.app/Contents/Resources/components/imAccounts.js, line 786: NS_ERROR_NOT_IMPLEMENTED: Method not implemented'Method not implemented' when calling method: [prplIAccount::unInit]

The fix is easy, we just need to implement an unInit that does nothing.

Comment 1

[Patrick Cloke [:clokep]]

I also debated if it made sense to have an AbandonedAccount implementation for dead protocols, but it doesn't seem like much code would be shared between Facebook and Yahoo. The vast majority of the code is custom error messages.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/1da5ee1ef0ea
'Method not implemented' when calling method: [prplIAccount::unInit]. r=florian

Comment 3

[Jorg K (CEST = GMT+2)]

Hi Partrick, it would be good, if you could leave smaller patches for me to check in. M-C merges four times a day, so I need a patch for times a day. Currently Paul is not available and hence not producing patches, and Magnus is on PTO and hence also not producing patches or reviewing patches. Right now, for the merges in about two and seven hours I have nothing. This patch and the one from bug 1427205 would have served me very well for my sheriffing purposes.

Comment 4

[Patrick Cloke [:clokep]]

Comment on attachment 9098808 [details] [diff] [review]
Patch v1

[Approval Request Comment]
Regression caused by (bug #): N/A
User impact if declined: Errors show up in the console if an old Facebook/Yahoo account exists.
Testing completed (on c-c, etc.): Has been on c-c for a while.
Risk to taking this patch (and alternatives if risky): I don't think there's much risk, this is adding an empty method.

Comment 5

[Jorg K (CEST = GMT+2)]

TB 68.2.0 ESR:
https://hg.mozilla.org/releases/comm-esr68/rev/cf778585a395ba5cecfcffe949a83ce97132d73c