Closed Bug 1232422 Opened 9 years ago Closed 9 years ago

Convert 5 tests within netwerk/test to use AsyncOpen2

Categories

(Core :: DOM: Security, defect)

Product:
Core
Component:
DOM: Security
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla46
Tracking Flags:
Tracking Status
firefox46 --- fixed

People

(Reporter: ckerschb, Assigned: ckerschb)

References

Details

Attachments

(1 file)

bug_1232422_convert_5_callsites_within_netwerk_test.patch
40.88 KB, patch
mcmanus
: review+
Details | Diff | Splinter Review
No description provided.
Assignee: nobody → mozilla
Blocks: 1193558
Status: NEW → ASSIGNED
Hey Pat, as briefly discussed I think it makes sense that we try to use the NetUtil.newChannel API consistently wherever we can. As for the general plan, I think it makes sense to always bundle 5 tests within one bug and convert those callsites. Some of them are harder then others. Overall, we should have all tests converted fairly soon. Thanks!
Attachment #8698156 - Flags: review?(mcmanus)
Attachment #8698156 - Flags: review?(mcmanus) → review+
https://hg.mozilla.org/mozilla-central/rev/ca3a2722e69d
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: