1251151 - [e10s] Enable test_autofill_password-only.html

Status: RESOLVED FIXED

(Toolkit :: Password Manager, defect)

Description

[Matthew N. [:MattN]]

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 1

[Matthew N. [:MattN]]

Attachment: 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/

Comment 2

[Matthew N. [:MattN]]

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 3

[Matthew N. [:MattN]]

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/

Comment 4

[Justin Dolske [:Dolske]]

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 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/26e576786f9d

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/26e576786f9d