Commit Graph

5982 Commits

Author SHA1 Message Date
Shih-Chiang Chien
0381369d6c Bug 1363421 - Part 1, lazy load HTTP protocol handler to improve start-up performance. r=mcmanus
system-info might need to be construct while creating nsHttpHandler and it might take up to 30ms.
Lazy loading the DEFAULT_UA can delay the creation of nsHttpHandler after start-up.

MozReview-Commit-ID: FtIpKjcY38r

--HG--
extra : rebase_source : 8061ed3ce6c42955e52f494166958f5b63ab940b
2017-05-10 11:06:05 +08:00
Nicholas Hurley
23ce343ec5 Bug 1360574 - Detect & handle connection auth at the http/2 layer. r=mcmanus
This is also the non-broken way to fix bug 1346392. Instead of waiting
until the auth handler gets its hands on things, we break layering a bit
and inspect the response headers as soon as we decompress them to see if
there's any connection-oriented auth being requested. If there is, we
treat the situation as if we got a RST_STREAM or GOAWAY with
HTTP_1_1_REQUIRED.

We were able to re-purpose the NS_ERROR_ABORT code path that was
previously used with an inappropriate HTTP status code when talking to
an HTTPS proxy over http/2, as that usage was removed a while back from
the stream, though we still had the (dead) code in the session to handle
the stream giving us that return value. The error code was changed to
NS_ERROR_NET_RESET, however, to give a better description of what's
going on.

MozReview-Commit-ID: DLjOIIiXGrV

--HG--
extra : rebase_source : 703fde39432808cabd05b48aa40165e53ebc5ed1
2017-05-11 14:42:05 -07:00
Nicholas Hurley
e34c25dfc7 Bug 1360574 - Backout original fix to bug 1346392 r=mcmanus
MozReview-Commit-ID: 1ljoiCXciRq

--HG--
extra : rebase_source : e7cead30d7e6ffef73a988d3c7afcbafce8baedd
2017-05-11 10:20:14 -07:00
Wes Kocher
8eaf64f920 Backed out 2 changesets (bug 1362388) for android build bustage in nsHttpChannel.cpp a=backout
Backed out changeset f5b4fbe31c01 (bug 1362388)
Backed out changeset 446ecc3f5715 (bug 1362388)

MozReview-Commit-ID: DI01cDPrgM7
2017-05-12 13:58:28 -07:00
Patrick McManus
1a4ed2cc65 Bug 1362388 - Remove nsAHttpTransaction::Available as footgun r=nwgh
nsAHttpTransaction::Available() obtained a bytecount from the abstract
transaction's input stream. If that stream was derived from a file://
it would create janky IO - so remove the interface.

Http2Push maintains a non-inherited interface which is used to check
the number of bytes it has internally buffered in memory.

MozReview-Commit-ID: IQHt8yGsqDE

--HG--
extra : rebase_source : 78dbd5cae35bc6cb1ce2f03192226cb85564298e
2017-05-05 18:12:36 -04:00
Patrick McManus
3db60df302 Bug 1362388 - dont let File backed http requests find size on main thread r=nwgh
MozReview-Commit-ID: AFM4R0M7dmj

--HG--
extra : rebase_source : ad15d3aad5d0d271acd5cd35dbfbf72e432bcdd7
2017-05-05 17:51:13 -04:00
Valentin Gosu
b830ea1f6d Bug 1363653 - Make AltDataOutputStreamParent not send any messages after object destruction r=mayhemer
This patch prevents the following error:
AltDataOutputStreamChild::Close -> SendClose()
AltDataOutputStreamChild::Release -> Send__delete__()
AltDataOutputStreamParent::RecvClose() -> SendError()
AltDataOutputStreamParent::ActorDestroy -> mIPCOpen = false
AltDataOutputStreamChild::RecvError -> === Crash - object was deleted ===

We introduce the DeleteSelf message.
AltDataOutputStreamChild::Release -> SendDeleteSelf()
AltDataOutputStreamParent::RecvDeleteSelf -> mIPCOpen = false; SendDeleteSelf()
AltDataOutputStreamChild::RecvDeleteSelf -> Send__delete__()

The parent will not send any more messages after receiving the DeleteSelf message.

MozReview-Commit-ID: I9RQ5I7eSt9

--HG--
extra : rebase_source : 8f918d24595248149ebd3857e05e38dc5237059b
2017-05-11 21:35:37 +02:00
Sebastian Hengst
a477978a5d Backed out changeset a418e4a81c02 (bug 1362388) for build bustage from netwerk/protocol/http/nsHttpChannel.cpp:597 not being declared. r=backout 2017-05-11 18:01:01 +02:00
Sebastian Hengst
4dc5072fcb Backed out changeset fef09999e003 (bug 1362388) 2017-05-11 18:00:09 +02:00
Patrick McManus
f48decf9ef Bug 1362388 - Remove nsAHttpTransaction::Available as footgun r=nwgh
nsAHttpTransaction::Available() obtained a bytecount from the abstract
transaction's input stream. If that stream was derived from a file://
it would create janky IO - so remove the interface.

Http2Push maintains a non-inherited interface which is used to check
the number of bytes it has internally buffered in memory.

MozReview-Commit-ID: IQHt8yGsqDE

--HG--
extra : rebase_source : 6b3b210a8f6736887b5076c90621d678b038bcad
2017-05-05 18:12:36 -04:00
Patrick McManus
20a6e4849c Bug 1362388 - dont let File backed http requests find size on main thread r=nwgh
MozReview-Commit-ID: AFM4R0M7dmj

--HG--
extra : rebase_source : 3cd535eb604a78551f6a6043088701e1d1fc40ce
2017-05-05 17:51:13 -04:00
Ryan VanderMeulen
ff1d6f6f88 Merge m-c to autoland. a=merge 2017-05-11 11:30:47 -04:00
Wes Kocher
02bf8c4425 Merge m-c to inbound, a=merge
MozReview-Commit-ID: FfNikA7un6F
2017-05-10 17:31:18 -07:00
Wes Kocher
1b5ad2c128 Merge inbound to central, a=merge
MozReview-Commit-ID: DfRZi0gKjit
2017-05-10 16:48:03 -07:00
Dragana Damjanovic
1b299341de Bug 1362978 - Remove callbacks on socketInput stream if we cancel a TFO socketTransport. r=mcmanus 2017-05-10 22:54:27 +02:00
Valentin Gosu
4ccb51bae2 Bug 1363700 - Add RCWN stats to about:networking r=bz,michal
MozReview-Commit-ID: GVt1omCfL6t

--HG--
extra : rebase_source : fdb321360f21f0f54692bc6b715abb4c155171b7
2017-05-10 19:23:54 +02:00
Patrick McManus
580ef98764 Bug 1362498 - cache some networking mozilla::services r=bagder
MozReview-Commit-ID: G61lDIOKcm

--HG--
extra : rebase_source : 83d3f8258234f85145f485ae35e6ac07e77bac98
2017-05-05 14:32:19 -04:00
Junior Hsu
d30acfedb7 Bug 1363472 - Ensure that the wyciwyg constructor isn't sent when shutting down. r=valentin 2017-05-09 23:57:00 -04:00
Dragana Damjanovic
5512738e6d Bug 1363448 - Let nsHttpConnectionMgr behave properly if StartFastOpen fails (which is conn->Activate() failing). r=mcmanus 2017-05-10 00:18:27 +02:00
Carsten "Tomcat" Book
76ca853e3e Merge mozilla-central to mozilla-inbound 2017-05-09 14:40:11 +02:00
Dragana Damjanovic dd.mozilla@gmail.com
62b9fc4ab8 Bug 1357835 - Extend telemetry for http autentication dialog prompt to show the subresource cross-origin auth dialog prompt per suresource type. Also add a pre to putentially disable auth propts for the cross-origin images. r=mayhemer r=ckerschb r=francois 2017-05-09 12:36:21 +02:00
Dragana Damjanovic dd.mozilla@gmail.com
bf4c15a65c Bug 136178 - Keep list of weakPtrs instead of streamIds of streams started during 0rtt. r=mcmanus 2017-05-09 11:37:45 +02:00
Wes Kocher
73b318df0a Backed out changeset 7cd208304d1f (bug 1363472) for being a possible cause of failures in test_animSVGImage.html a=backout
MozReview-Commit-ID: KU4GKyDD2ne

--HG--
extra : rebase_source : d66a936a81723d1dcb7032f28d50b5582d748fa9
2017-05-10 10:15:39 -07:00
Junior Hsu
d9ef6fd797 Bug 1363472 - Ensure not sending wyciwyg constructor when shutting down, r=valentin 2017-05-09 23:57:00 +02:00
Kershaw Chang
9ea8bbb5d9 Bug 1352176 - Part2: Pass an event target when calling nsInputStreamPump::Create, r=mayhemer 2017-05-09 20:05:00 +02:00
Jonathan Watt
33cfc4aa43 Bug 1362891, part 2 - Make use of XRE_IsE10sParentProcess in code that could use it. r=froydnj
MozReview-Commit-ID: 81PFEgNH37m

--HG--
extra : rebase_source : 350b11b3bec652472565bd590ef13680e480a75e
2017-04-24 10:40:12 +01:00
Wes Kocher
f5ba7a67d6 Merge m-c to inbound, a=merge
MozReview-Commit-ID: 9QuGHXqdxTb
2017-05-08 16:14:34 -07:00
Wes Kocher
cbfdaf8fb2 Merge inbound to central, a=merge CLOSED TREE
MozReview-Commit-ID: 5kxOZZxjMEl
2017-05-08 16:07:25 -07:00
Dragana Damjanovic
b035817315 Bug 1362821 - Properly destroy a HalfOpeSocket that is used as a backup for a TFO connection if another H2 connection is established for the same host. r=mcmanus 2017-05-08 18:22:45 +02:00
Amar Lakshya
077409e068 Bug 633747 - about:cache now displays 'Expired Immediately' for 1970-01-01 01:00:00 r=mayhemer
MozReview-Commit-ID: 89CX7UqwuXl

--HG--
extra : rebase_source : b8d9a8fba8ff017b5633c093c57127076c77757a
2017-05-02 20:48:34 +05:30
Dragana Damjanovic dd.mozilla@gmail.com
87f75e4bea Bug 1362861 - Call SetFastOpenStatus only if we have httpTransaction. r=mcmanus 2017-05-08 15:33:44 +02:00
Amar Lakshya
badcf950f5 Bug 633747 - about:cache displays "Expires: 1970-01-01 01:00:00" r=mayhemer
MozReview-Commit-ID: BAn8EFYGeux

--HG--
extra : rebase_source : 1bf1626578e22f14ee3c7511227c4106758d07b3
2017-04-29 01:24:29 +05:30
Dragana Damjanovic
6ad0af18de Bug 1363108 - Fix some small bugs connectied to tcp fast open. r=mcmanus 2017-05-08 22:15:26 +02:00
Gijs Kruitbosch
2c13f23a54 Bug 1360842, r=valentin
MozReview-Commit-ID: B7uG3ShaGuH

--HG--
extra : rebase_source : 92813be22a7554fec17981bf14903a526145fed2
extra : amend_source : 30643e1d5a4b942d4069e3dd28b5f736cb7fdc18
2017-05-04 11:28:04 +01:00
Dragana Damjanovic
7a6c685fd1 Bug 1363034 - Do not make new transactions for backupSocketTransport, behave as nsHttpConnectionMgr used to behave. r=mcmanus 2017-05-08 20:17:47 +02:00
Honza Bambas
a1b64b4694 Bug 1319111 - Expose URI to make security check against on LoadInfo (no LOAD_REPLACE flag). r=bz 2017-01-27 19:10:01 +01:00
Ehsan Akhgari
c696722eda Bug 1360973 - Don't do the work needed for same origin referrer policies unless if we need to; r=tnguyen
CheckSameOriginURI() can be expensive and shows up under profiles, so we
should try to avoid calling it unless if we are going to use
isCrossOrigin.
2017-05-04 13:30:13 -04:00
Michal Novotny
133308dd0d Bug 1361435 - Remove references to unused pref network.http.bypass-cachelock-threshold, r=mcmanus
Removes references to now unused pref that was added in bug 868441 and removed in bugs 913807 and 1054572. It also removes some leftovers from http channel which have not been removed in those 2 bugs.
2017-05-04 17:37:42 +02:00
Junior Hsu
5130d2e6d3 Bug 1359951 - Ensure that the WebSocket not sending constructor when shutting down, r=nwgh
--HG--
extra : rebase_source : c11a6d29d6da9a83f15a00f02c877960a0d5bf06
2017-05-02 19:50:00 +02:00
Dragana Damjanovic
0f080b064b Bug 1360515 - During TCP Fast Open let necko works as it is used to, but only limit amount of data we write to lower layers, i.e. nss, tcpFastOpenLayer, tcp socket. r=mcmanus 2017-05-04 12:15:33 +02:00
Dragana Damjanovic
2c802ffe1c Bug 1360570 - Make sure to write as much data as possible during FastOpen. r=mcmanus 2017-05-04 12:15:23 +02:00
Dragana Damjanovic
a6965692d1 Bug 1359938 - Fix socket status events for TCP Fast Open. r=mcmanus 2017-05-04 12:15:16 +02:00
Dragana Damjanovic
fc3b8dbe53 Bug 1359847 - Improve transaction restart if tfo is not supported. r=mcmanus 2017-05-04 12:15:10 +02:00
Dragana Damjanovic
6fb8664ccb Bug 1352271 - Add telemetry about TCP Fast Open. r=mcmanus r=francois 2017-05-04 12:15:01 +02:00
Dragana Damjanovic
099e776a6d Bug 1352274 - Add response header indicator for TCP FastOpen suceeded. r=mcmanus 2017-05-04 12:14:54 +02:00
Dragana Damjanovic
04e98c441a Bug 1188435 - Fix http2 restart on net-reset and fix activeConn count. r=mcmanus 2017-05-04 12:14:41 +02:00
Dragana Damjanovic
abf0402f0a Bug 1188435 - Make TFO work with 0RTT and H2 work. r=mcmanus 2017-05-04 12:14:29 +02:00
Dragana Damjanovic
827d3bbab5 Bug 1188435 - Make TLS 0RTT work with TLS Fast Open. r=mcmanus 2017-05-04 12:14:13 +02:00
Dragana Damjanovic
accc6e95b1 Bug 1188435 - Implement fast open necko part. r=mcmanus 2017-05-04 12:14:04 +02:00
Yoshi Huang
7652396698 Bug 1284579 - Part 2: revise NS_ShouldCheckAppCache. r=valentin
There's one redudant NS_ShouldCheckAppCache(nsIURI*, bool) is not used
anymore. Also we remove the extra usePrivateBrowsing argument, since we
can get this information from nsIPrincipal.
2017-05-04 09:40:00 +08:00