1588412 - nsHTTPChannel doesn't need to perform process-switching anymore

Status: RESOLVED FIXED

(Core :: Networking, task, P3)

Description

[Paul Bone [:pbone]]

The calls to onMayChangeProcess can be removed from nsHTTPChannel. they're dead code and are unused here.

Comment 1

[Paul Bone [:pbone]]

Attachment: Bug 1588412 - (part 1) Fix grammer in comment r=mayhemer

Comment 2

[Paul Bone [:pbone]]

Attachment: Bug 1588412 - (part 3) Remove MayChangeProcess calls fron nsHttpChannel r=mayhemer,jya

Depends on D49100

Comment 3

[Paul Bone [:pbone]]

Attachment: Bug 1588412 - (part 2) moz:// documents should use documentchannel r=nika

the URL moz://a redirects to mozilla's website, Because it redirects to
https we want to use documentchannel.

Depends on D49100

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/595f252ef63d
(part 1) Fix grammer in comment r=mayhemer
https://hg.mozilla.org/integration/autoland/rev/e0e3a3390f76
(part 2) moz:// documents should use documentchannel r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/55c0d49b821c
(part 3) Remove MayChangeProcess calls fron nsHttpChannel r=mayhemer

Comment 5

[Paul Bone [:pbone]]

Attachment: Bug 1588412 - Fix a spelling error r=mattwoodrow

The already-landed patches contained a spelling error. This patch fixes it.

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/595f252ef63d
https://hg.mozilla.org/mozilla-central/rev/e0e3a3390f76
https://hg.mozilla.org/mozilla-central/rev/55c0d49b821c

Comment 7

[Mike Conley (:mconley) (:⚙️)]

Hey mattwoodrow - was e0e3a3390f76 supposed to fix browsing to moz://a? Because I just pulled these patches down, and the redirect doesn't appear to be working... or is there something else I need to do in order to enable DocumentChannel process switching?

Comment 8

[Matt Woodrow (:mattwoodrow)]

(In reply to Mike Conley (:mconley) (:⚙️) (Wayyyy behind on needinfos) from comment #7)

Hey mattwoodrow - was e0e3a3390f76 supposed to fix browsing to moz://a? Because I just pulled these patches down, and the redirect doesn't appear to be working... or is there something else I need to do in order to enable DocumentChannel process switching?

Replied in bug 1591942.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/aafdd02eb30a
Fix a spelling error r=mattwoodrow

Comment 10

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/aafdd02eb30a