Closed Bug 492725 Opened 16 years ago Closed 16 years ago

make chrome-based website authenticator use core Firefox form fill notifications

Categories

(Mozilla Labs :: Identity, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: myk, Assigned: myk)

References

Details

Attachments

(1 file, 1 obsolete file)

Currently the chrome-based website authenticator uses a bunch of code copied over from the login manager. In bug 492153 I'm working on making the login manager notify observers when it finds logins for a form, which would obviate the need for Weave to include the copied code. Weave should switch to the notification once that bug is fixed. Here's a patch that seems to work against a copy of Firefox with my latest patch in bug 492153.
This patch is updated to work with the latest version of the core browser patch, which now passes form info in an nsIPropertyBag2 instead of a wrapped JS object.
Assignee: nobody → myk
Attachment #377121 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Component: Weave → General
Product: Mozilla Labs → Weave
Target Milestone: -- → ---
Component: Weave → General
Product: Mozilla Labs → Weave
Target Milestone: -- → ---
QA Contact: weave → general
Component: General → Identity
QA Contact: general → identity
Target Milestone: --- → 0.4
Product: Mozilla Services → Mozilla Labs
Target Milestone: 0.4 → --
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: