986685 - [e10s] Disable some DOM tests

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

I'm working on disabling enough tests that we can get a green e10s run. After that, we can start enabling them again.

Comment 1

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Attachment: disable-dom-events

Comment 2

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Attachment: disable-pointer-lock

Comment 3

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Attachment: disable-dom-imptest

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8395050 [details] [diff] [review]
disable-dom-events

I don't understand this one. This should just work.
What kind of error do you get?

I assume this is about test_bug432698.html
.ini syntax is so bizarre

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8395052 [details] [diff] [review]
disable-dom-imptest

odd odd, but given that this is disabled on gonk too, fine.

Comment 6

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

(In reply to Olli Pettay [:smaug] from comment #4)
> Comment on attachment 8395050 [details] [diff] [review]
> disable-dom-events
> 
> I don't understand this one. This should just work.
> What kind of error do you get?
> 
> I assume this is about test_bug432698.html
> .ini syntax is so bizarre

I haven't been investigating too much. I just want to start with a clean baseline. Here is the failure:
https://tbpl.mozilla.org/php/getParsedLog.php?id=36479161&tree=Holly#error0
It seems to happen intermittently.

Comment 7

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

(In reply to Olli Pettay [:smaug] from comment #5)
> Comment on attachment 8395052 [details] [diff] [review]
> disable-dom-imptest
> 
> odd odd, but given that this is disabled on gonk too, fine.

This test uses the <keygen> tag, which requires window.crypto, which is disabled in e10s. Apparently some part of NSS/PSM is not safe to run in content processes.

Comment 8

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8395050 [details] [diff] [review]
disable-dom-events


In case of test_bug432698.html I think the following could fix the test
checkRelatedTarget = false;
right before SimpleTest.finish();

Either that, or disable.


Disabling test_bug967796.html is fine.

Comment 9

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/083592b92e63

I made the "checkRelatedTarget = false" change. It passes on inbound. I'm not sure if it works in e10s since it's intermittent, but we'll find out in time.

Comment 10

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/083592b92e63