Commit Graph

6873 Commits

Author SHA1 Message Date
Agi Sferro
3b0e7a3659 Bug 1695056 - Don't provide details info about mobile networks. r=valentin,esawin,mixedpuppy
Since Android 11, READ_PHONE_STATE is required to get detailed information
about mobile networks. Since we don't actually use this information anywhere
right now, we just won't provide it isntead of adding a new permission.

Differential Revision: https://phabricator.services.mozilla.com/D106557
2021-02-26 18:43:40 +00:00
Valentin Gosu
8c97be5391 Bug 1692657 - OSX dns suffix is not detected properly r=kershaw,necko-reviewers
- Uses __res_state.dnsrch to get the DNS suffix domains
- Adds a delay to the DNS suffix check when triggered by a network change
- Implements nsStreamTransportService::DelayedDispatch

Differential Revision: https://phabricator.services.mozilla.com/D106610
2021-02-26 13:56:27 +00:00
Simon Giesecke
9af107a839 Bug 1691913 - Rename nsBaseHashtable::Put to InsertOrUpdate. r=xpcom-reviewers,necko-reviewers,jgilbert,dragana,nika
This makes the naming more consistent with other functions called
Insert and/or Update. Also, it removes the ambiguity whether
Put expects that an entry already exists or not, in particular because
it differed from nsTHashtable::PutEntry in that regard.

Differential Revision: https://phabricator.services.mozilla.com/D105473
2021-02-26 09:11:46 +00:00
Dragana Damjanovic
9db978c11d Bug 1689554 - Use nsUDPSocket for HTTP3/QUIC r=necko-reviewers,valentin
The patch also removes some unnecessary from HttpConnectionUDP and Http3Session which will make easier to merge them.

Differential Revision: https://phabricator.services.mozilla.com/D105085
2021-02-26 08:42:49 +00:00
Dragana Damjanovic
384813f8f9 Bug 1689554 - Extend nsUDPSocket to accomodate more efficiant receiving of data. r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D105079
2021-02-26 08:42:48 +00:00
Valentin Gosu
c33fe1fe47 Bug 1264117 - Limit length of hostnames to 253 characters r=nhnt11,necko-reviewers,dragana
Differential Revision: https://phabricator.services.mozilla.com/D105609
2021-02-19 17:02:34 +00:00
Butkovits Atila
67c4ae9c97 Backed out changeset 0b48f6f8e965 (bug 1264117) for causing failures on test_dns_service.js. CLOSED TREE 2021-02-19 16:41:04 +02:00
Valentin Gosu
b6c3ecbc99 Bug 1264117 - Limit length of hostnames to 253 characters r=nhnt11,necko-reviewers,dragana
Differential Revision: https://phabricator.services.mozilla.com/D105609
2021-02-19 11:45:05 +00:00
Butkovits Atila
9fd21f67da Backed out changeset fd00d1b7fd67 (bug 1264117) for causing failures on test_dns_service_wrap.js. 2021-02-19 12:20:49 +02:00
Valentin Gosu
b3e6410990 Bug 1264117 - Limit length of hostnames to 253 characters r=nhnt11,necko-reviewers,dragana
Differential Revision: https://phabricator.services.mozilla.com/D105609
2021-02-19 09:17:42 +00:00
Dragana Damjanovic
f51b25f77b Bug 1511151 - Add a flag to allow client certs on CORS preflight connections r=necko-reviewers,keeler,valentin,kershaw
This is only used for CORS preflight requests. It is controlled by a pref.
Connections that server such request will be isolated from other anonymous connections.

Differential Revision: https://phabricator.services.mozilla.com/D96775
2021-02-18 18:58:17 +00:00
Alexis Beingessner
3f1a425606 Bug 1686616 - make IO use Components instead of Services. r=kmag
Differential Revision: https://phabricator.services.mozilla.com/D105530
2021-02-18 13:26:32 +00:00
Kershaw Chang
3d66c37e76 Bug 1689987 - P5: Add probes to collect DNS lookup time when ODoH is used r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D105215
2021-02-18 12:50:04 +00:00
Kershaw Chang
67fbb1e79d Bug 1692120 - Avoid accessing nsSocketTransport::mCondition on main thread r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D105596
2021-02-18 10:52:59 +00:00
Valentin Gosu
dbc436b39f Bug 1675540 - Perform URL sanity checks when changing it r=necko-reviewers,kershaw
Differential Revision: https://phabricator.services.mozilla.com/D104541
2021-02-17 08:54:03 +00:00
Gerald Squelart
2416d881e2 Bug 1691589 - Reduce reliance on GeckoProfiler.h when only labels (and maybe markers) are needed - r=necko-reviewers,geckoview-reviewers,sg,agi,florian
There are no code changes, only #include changes.
It was a fairly mechanical process: Search for all "AUTO_PROFILER_LABEL", and in each file, if only labels are used, convert "GeckoProfiler.h" into "ProfilerLabels.h" (or just add that last one where needed).
In some files, there were also some marker calls but no other profiler-related calls, in these cases "GeckoProfiler.h" was replaced with both "ProfilerLabels.h" and "ProfilerMarkers.h", which still helps in reducing the use of the all-encompassing "GeckoProfiler.h".

Differential Revision: https://phabricator.services.mozilla.com/D104588
2021-02-16 04:44:19 +00:00
Simon Giesecke
780a1636a9 Bug 1691894 - Replace several suboptimal uses of Put after Get by GetOrInsertWith or WithEntryHandle. r=xpcom-reviewers,necko-reviewers,kmag,valentin,geckoview-reviewers,agi
Differential Revision: https://phabricator.services.mozilla.com/D104849
2021-02-15 15:12:17 +00:00
smolnar
1afbbe67e1 Backed out 5 changesets (bug 1691894) for causing hazard failures in nsXULPrototypeCache. CLOSED TREE
Backed out changeset 22dc870ee609 (bug 1691894)
Backed out changeset 58c31e9d6ae3 (bug 1691894)
Backed out changeset 7483e84149d8 (bug 1691894)
Backed out changeset f977d6cfa973 (bug 1691894)
Backed out changeset db4503476f34 (bug 1691894)
2021-02-15 16:43:23 +02:00
Simon Giesecke
8973094ec1 Bug 1691894 - Replace several suboptimal uses of Put after Get by GetOrInsertWith or WithEntryHandle. r=xpcom-reviewers,necko-reviewers,kmag,valentin,geckoview-reviewers,agi
Differential Revision: https://phabricator.services.mozilla.com/D104849
2021-02-15 10:04:45 +00:00
Gijs Kruitbosch
b0c70ac17d Bug 1685313 - allow window modal dialogs to display inside windows, r=jaws,mtigley
Differential Revision: https://phabricator.services.mozilla.com/D103388
2021-02-12 22:41:53 +00:00
Christoph Kerschbaumer
1bacbe47cc Bug 1692422: NS_NewURI should return an error if the ioService is not available (e.g due to being called during shutdown) r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D105008
2021-02-12 16:57:54 +00:00
Dragana Damjanovic
efa5705d23 Bug 1690615 - Move DNS lookup into DnsAndConnectSocket r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D104464
2021-02-12 14:57:09 +00:00
Andreea Pavel
c9c6e3158d Backed out 3 changesets (bug 1685313) for failing test_prompt.html on a CLOSED TREE
Backed out changeset 7bdcbdc2d57f (bug 1685313)
Backed out changeset 319bc4d7bc0d (bug 1685313)
Backed out changeset 713de42fe3b4 (bug 1685313)
2021-02-12 06:50:58 +02:00
Gijs Kruitbosch
86da8f87a3 Bug 1685313 - allow window modal dialogs to display inside windows, r=jaws,mtigley
Differential Revision: https://phabricator.services.mozilla.com/D103388
2021-02-11 23:57:57 +00:00
Dragana Damjanovic
b36f91fa41 Bug 1690615 - Rename HalfOpen into DnsAndConnectSocket r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D103932
2021-02-11 16:04:45 +00:00
Jan de Mooij
14360fdec6 Bug 1674777 part 8 - Change nsIArrayBufferInputStream.setData arguments from uint32_t to uint64_t. r=kmag
Similar to the previous patch. Prevent truncation when assigning to mBufferLength.

Differential Revision: https://phabricator.services.mozilla.com/D103760
2021-02-10 08:30:05 +00:00
Kershaw Chang
e0a2b0844a Bug 1680688 - Add a delay before we reload PAC url r=necko-reviewers,valentin
Apparently, some VPN software also manipulate PAC url in system settings. In this case, the PAC url may not be available when we detect the network change, so this patch introduce a pref for adding a delay before reloading the PAC.

Differential Revision: https://phabricator.services.mozilla.com/D104449
2021-02-09 09:05:45 +00:00
Kershaw Chang
22a60154b9 Bug 1685942 - Only fallback to original conn info when network error happens r=necko-reviewers,valentin,dragana
Differential Revision: https://phabricator.services.mozilla.com/D102703
2021-02-08 11:46:51 +00:00
Rahul Yedida
d6bd5bcfa1 Bug 1671844 - PAC isPlainHostName() matching ipv6 addresses r=valentin,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D99021
2021-02-08 13:02:45 +00:00
Nika Layzell
daf01f9769 Bug 1681529 - Part 10: Set StartedReading after calling SetData, r=baku,necko-reviewers,valentin
SetData will check that `mStartedReading` has not been set yet to ensure the
stream is in a non-partially-read state. This can lead to assertion failures
when sending a stream not seeked to the beginning over IPC.

Differential Revision: https://phabricator.services.mozilla.com/D103226
2021-02-04 18:13:19 +00:00
Nika Layzell
9de178dfae Bug 1681529 - Part 7: Wrap the inner streams in a nsMIMEInputStream to make them seekable, r=baku,necko-reviewers,dragana
nsMIMEInputStream has a requirement that the inner nsIInputStream object
implement nsISeekableStream, which is usually enforced by the SetData method.
This check was bypassed by the Deserialize method, which unfortunately meant
that non-seekable IPC payloads could end up within a nsMIMEInputStream when sent
from another process (e.g. due to large nsStringStreams being serialized as
nsPipes over IPC).

This patch uses the SeekableStreamWrapper introduced in the previous patch to
wrap the inner stream when deserializing nsMIMEInputStream, avoiding the
previously mentioned issue.

Differential Revision: https://phabricator.services.mozilla.com/D101806
2021-02-04 18:13:12 +00:00
Dragana Damjanovic
9ff2575792 Bug 1689604 - Remove TCPFastOpen part 3 r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D103517
2021-02-03 17:23:45 +00:00
Dragana Damjanovic
61e4c834b7 Bug 1689604 - Remove TCPFastOpen part 2 r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D103478
2021-01-29 21:59:55 +00:00
Bogdan Tara
abc7c8eeee Backed out 11 changesets (bug 1681529) for talos crashes CLOSED TREE
Backed out changeset c87d0f32d7a6 (bug 1681529)
Backed out changeset b1269f35d525 (bug 1681529)
Backed out changeset 29df8d4c984a (bug 1681529)
Backed out changeset 4def7578ced0 (bug 1681529)
Backed out changeset ce57c5a26c25 (bug 1681529)
Backed out changeset 78b186ec645a (bug 1681529)
Backed out changeset b1d1550a66ca (bug 1681529)
Backed out changeset e8620622208a (bug 1681529)
Backed out changeset 636b1a7c13e4 (bug 1681529)
Backed out changeset a5a8eac68b87 (bug 1681529)
Backed out changeset 968e17db71df (bug 1681529)
2021-02-03 09:29:38 +02:00
Nika Layzell
d42f3a9fc3 Bug 1681529 - Part 10: Set StartedReading after calling SetData, r=baku,necko-reviewers,valentin
SetData will check that `mStartedReading` has not been set yet to ensure the
stream is in a non-partially-read state. This can lead to assertion failures
when sending a stream not seeked to the beginning over IPC.

Differential Revision: https://phabricator.services.mozilla.com/D103226
2021-02-02 23:26:43 +00:00
Nika Layzell
f172ee476b Bug 1681529 - Part 7: Wrap the inner streams in a nsMIMEInputStream to make them seekable, r=baku,necko-reviewers,dragana
nsMIMEInputStream has a requirement that the inner nsIInputStream object
implement nsISeekableStream, which is usually enforced by the SetData method.
This check was bypassed by the Deserialize method, which unfortunately meant
that non-seekable IPC payloads could end up within a nsMIMEInputStream when sent
from another process (e.g. due to large nsStringStreams being serialized as
nsPipes over IPC).

This patch uses the SeekableStreamWrapper introduced in the previous patch to
wrap the inner stream when deserializing nsMIMEInputStream, avoiding the
previously mentioned issue.

Differential Revision: https://phabricator.services.mozilla.com/D101806
2021-02-02 23:26:36 +00:00
Brindusan Cristian
cbdb020883 Backed out 11 changesets (bug 1681529) for mochitest failures at test_reload_large_postdata.html. CLOSED TREE
Backed out changeset f1f988155c82 (bug 1681529)
Backed out changeset f0ba367de05e (bug 1681529)
Backed out changeset dbea9952ec79 (bug 1681529)
Backed out changeset 6e185ec2c4a4 (bug 1681529)
Backed out changeset d0b11c08666a (bug 1681529)
Backed out changeset f2515096b378 (bug 1681529)
Backed out changeset ecd8c3b8fdb4 (bug 1681529)
Backed out changeset 7ea2e9cc8bad (bug 1681529)
Backed out changeset dbc85d0bffaf (bug 1681529)
Backed out changeset f0893f544219 (bug 1681529)
Backed out changeset 91979e21aa8e (bug 1681529)
2021-02-02 22:02:59 +02:00
Nika Layzell
f918e191b0 Bug 1681529 - Part 10: Set StartedReading after calling SetData, r=baku,necko-reviewers,valentin
SetData will check that `mStartedReading` has not been set yet to ensure the
stream is in a non-partially-read state. This can lead to assertion failures
when sending a stream not seeked to the beginning over IPC.

Differential Revision: https://phabricator.services.mozilla.com/D103226
2021-01-29 19:04:03 +00:00
Nika Layzell
55c9ceedc7 Bug 1681529 - Part 7: Wrap the inner streams in a nsMIMEInputStream to make them seekable, r=baku,necko-reviewers,dragana
nsMIMEInputStream has a requirement that the inner nsIInputStream object
implement nsISeekableStream, which is usually enforced by the SetData method.
This check was bypassed by the Deserialize method, which unfortunately meant
that non-seekable IPC payloads could end up within a nsMIMEInputStream when sent
from another process (e.g. due to large nsStringStreams being serialized as
nsPipes over IPC).

This patch uses the SeekableStreamWrapper introduced in the previous patch to
wrap the inner stream when deserializing nsMIMEInputStream, avoiding the
previously mentioned issue.

Differential Revision: https://phabricator.services.mozilla.com/D101806
2021-01-27 21:55:19 +00:00
Valentin Gosu
1422dc2925 Bug 1689208 - Add LINK_TYPE_5G to nsINetworkLinkService and report it on Android r=necko-reviewers,kershaw
Differential Revision: https://phabricator.services.mozilla.com/D103305
2021-01-28 17:06:51 +00:00
Paul Zuehlcke
119a237e12 Bug 1684469 - Allow showing multiple async auth prompts at the same time. r=necko-reviewers,sfoster,agi
- Made asyncPromptAuth fully async.
- Removed auth prompt queuing from LoginManagerAuthPrompter asyncPromptAuth implementation.
- If there are multiple auth prompts with the same target in a tab, consolidate them.
- Removed unused method asyncPromptAuthBC.
- Fixed an issue with PromptTestUtils#waitForPrompt where it didn't always return
  the correct prompt.
- Added test for multi tab auth prompts.

Differential Revision: https://phabricator.services.mozilla.com/D102306
2021-01-27 11:34:05 +00:00
smolnar
edb1ce7144 Backed out 2 changesets (bug 1686989, bug 1684469) for causing mochitest failures in browser_basicAuth_multiTab. CLOSED TREE
Backed out changeset b2aa38ecfb55 (bug 1684469)
Backed out changeset d96ac6e7874e (bug 1686989)
2021-01-26 15:36:54 +02:00
Paul Zuehlcke
37ca601614 Bug 1684469 - Allow showing multiple async auth prompts at the same time. r=necko-reviewers,sfoster,agi
- Made asyncPromptAuth fully async.
- Removed auth prompt queuing from LoginManagerAuthPrompter asyncPromptAuth implementation.
- If there are multiple auth prompts with the same target in a tab, consolidate them.
- Removed unused method asyncPromptAuthBC.
- Fixed an issue with PromptTestUtils#waitForPrompt where it didn't always return
  the correct prompt.
- Added test for multi tab auth prompts.

Differential Revision: https://phabricator.services.mozilla.com/D102306
2021-01-26 10:33:59 +00:00
Csoregi Natalia
d97e4efd8e Backed out 9 changesets (bug 1681529) for causing bustage on TestSeekableStreamWrapper.cpp. CLOSED TREE
Backed out changeset 99d1c9682dc2 (bug 1681529)
Backed out changeset b562b6038855 (bug 1681529)
Backed out changeset 5a5f514a6cfe (bug 1681529)
Backed out changeset ceb55436928a (bug 1681529)
Backed out changeset 9852de883959 (bug 1681529)
Backed out changeset 1a33ea8b533d (bug 1681529)
Backed out changeset 3385635e9521 (bug 1681529)
Backed out changeset 49c28bfc4da4 (bug 1681529)
Backed out changeset 43cc14af229d (bug 1681529)
2021-01-25 23:40:44 +02:00
Nika Layzell
d0b31b8014 Bug 1681529 - Part 7: Wrap the inner streams in a nsMIMEInputStream to make them seekable, r=baku,necko-reviewers,dragana
nsMIMEInputStream has a requirement that the inner nsIInputStream object
implement nsISeekableStream, which is usually enforced by the SetData method.
This check was bypassed by the Deserialize method, which unfortunately meant
that non-seekable IPC payloads could end up within a nsMIMEInputStream when sent
from another process (e.g. due to large nsStringStreams being serialized as
nsPipes over IPC).

This patch uses the SeekableStreamWrapper introduced in the previous patch to
wrap the inner stream when deserializing nsMIMEInputStream, avoiding the
previously mentioned issue.

Differential Revision: https://phabricator.services.mozilla.com/D101806
2021-01-20 16:19:48 +00:00
Csoregi Natalia
086251db75 Backed out 2 changesets (bug 1686989, bug 1684469) for auth and dialog box failures. CLOSED TREE
Backed out changeset 111af4c2bf6b (bug 1684469)
Backed out changeset 81794f8a220b (bug 1686989)
2021-01-25 21:47:43 +02:00
Paul Zuehlcke
54cce40f63 Bug 1684469 - Allow showing multiple async auth prompts at the same time. r=necko-reviewers,sfoster,agi
- Made asyncPromptAuth fully async.
- Removed auth prompt queuing from LoginManagerAuthPrompter asyncPromptAuth implementation.
- If there are multiple auth prompts with the same target in a tab, consolidate them.
- Removed unused method asyncPromptAuthBC.
- Fixed an issue with PromptTestUtils#waitForPrompt where it didn't always return
  the correct prompt.
- Added test for multi tab auth prompts.

Differential Revision: https://phabricator.services.mozilla.com/D102306
2021-01-22 10:18:38 +00:00
Kershaw Chang
6051668e53 Bug 1684038 - Add new SVCParam: ODoHConfig r=necko-reviewers,dragana
Differential Revision: https://phabricator.services.mozilla.com/D101683
2021-01-22 12:59:24 +00:00
Kershaw Chang
27daadc420 Bug 1686828 - Use https OriginAttributes to fetch HTTPS RR r=necko-reviewers,timhuang,dragana
Differential Revision: https://phabricator.services.mozilla.com/D102234
2021-01-21 22:08:26 +00:00
Dana Keeler
b20d3edb85 Bug 1682989 - remove CertBlocklist implementation and MOZ_NEW_CERT_STORAGE build variable r=rmf
Differential Revision: https://phabricator.services.mozilla.com/D100034
2021-01-19 22:11:25 +00:00