Open Bug 1196851 Opened 9 years ago Updated 2 years ago

Better documentation and assertions for the LoginManagerContent _fillForm method.

Categories

(Toolkit :: Password Manager, enhancement, P3)

enhancement

Tracking

()

Tracking Status
firefox43 --- affected

People

(Reporter: rittme, Unassigned)

References

Details

(Whiteboard: [passwords:tech-debt])

The LoginManagerContent _fillForm method have lots of different parameters and we are not sure every combination of arguments would work or fail in an expected way. 

We should check the possible combinations and add more documentation and assertions so the function caller knows what to expect.
Flags: qe-verify-
Whiteboard: [passwords:tech-debt]
Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.