Commit Graph

8273 Commits

Author SHA1 Message Date
Eden Chuang
85349a441f Bug 1642149 - P1 Fix the web-platform-test html/cross-origin-embedder-policy/require-corp.https.html r=necko-reviewers,valentin
Current HttpBaseChannel::ProcessCrossOriginResourcePolicyHeader does not implement the [[ https://wicg.github.io/cross-origin-embedder-policy/#corp-check | corp-check ]] 3.2.1.2.

Differential Revision: https://phabricator.services.mozilla.com/D77594
2020-06-18 10:01:15 +00:00
Kershaw Chang
dbc3a9628c Bug 1643382 - P2: Clean background channel after redirect is completed r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D79255
2020-06-18 09:28:46 +00:00
Kershaw Chang
f24e554671 Bug 1643382 - P1: BackgroundDataBridgeChild and HttpBackgroundChannelChild don't need to be a cycle r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D79254
2020-06-18 09:28:33 +00:00
Valentin Gosu
4b2dbd9043 Bug 1645715 - Check if uri is null in ComputeCrossOriginOpenerPolicyMismatch r=michal,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D79657
2020-06-17 13:59:53 +00:00
Michal Novotny
a19fb4691b Bug 1644374 - Return NS_BASE_STREAM_WOULD_BLOCK from Http3Stream::OnWriteSegment() when no data was written r=necko-reviewers,dragana
HTTP/3 server can start sending the response before the whole request is sent (e.g. server decides
to ignore the POST data). In such case Http3Stream::OnWriteSegment() can be called in a state when
we don't want to read the data yet. We need to return NS_BASE_STREAM_WOULD_BLOCK instead of NS_OK
to avoid hitting an assertion in nsHttpTransaction::WritePipeSegment().

Differential Revision: https://phabricator.services.mozilla.com/D79984
2020-06-17 10:57:39 +00:00
Nicholas Nethercote
a8f5f49b8a Bug 1645982 - Rename some service getters in Services.py to better match the types. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D79791
2020-06-16 23:32:21 +00:00
Kershaw Chang
4d210f91f7 Bug 1544378 - Implement RetargetDeliveryTo for HttpTransactionParent r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D79429
2020-06-16 14:59:40 +00:00
Junior Hsu
97115bd8ef Bug 1645527 - P2 move OverrideReferrerInfoDuringBeginConnect to PHttpChannel::OnStartRequest, r=baku,valentin,necko-reviewers,mayhemer
We're going to send OnStartRequest via PHttpBackgroundChannel, anything sending via PHttpChannel before OnStartRequest
would be a race.

https://phabricator.services.mozilla.com/D49773#inline-303635
This link implicates that OverrideReferrerInfoDuringBeginConnect is fine just before OnStartRequest. To reduce the
racy and number of IPCs, I move the referrer info overridden to HttpChannelParent::OnStartRequest.

Differential Revision: https://phabricator.services.mozilla.com/D79580
2020-06-16 15:10:03 +00:00
Junior Hsu
e195192267 Bug 1645527 - P1 AssociateApplicationCache should be sent in PHttpChannel::OnStartRequest, r=mayhemer,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D79579
2020-06-16 01:42:55 +00:00
Sylvestre Ledru
85c05f3a47 Bug 1519636 - Reformat recent changes to the Google coding style r=andi
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D79795
2020-06-16 14:37:23 +00:00
Kershaw Chang
6e1e227d7c Bug 1645346 - Clear transaction pump in OnStopRequest to avoid meaningless suspend and resume, r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D79437
2020-06-16 13:04:19 +00:00
Michal Novotny
542d54d4f7 Bug 1644374 - Remember NS_BASE_STREAM_WOULD_BLOCK error in Http3Stream::OnReadSegment() r=necko-reviewers,dragana
Errors are lost between Http3Stream::OnReadSegment() and Http3Stream::ReadSegments() in nsBufferedInputStream::ReadSegments(). In case of NS_BASE_STREAM_WOULD_BLOCK this means we proceed to the next state without sending all request data.

Differential Revision: https://phabricator.services.mozilla.com/D79262
2020-06-15 17:21:59 +00:00
Kershaw Chang
d4338024af Bug 1640105 - Don't initialize user agent components in socket process r=dragana
Since system info service may not be available in socket process, user agent components should be initialized in parent process only.

Differential Revision: https://phabricator.services.mozilla.com/D79153
2020-06-15 18:14:29 +00:00
Junior Hsu
3ce74c7970 Bug 1645254 - Remove PHttpChannel::CancelRedirected, r=valentin,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D79397
2020-06-12 14:13:57 +00:00
Razvan Maries
6425307954 Backed out 2 changesets (bug 1642667) for perma failures on test_peerConnection_basicAudioNATRelay.html. CLOSED TREE
Backed out changeset 94d3e7706178 (bug 1642667)
Backed out changeset f17d5aa45ecc (bug 1642667)
2020-06-12 00:46:08 +03:00
Andrea Marchesini
c1c4f9f3ca Bug 1642667 - Isolate alt-srv and connection pool per first-party when privacy.partition.network_state is set to true - part 2 - tests, r=dragana,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78083
2020-06-10 13:38:24 +00:00
Andrea Marchesini
6a5e156133 Bug 1642667 - Isolate alt-srv and connection pool per first-party when privacy.partition.network_state is set to true - part 1 - implementation, r=dragana,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78082
2020-06-10 13:38:26 +00:00
Michal Novotny
743c33fd15 Bug 1569715 - Don't lookup CORS preflight cache when request bypasses HTTP cache, r=mayhemer,necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D78075
2020-06-11 10:44:11 +00:00
Dragana Damjanovic
f5681bc3e2 Bug 1644377 - Put Draft-28 behind a pref. r=michal,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D79031
2020-06-10 15:48:43 +00:00
Dragana Damjanovic
a886b3e138 Bug 1644377 - Use only the first Http3 versionfrom AltSvc and add X_Firefox_Http3 header. r=michal,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D79070
2020-06-10 10:24:49 +00:00
Dragana Damjanovic
fb111b765c Bug 1644377 - Add support for multiple http3 versions. r=michal,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78995
2020-06-10 16:17:13 +00:00
Valentin Gosu
b8e5f41b3f Bug 1644478 - Don't increase priority for TRR service channel r=mayhemer,necko-reviewers
This was done before using a different thread for the TRR channel
and it didn't actually make a difference in performance, so
maybe we should remove this code.

Differential Revision: https://phabricator.services.mozilla.com/D78939
2020-06-10 15:53:15 +00:00
Alexis Beingessner
b9bb5e1106 Bug 1642724 - convert network.http.sendOriginHeader to a StaticPref. r=KrisWright,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78911
2020-06-10 01:18:52 +00:00
Andrea Marchesini
995aafe3a7 Bug 1643288 - Isolate HTTP channel authentication per first-party when privacy.partition.network_state is set to true - part 2 - nsIHttpProtocolHandler.authCacheKeys, r=mayhemer,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78304
2020-06-10 14:44:08 +00:00
Andrea Marchesini
6bf8d9a1b5 Bug 1643288 - Isolate HTTP channel authentication per first-party when privacy.partition.network_state is set to true, r=mayhemer,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78303
2020-06-10 14:43:46 +00:00
Narcis Beleuzu
86aa1a058e Backed out 2 changesets (bug 1642667) for mda failures on test_peerConnection_basicAudioNATRelay.html . CLOSED TREE
Backed out changeset c1672891baf5 (bug 1642667)
Backed out changeset 4745600e205a (bug 1642667)
2020-06-10 16:34:45 +03:00
Andrea Marchesini
3ba071fd19 Bug 1642667 - Isolate alt-srv and connection pool per first-party when privacy.partition.network_state is set to true - part 2 - tests, r=dragana,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78083
2020-06-09 11:55:21 +00:00
Andrea Marchesini
9cc4f89021 Bug 1642667 - Isolate alt-srv and connection pool per first-party when privacy.partition.network_state is set to true - part 1 - implementation, r=dragana,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78082
2020-06-10 11:32:05 +00:00
Simon Giesecke
82dc9b2271 Bug 1642949 - Replace uses of RemoveElementAt by RemoveLastElement/PopLastElement where possible. r=necko-reviewers,froydnj
Differential Revision: https://phabricator.services.mozilla.com/D78027
2020-06-10 10:46:14 +00:00
Butkovits Atila
e3dce68834 Backed out 3 changesets (bug 1643289, bug 1642949) for causing failure at test_headless_screenshot.html. CLOSED TREE
Backed out changeset 98c420f73380 (bug 1643289)
Backed out changeset 9447ea8910aa (bug 1643289)
Backed out changeset 0c827da9d847 (bug 1642949)
2020-06-10 10:07:23 +03:00
Simon Giesecke
d419f0ff08 Bug 1642949 - Replace uses of RemoveElementAt by RemoveLastElement/PopLastElement where possible. r=necko-reviewers,froydnj
Differential Revision: https://phabricator.services.mozilla.com/D78027
2020-06-10 05:49:28 +00:00
Junior Hsu
58f2c8bc94 Bug 1638513 - P3 strip request body header for preflight, r=valentin,necko-reviewers
(a) Steal nsCORSListenerProxy::UpdateType since the whole usage is removed
    by bug 1312864. Now I'm using it to indicate if the headers should be
	stripped.
(b) Intentionally to let HttpBaseChannel strip the header to save one
    headers copy. It's set via idl, hence I don't think it's moveable.

Differential Revision: https://phabricator.services.mozilla.com/D78831
2020-06-09 18:38:50 +00:00
Junior Hsu
c967db3c67 Bug 1638513 - P2 let nsIHttpChannel implement shouldStripRequestBodyHeader, r=valentin,necko-reviewers
We need to extract the common method into the idl since the connection between necko
and XHR is idl. Need to import the whole necko if we do something like
`#include "HttpBaseChannel.h"

Differential Revision: https://phabricator.services.mozilla.com/D78830
2020-06-09 18:20:15 +00:00
Junior Hsu
b7d7ea4350 Bug 1638513 - P1 extract parsing method, r=valentin,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78829
2020-06-09 18:19:42 +00:00
Dorel Luca
b4228601a2 Backed out 2 changesets (bug 1642667) for XPCShell failures on netwerk/test/unit/test_http2-proxy.js. CLOSED TREE
Backed out changeset ff7fab70e315 (bug 1642667)
Backed out changeset 368a7cd06c69 (bug 1642667)
2020-06-08 20:21:20 +03:00
Andrea Marchesini
bbe813fabe Bug 1642667 - Isolate alt-srv and connection pool per first-party when privacy.partition.network_state is set to true - part 1 - implementation, r=dragana,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78082
2020-06-08 16:22:22 +00:00
Valentin Gosu
cc46191834 Bug 1640091 - Make sure we TRRServiceChannel::mProxyRequest is initialized and used r=kershaw,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78525
2020-06-08 11:38:22 +00:00
Mihai Alexandru Michis
b0a610bce5 Backed out 3 changesets (bug 1640091) for causing xpcshell failures in nsCOMPtr.h
CLOSED TREE

Backed out changeset 03c4c517ee37 (bug 1640091)
Backed out changeset a1d86c177d79 (bug 1640091)
Backed out changeset f481d4ec255a (bug 1640091)
2020-06-08 12:51:42 +03:00
Simon Giesecke
13502cf27c Bug 1642991 - Use RemoveElementsBy where easily possible. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D78055
2020-06-08 09:01:48 +00:00
Valentin Gosu
42d7490489 Bug 1640091 - Make sure we TRRServiceChannel::mProxyRequest is initialized and used r=kershaw,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D78525
2020-06-08 06:57:23 +00:00
Kershaw Chang
e6bff0a49c Bug 1641496 - P5: Make nsHttpActivityDistributor::AddObserver work if socket process is not ready r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D77276
2020-06-04 22:56:40 +00:00
Kershaw Chang
cba8b4bdfc Bug 1641496 - P4: Send request size in OnStopRequest message r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D77275
2020-06-04 22:56:43 +00:00
Kershaw Chang
ef6ab5d3fb Bug 1641496 - P3: Avoid ODA message being processed when the channel is already failed r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D77274
2020-06-04 22:56:45 +00:00
Csoregi Natalia
63635ef2aa Backed out 7 changesets (bug 1641496) for failures on test_trr.js. CLOSED TREE
Backed out changeset 1c927e612dda (bug 1641496)
Backed out changeset 0f1ba77e3942 (bug 1641496)
Backed out changeset a626e86ce5c9 (bug 1641496)
Backed out changeset 196381953eac (bug 1641496)
Backed out changeset 70b0af678bfd (bug 1641496)
Backed out changeset bf41bd8ca89c (bug 1641496)
Backed out changeset 745da0e1cb80 (bug 1641496)
2020-06-05 01:39:19 +03:00
Kershaw Chang
57cbc05cf3 Bug 1641496 - P5: Make nsHttpActivityDistributor::AddObserver work if socket process is not ready r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D77276
2020-06-04 11:49:40 +00:00
Kershaw Chang
fb7f2879f7 Bug 1641496 - P4: Send request size in OnStopRequest message r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D77275
2020-06-04 12:23:57 +00:00
Kershaw Chang
f677ce3a79 Bug 1641496 - P3: Avoid ODA message being processed when the channel is already failed r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D77274
2020-06-04 11:49:18 +00:00
Liang-Heng Chen
01f8cce7e1 Bug 1640135 - P4: update usage of PartitionKey; r=timhuang
Differential Revision: https://phabricator.services.mozilla.com/D77917
2020-06-04 10:51:21 +00:00
Andy Grover
95b90fb23e Bug 1633006 - Update to Neqo 0.4.0 r=dragana,necko-reviewers
Document qlog uses 2 clause bsd license.

Adapt to API changes in glue and http3server

Differential Revision: https://phabricator.services.mozilla.com/D77903
2020-06-02 21:28:31 +00:00
Valentin Gosu
243786d713 Bug 1640867 - Add new HTTP_CHANNEL_ONSTART_SUCCESS_TRR probe r=dragana,tdsmith,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D77127
2020-05-30 08:39:17 +00:00