775817 - Update tests to use the new permission manager api

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Mounir Lamouri (:mounir)]

This patch should updates [nearly] all tests using the permission manager. Tests that this patch are not updating are:
- tests using SpecialPowers (there are a couple of them), see bug 775815;
- tests using Services.perms;
- tests I missed.

I will try to write a new patch for the second category.

Comment 1

[Mounir Lamouri (:mounir)]

Attachment: Patch

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 644138 [details] [diff] [review]
Patch

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

r=me with that fixed.

::: netwerk/test/unit/test_fallback_request-error_passing.js
@@ +69,5 @@
>    var pm = Cc["@mozilla.org/permissionmanager;1"]
>      .getService(Ci.nsIPermissionManager);
>    var uri = make_uri("http://localhost:4444");
> +  var principal = CC["@mozilla.org/scriptsecuritymanager;1"]
> +                    .getService(Ci.nsIScriptSecurityManager)

All of these, change to just Services.perms

Comment 3

[Mounir Lamouri (:mounir)]

This is not updating all tests but quite most of them. We will open new bugs for the others.

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/0ff084e6a9e3