1800344 - Simplify nsSyncStreamListener instantiation APIs

Status: RESOLVED FIXED

(Core :: Networking, task)

Description

[Daniel Holbert [:dholbert]]

Once we have bug 1800342's changes to make NS_NewPipe infallible, we can remove a few of the wrapper functions that get used for nsSyncStreamListener construction.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: Bug 1800344 part 1: Fold nsSyncStreamListener::Init into the constructor. r?#necko-reviewers

Depends on D161947

Comment 2

[Daniel Holbert [:dholbert]]

Attachment: Bug 1800344 part 2: Fold nsSyncStreamListener::Create() into NS_NewSyncStreamListener(). r?#necko-reviewers

While we're at it: remove one layer of unnecessary error handling in
NS_NewSyncStreamListener, since it's clearer now that we'll be getting
infallible allocation.

Depends on D161948

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d10c1c4ca176
part 1: Fold nsSyncStreamListener::Init into the constructor. r=necko-reviewers,kershaw
https://hg.mozilla.org/integration/autoland/rev/57ea39b22985
part 2: Fold nsSyncStreamListener::Create() into NS_NewSyncStreamListener(). r=necko-reviewers,kershaw

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/d10c1c4ca176
https://hg.mozilla.org/mozilla-central/rev/57ea39b22985