296974 - mailing list resolution code walks all lists in all addressbooks

Status: NEW

(Thunderbird :: Message Compose Window, defect)

Description

[Dan Mosedale (:dmosedale, :dmose)]

This is going to have scaling problems as soon as we start autocompleting
against LDAP addressbooks for lists.  Instead of re-resolving at send time, we
could simply tweak nsMsgCompFields to have a higher-level representation of what
a message recipient is than a string which has to be parsed and reserialized at
every touch.  Proposed interface changes forthcoming.

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

Attachment: proposed interface changes, v1

This will allow the autocomplete code to simply attach the nsIAbCard of the
relevant list to the recipient field in question, so that when resolving at
send time, it doesn't have to walk the addressbooks.  Thoughts?

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 185580 [details] [diff] [review]
proposed interface changes, v1

OK, so while the interface changes I've proposed above may turned out to be a
useful optimization for a later time, they won't be sufficient to solve the
core problem here.  In order to fix the core issues, the walking of
addressbooks, requires that a bunch code that is current synchronous needs to
become async.

Comment 3

[Dan Mosedale (:dmosedale, :dmose)]

The switch that will need to happen here is to using nsIAbDirectoryQuery and
friends to find out about these lists.  Making this fix may inherently fix 12409
without any additional work.

Comment 4

[Dan Mosedale (:dmosedale, :dmose)]

Assigning bugs that I'm not actively working on back to nobody; use
SearchForThis as a search term if you want to delete all related bugmail at
once.