1222562 - Fix dom/workers/test/test_xhr_parameters.html and test_tcpsocket_enabled_with_perm.html to work with e10s

Status: RESOLVED FIXED

(Core :: DOM: Workers, defect)

Description

[Andrew McCreight [:mccr8]]

It uses SpecialPowers.addPermission() instead of SpecialPowers.pushPermissions().

Comment 1

[Andrew McCreight [:mccr8]]

test_tcpsocket_enabled_with_perm.html has the same problem, and they are both simple fixes.

Comment 2

[Andrew McCreight [:mccr8]]

Attachment: part 1 - Fix use of addPermission in test_tcpsocket_enabled_with_perm.html and test_xhr_parameters.html.

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=002fe94dc8fe

Comment 3

[Andrew McCreight [:mccr8]]

Attachment: part 2 - Add some comments about failing tests.

Comment 4

[Andrew McCreight [:mccr8]]

Attachment: part 3 - Enable a single test in uriloader.

This test passes with e10s. The other two do not.

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8685467 [details] [diff] [review]
part 1 - Fix use of addPermission in test_tcpsocket_enabled_with_perm.html and test_xhr_parameters.html.

Review of attachment 8685467 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/network/tests/test_tcpsocket_enabled_with_perm.html
@@ +17,5 @@
>    the tcp-socket permission has been granted.
>  **/
>  SimpleTest.waitForExplicitFinish();
> +SpecialPowers.pushPrefEnv({"set": [['dom.mozTCPSocket.enabled', true]]}, function() {
> +  SpecialPowers.pushPermissions([{type: "tcp-socket", allow: true, context: document}], runTest); });

Nit: I'd indent this as:

SpecialPowers.pushprefEnv(..., function() {
  SpecialPowers.pushPermissions(...);
});

Comment 6

[Andrew McCreight [:mccr8]]

(In reply to Blake Kaplan (:mrbkap) (please use needinfo!) from comment #5)
> Nit: I'd indent this as:

Oh yeah, that's less weird. Fixed.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0a2ab9e74af2
https://hg.mozilla.org/integration/mozilla-inbound/rev/4796c2145d33
https://hg.mozilla.org/integration/mozilla-inbound/rev/e915959648c4

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/0a2ab9e74af2
https://hg.mozilla.org/mozilla-central/rev/4796c2145d33
https://hg.mozilla.org/mozilla-central/rev/e915959648c4