Created attachment 8516964 [details] foo.html bug 1080030 broke the money transfer feature on bankofamerica.com if e10s is enabled. The reason is that an optgroup w/o a label attribute causes the select from being populated, so you don't see any options when you try to open the select. I'm attaching a minimized testcase and a fix, but if anyone has ideas on how to add automated tests that would catch this, let me know.
Created attachment 8516973 [details] [diff] [review] Fix.
Attachment #8516973 - Flags: review?(felipc)
Attachment #8516973 - Flags: review?(felipc) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c12d13fb6570 Ehsan, I'm guessing this would've been tested had our chrome tests been enabled in e10s builds. Can you look into that? If not, it'd be great if we had a way to test this stuff... Thanks.
Assignee: nobody → jst
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
(In reply to Johnny Stenback (:jst, firstname.lastname@example.org) from comment #2) > https://hg.mozilla.org/integration/mozilla-inbound/rev/c12d13fb6570 > > Ehsan, I'm guessing this would've been tested had our chrome tests been > enabled in e10s builds. Can you look into that? If not, it'd be great if we > had a way to test this stuff... Thanks. Last I heard about our testing plans (I forget from who) we were hoping to eventually run all of our tests in e10s. I'm not sure if we have any way of testing something e10s specific in the middle of of our normal test suites. Brad, do you know who can confirm this, please?
Flags: needinfo?(ehsan.akhgari) → needinfo?(blassey.bugs)
I'm not sure that I understand your question. We can turn tests on and off in the manifest based on e10s being enabled or not.
Nevermind, seems like the status of e10s testing has improved a lot since the last time I looked. So I tried to write a test for this, but hit bug 933103 and finally gave up. As far as I can tell, all of this stuff is completely untested at this point. :/
You need to log in before you can comment on or make changes to this bug.