Created attachment 606530 [details] [diff] [review]
This error turns out to be because nsMsgAccountManager::GetIncomingServer gets a .userName pref that imIncomingServer.js saves as .username.
The JS stack printed by the debugger; keyword was completely useless here (it ended in gloda). I found the problem by putting a gdb breakpoint on xpc_DebuggerKeywordHandler, which gave me this stack (I removed the frames before and after which are in the JS engine):
#15 0x06c712b0 in nsCOMPtr<nsIMsgIncomingServer>::operator-> () at nsCOMPtr.h:589
#16 nsMsgAccountManager::createKeyedServer (this=0x1d089d10, key=@0xbfff884c, username=@0xbfff8760, hostname=@0xbfff8754, type=@0xbfff876c, aServer=0xbfff8874) at /Users/florian/buildhg/comm-central/mailnews/base/src/nsMsgAccountManager.cpp:590
#17 0x06c71f41 in nsMsgAccountManager::GetIncomingServer (this=0x1d089d10, key=@0xbfff884c, _retval=0xbfff8874) at /Users/florian/buildhg/comm-central/mailnews/base/src/nsMsgAccountManager.cpp:486
#18 0x06c7cf4f in nsMsgAccount::createIncomingServer (this=0x1d08da40) at /Users/florian/buildhg/comm-central/mailnews/base/src/nsMsgAccount.cpp:133
#19 0x06c7d365 in nsMsgAccount::GetIncomingServer (this=0x1d08da40, aIncomingServer=0xbfff8adc) at /Users/florian/buildhg/comm-central/mailnews/base/src/nsMsgAccount.cpp:96
#20 0x06c67b8c in nsCOMPtr<nsIMsgIncomingServer>::get () at nsCOMPtr.h:1396
#21 0x06c67b8c in ~nsGetterAddRefs [inlined] () at nsCOMPtr.h:1296
#22 nsMsgAccountManager::LoadAccounts (this=0x1d089d10) at nsCOMPtr.h:1396
#23 0x06c626ca in nsMsgAccountManager::GetAllIdentities (this=0x1d089d10, _retval=0xbfff8e80) at /Users/florian/buildhg/comm-central/mailnews/base/src/nsMsgAccountManager.cpp:1100
I wonder if this could be somehow related to bug 736035.
Comment on attachment 606530 [details] [diff] [review]
[Approval Request Comment]
This is mostly a patch to reduce the noise on the error console, but I would like to take it on aurora as not having this fix causes us to set a pref with the wrong name in the user's profile.