This is a meta bug of issues relating to migration of autocomplete in TB to toolkit

Nominating wanted-thunderbird3 to keep it on the radar.

I think the general opinion is that shipping both xpfe and toolkit implementations isn't the way to go, instead, moving completely to toolkit's autocomplete. I'm fairly sure Paul isn't doing much these days, so taking bug.

Mark, do you think you can get this in by b1?

What bug tracks changing mozilla/toolkit/content/xul.css to not have a special "ifdef MOZ_THUNDERBIRD" which results in "define AUTOCOMPLETE_OLD_STYLE", and modifies mozilla/toolkit/components/Makefile.in to add "autocomplete" to DIRS instead of just "autocomplete/public"? Without this, we can't have the pretty sweet richlist implementation. (Granted, it has a browser bias, but it does lots of neat stuff.)

bug 360648 ?

oh, sorry, heh. Mark would know.

(In reply to comment #4) > What bug tracks changing mozilla/toolkit/content/xul.css to not have a special > "ifdef MOZ_THUNDERBIRD" which results in "define AUTOCOMPLETE_OLD_STYLE", and > modifies mozilla/toolkit/components/Makefile.in to add "autocomplete" to DIRS > instead of just "autocomplete/public"? This one! I've just not posted a patch yet...

This patch removes the old xpfe autocomplete based local address book search (which is currently kept for calendar integration).

This patch removes the ldap autocomplete so that we can build with just toolkit interfaces.

This patch actually switches TB to use toolkit autocomplete - it is a mozilla-central patch.

Switching for b1 flags to target milestones, to avoid flag churn.

3.0b1 flag is going away in favour of 3.0 flag and milestone combination.

TB would like to enable building of the toolkit autocomplete code (not just the interfaces) ready for beta 1. This is a simple toolkit makefile patch - the first block just joins up two lots of DIRS = statements that don't need to be separate. The second block enables building of all of autocomplete for Thunderbird. Although this doesn't enable the bindings, this does do enough for TB devs to play around with things in extensions and hopefully show them off in TB 3 beta 1.

Comment on attachment 339844 [details] [diff] [review] [checked in] Enable building of toolkit autocomplete for Thunderbird (not enabling the UI!) > > ifdef MOZ_XUL >-DIRS += \ >- autocomplete \ >- satchel \ >- $(NULL) >+DIRS += satchel > endif # MOZ_XUL Seems like it would be more readable if you just coalesced this into the MOZ_XUL block above it. r=dmose from the "Thunderbird changes" point of view

(In reply to comment #14) > (From update of attachment 339844 [details] [diff] [review]) > > > > > ifdef MOZ_XUL > >-DIRS += \ > >- autocomplete \ > >- satchel \ > >- $(NULL) > >+DIRS += satchel > > endif # MOZ_XUL > > Seems like it would be more readable if you just coalesced this into the > MOZ_XUL block above it. Except that there are various other directories that are ifndef MOZ_THUNDERBIRD, so it doesn't really clean much up, and would only add an extra MOZ_THUNDERBIRD instead of a MOZ_XUL.

This isn't making beta 1... moving out.

Comment on attachment 339844 [details] [diff] [review] [checked in] Enable building of toolkit autocomplete for Thunderbird (not enabling the UI!) This was checked in ages ago (September): http://hg.mozilla.org/mozilla-central/rev/0e49b86be866

This doesn't block the current features we want to get into Thunderbird 3, hence no longer needs to block TB 3. Still very much wanted. Reassigning to default owner as I haven't got time to work on this at the moment. See status whiteboard for what needs fixing before this can be fixed. Once they are fixed then we'll need some build config changes in mozilla-central to build toolkit rather than xpfe autocomplete and include the appropriate style sheets.

Port the autocomplete module to use frozen linkages.

Ooops ... I think I'm wrong with this patch. Sorry.

(In reply to comment #19) > Created an attachment (id=369584) [details] > Convert autocomplete to frozen/external linkages (In reply to comment #20) > Ooops ... I think I'm wrong with this patch. Sorry. Well if you want to move xpfe autocomplete to frozen linkage you can do so in a separate bug. However we'd really like to just drop xpfe autocomplete completely and switch to toolkit autocomplete (so we can further align ourselves with core) - which is what this bug (and bug 452232) are about.

Does this bug still depend on bug 452232? All the other dependencies are now resolved. Does not having LDAP autocomplete in toolkit prevent switching TB to toolkit autocomplete?

(In reply to comment #22) > Does this bug still depend on bug 452232? All the other dependencies are now > resolved. Does not having LDAP autocomplete in toolkit prevent switching TB to > toolkit autocomplete? Yes it does depend on bug 452232 - until LDAP autocomplete is switched over to the toolkit interfaces, we can't do the full switch of TB to the toolkit autocomplete. Note that xpfe supports the toolkit interfaces so that you can still use xpfe autocomplete in the same way as toolkit.

Thunderbird is now fully migrated, yay!