Use Services.io instead of Cc["@mozilla.org/network/io-service;1"].getService(Ci.nsIIOService) in xpcshell test

RESOLVED FIXED in Thunderbird 18.0

Status

MailNews Core
Testing Infrastructure
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: hiro, Assigned: hiro)

Tracking

(Blocks: 1 bug)

Trunk
Thunderbird 18.0
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 643706 [details] [diff] [review]
Fix
Attachment #643706 - Flags: review?(mconley)
(Assignee)

Updated

5 years ago
Summary: Use Services.io instead of Cc["@mozilla.org/network/io-service;1"].getService(Ci.nsIIOService) → Use Services.io instead of Cc["@mozilla.org/network/io-service;1"].getService(Ci.nsIIOService) in xpcshell test
(Assignee)

Comment 1

5 years ago
I just pushed a try:

https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=98e8fc1442a3

Comment 2

5 years ago
Nice work :)
Blocks: 720356
Comment on attachment 643706 [details] [diff] [review]
Fix

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

This is awesome - thanks Hiro!

I've only inspected these tests, and haven't run the patch. I assume these all run correctly?

Assuming that, and with my one little nit fixed, r=me. Great work!

::: mailnews/news/test/unit/test_bug37465.js
@@ +8,5 @@
>    server.start(NNTP_PORT);
>  
>    // Correct URI?
> +  let uri = Services.io.newURI("news://localhost:1143/1@regular.invalid",
> +                               null,

Let's put this last null on the same line as the one before.
Attachment #643706 - Flags: review?(mconley) → review+
(Assignee)

Comment 4

5 years ago
Created attachment 645910 [details] [diff] [review]
null position fix

The last try server result had some build errors.

Now I pushed to a try again for the safety.
Assignee: nobody → hiikezoe
Attachment #643706 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #645910 - Flags: review+
What's the status on this ?
Hm - I guess it never got checkin-needed.

This is about a month old - I'll make sure the patch still applies, and then set the flag.
Ok, this still looks good.
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/2bdd449748dd
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
You need to log in before you can comment on or make changes to this bug.