Closed Bug 1251151 Opened 5 years ago Closed 5 years ago

[e10s] Enable test_autofill_password-only.html

Categories

(Toolkit :: Password Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
e10s + ---
firefox48 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Make test_autofill_password-only.html work in e10s. This will be the first converted password manager mochitest related to autofill hopefully laying the foundation for others doing similar things.

Depends on bug 1153128 for sendSyncMessage on loadChromeScript.
Comment on attachment 8723417 [details]
MozReview Request: Bug 1251151 - Make test_autofill_password-only.html work for e10s. r=dolske

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/36545/diff/1-2/
Comment on attachment 8723417 [details]
MozReview Request: Bug 1251151 - Make test_autofill_password-only.html work for e10s. r=dolske

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/36545/diff/2-3/
Attachment #8723417 - Flags: review?(dolske) → review+
Comment on attachment 8723417 [details]
MozReview Request: Bug 1251151 - Make test_autofill_password-only.html work for e10s. r=dolske

https://reviewboard.mozilla.org/r/36545/#review33579

::: toolkit/components/passwordmgr/test/mochitest/test_autofill_password-only.html:51
(Diff revision 3)
> +  addMessageListener("removeLogins", function removeLogins() {

Could make this a cleanup function instead...
https://hg.mozilla.org/mozilla-central/rev/26e576786f9d
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.