bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

[e10s] Enable test_autofill_password-only.html

RESOLVED FIXED in Firefox 48

Status

()

Toolkit
Password Manager
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: MattN, Assigned: MattN)

Tracking

(Blocks: 1 bug)

unspecified
mozilla48
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(e10s+, firefox48 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

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.
Created attachment 8723417 [details]
MozReview Request: Bug 1251151 - Make test_autofill_password-only.html work for e10s. r=dolske

Review commit: https://reviewboard.mozilla.org/r/36545/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/36545/
Attachment #8723417 - Flags: review?(dolske)
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/

Updated

2 years ago
tracking-e10s: --- → +
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...

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/26e576786f9d
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.