Backed out 3 changesets (bug 1820807, bug 1853025, bug 1848763) for c…

…ausing frequent network-related startup crashes (bug 1853231). a=backout

Backed out changeset 92d80b858050 (bug 1848763)
Backed out changeset c08cba545d2c (bug 1853025)
Backed out changeset 2fae28575ca8 (bug 1820807)

UltraBlame original commit: f4b0e304ecebf431616ce014dd198c1d361b37c1

modules/libpref/init/StaticPrefList.yaml

@@ -68424,36 +68424,6 @@ mirror
 :
 always
 #
-whether
-to
-redirect
-the
-channel
-for
-auth
-redirects
-.
-See
-Bug
-1820807
--
-name
-:
-network
-.
-auth
-.
-use_redirect_for_retries
-type
-:
-RelaxedAtomicBool
-value
-:
-IS_EARLY_BETA_OR_EARLIER
-mirror
-:
-always
-#
 Whether
 to
 allow

netwerk/base/nsIChannelEventSink.idl

@@ -440,78 +440,6 @@ REDIRECT_STS_UPGRADE
 *
 *
 *
-This
-is
-a
-internal
-redirect
-used
-to
-handle
-http
-authentication
-retries
-.
-*
-Upon
-receiving
-a
-401
-or
-407
-the
-channel
-gets
-redirected
-to
-a
-new
-channel
-*
-(
-same
-URL
-)
-that
-performs
-the
-request
-with
-the
-appropriate
-credentials
-.
-*
-Auth
-retry
-to
-the
-server
-must
-be
-made
-after
-redirecting
-to
-a
-new
-channel
-*
-/
-const
-unsigned
-long
-REDIRECT_AUTH_RETRY
-=
-1
-<
-<
-4
-;
-/
-*
-*
-*
 Called
 when
 a

netwerk/protocol/http/HttpBaseChannel.cpp

@@ -35858,52 +35858,6 @@ REDIRECT_INTERNAL
 {
 /
 /
-for
-internal
-redirect
-due
-to
-auth
-retry
-we
-do
-not
-have
-any
-limit
-/
-/
-as
-we
-might
-restrict
-the
-number
-of
-times
-a
-user
-might
-retry
-/
-/
-authentication
-if
-(
-aRedirectFlags
-&
-nsIChannelEventSink
-:
-:
-REDIRECT_AUTH_RETRY
-)
-{
-return
-NS_OK
-;
-}
-/
-/
 Some
 platform
 features

netwerk/protocol/http/HttpChannelParent.cpp

@@ -11845,27 +11845,6 @@ return
 NS_OK
 ;
 }
-bool
-HttpChannelParent
-:
-:
-IsRedirectDueToAuthRetry
-(
-uint32_t
-redirectFlags
-)
-{
-return
-(
-redirectFlags
-&
-nsIChannelEventSink
-:
-:
-REDIRECT_AUTH_RETRY
-)
-;
-}
 /
 /
 -
@@ -12173,37 +12152,28 @@ for
 service
 worker
 interception
-or
+then
 /
 /
-internal
-redirect
-due
-to
-auth
-retries
-then
 hide
 it
 from
 the
 child
-/
-/
 process
 .
 The
 original
 e10s
 interception
 code
+/
+/
 was
 not
 designed
 with
 this
-/
-/
 in
 mind
 and
@@ -12212,12 +12182,12 @@ not
 necessary
 to
 replace
+/
+/
 the
 HttpChannelChild
 /
 Parent
-/
-/
 objects
 in
 this
@@ -12268,8 +12238,6 @@ newChannel
 ;
 /
 /
-1
-.
 We
 only
 want
@@ -12278,18 +12246,14 @@ hide
 the
 special
 internal
-redirects
+redirect
 from
+nsHttpChannel
 /
 /
-nsHttpChannel
 to
 InterceptedHttpChannel
 .
-/
-/
-2
-.
 We
 want
 to
@@ -12333,31 +12297,6 @@ reset
 /
 case
 .
-/
-/
-3
-.
-If
-this
-is
-an
-internal
-redirect
-due
-to
-auth
-retry
-then
-we
-will
-/
-/
-hide
-it
-from
-the
-child
-process
 if
 (
 (
@@ -12384,14 +12323,6 @@ IsReset
 (
 )
 )
-|
-|
-(
-IsRedirectDueToAuthRetry
-(
-redirectFlags
-)
-)
 )
 {
 /

netwerk/protocol/http/HttpChannelParent.h

@@ -1633,13 +1633,6 @@ NeedFlowControl
 (
 )
 ;
-bool
-IsRedirectDueToAuthRetry
-(
-uint32_t
-redirectFlags
-)
-;
 int32_t
 mSendWindowSize
 ;

netwerk/protocol/http/HttpTransactionChild.cpp

@@ -3261,12 +3261,6 @@ GetSupportsHTTP3
 )
 mode
 reason
-mTransaction
--
->
-Caps
-(
-)
 )
 ;
 return
@@ -3790,6 +3784,12 @@ Timings
 responseTrailers
 mTransactionObserverResult
 lastActTabOpt
+mTransaction
+-
+>
+Caps
+(
+)
 infoArgs
 )
 ;