521524 - Port |Bug 469443 - Form Manager Storage should be a JavaScript-based component| to comm-central (apps)

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Serge Gautherie (:sgautherie)]

Thunderbird fixed a part of it in bug 514665. (I didn't investigate further...)

SeaMonkey needs it.

(Fwiw, I'm not sure if calendar should be updated too or not...)

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1-SM) SeaMonkey part, Improve sort order [Checkin: Comment 3]

Comment 2

[Robert Kaiser]

Comment on attachment 405613 [details] [diff] [review]
(Av1-SM) SeaMonkey part, Improve sort order
[Checkin: Comment 3]

>+#ifdef MOZILLA_1_9_1_BRANCH
> pref("browser.formfill.enable",             true);
>+#endif
> pref("browser.formfill.expire_days",        180);

Wait a minute, does this mean that trunk doesn't support the .enable pref any more? IIRC we have a UI pref for it, we need to deal with this in that case...

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 405613 [details] [diff] [review]
(Av1-SM) SeaMonkey part, Improve sort order
[Checkin: Comment 3]


http://hg.mozilla.org/comm-central/rev/d7b7ee5682fd


(In reply to comment #2)
> Wait a minute, does this mean that trunk doesn't support the .enable pref any
> more?

No but the pref moved to all.js from firefox.js.

Comment 4

[[e-mail]]

(In reply to comment #3)
>(In reply to comment #2)
>>Wait a minute, does this mean that trunk doesn't support the .enable pref any more?
>No but the pref moved to all.js from firefox.js.
Yay, fewer toolkit prefs in firefox.js :-)

Comment 5

[Robert Kaiser]

Is there anything still left to do here? If not, let's resolve it...

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to comment #5)

Let me check:
*SB is "unsupported" on m-1.9.2+: I don't care anymore.
*SM is fixed.
*TB is partially fixed only, but they don't care, so why should I any longer?