1713796 - Refactor nsHostResolver

Status: RESOLVED FIXED

(Core :: Networking: DNS, task, P2)

Description

[Kershaw Chang [:kershaw]]

I'd like to refactor nsHostResolver a bit.
Things to be done in this bug would be:

• Move the implementation of nsHostRecord to another file.
• Move the logic of en/dequeue to another file.
• Make nsHostResolver::ResolveHost function a bit short by moving the code to another function.

Comment 1

[Kershaw Chang [:kershaw]]

Attachment: Bug 1713796 - P1: Move nsHostRecord to another file, r=#necko

Comment 2

[Kershaw Chang [:kershaw]]

Attachment: Bug 1713796 - P2: Take some code from nsHostResolver::ResolveHost and put them to other functions, r=#necko

Depends on D117322

Comment 3

[Kershaw Chang [:kershaw]]

Attachment: Bug 1713796 - P3: Move the code related to pending queue out of nsHostResolver, r=#necko

Depends on D117323

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9fd0ba528af1
P1: Move nsHostRecord to another file, r=necko-reviewers,valentin
https://hg.mozilla.org/integration/autoland/rev/38ca3b251cac
P2: Take some code from nsHostResolver::ResolveHost and put them to other functions, r=necko-reviewers,valentin
https://hg.mozilla.org/integration/autoland/rev/52a67ce78459
P3: Move the code related to pending queue out of nsHostResolver, r=necko-reviewers,valentin

Comment 5

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/9fd0ba528af1
https://hg.mozilla.org/mozilla-central/rev/38ca3b251cac
https://hg.mozilla.org/mozilla-central/rev/52a67ce78459