1733922 - Specialize TransportSetup into Primary/BackupTransportSetup

Status: RESOLVED FIXED

(Core :: Networking, enhancement, P2)

Description

[Valentin Gosu [:valentin] (he/him) {{ AFK in July }}]

This way we avoid passing mIsBackup as a boolean to each constructor in DnsAndConnectSocket()

Comment 1

[Valentin Gosu [:valentin] (he/him) {{ AFK in July }}]

Attachment: Bug 1733922 - Specialize TransportSetup into Primary/BackupTransportSetup r=#necko

Depends on D127407

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/293ff1a22c98
Specialize TransportSetup into Primary/BackupTransportSetup r=necko-reviewers,dragana

Comment 3

[Cristian Tuns]

Backed out for causing hybrid build bustages on nsAboutProtocolHandler.cpp

• Backout link
• Push with failures
• Failure Log

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/85b747a60e73
Specialize TransportSetup into Primary/BackupTransportSetup r=necko-reviewers,dragana

Comment 5

[Cosmin Sabou [:CosminS]]

Backed out for causing windows hybrid bustages.

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=85b747a60e7348d3c650c4188275160839983f86&selectedTaskRun=Gn8hyQMdQcqBkSwQEYWEdw.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=354694952&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/c274f0061d801591fa48c44c052f1b52d56b5335

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/57211d4fefc5
Specialize TransportSetup into Primary/BackupTransportSetup r=necko-reviewers,dragana

Comment 7

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/57211d4fefc5