Граф коммитов

2837 Коммитов

Автор SHA1 Сообщение Дата
Andrea Marchesini f3527b4c51 Bug 1643156 - Cleanup nsMultiplexInputStream - part 4 - support Seek(END) correctly, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D77100
2020-06-04 00:18:57 +00:00
Andrea Marchesini 8db10f63e0 Bug 1643156 - Cleanup nsMultiplexInputStream - part 2 - remove the use of nsITellable interface for substreams, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D76902
2020-06-04 00:18:57 +00:00
Andrea Marchesini d0f5bf2fd9 Bug 1643156 - Cleanup nsMultiplexInputStream - part 1 - keep the original stream, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D76901
2020-06-04 12:23:17 +00:00
ssengupta fd4af3407b Bug 1637742 - P1 - Protocol documentation now states no error should be returned if AsyncWait is called on closed stream r=baku
Differential Revision: https://phabricator.services.mozilla.com/D77407
2020-06-02 11:09:47 +00:00
Kagami Sascha Rosylight 716734fea5 Bug 1634454 - Resolve symlinks in nsLocalFileWin r=emk
Differential Revision: https://phabricator.services.mozilla.com/D73462
2020-06-01 15:12:31 +00:00
Dorel Luca ab3da7db14 Backed out 4 changesets (bug 1636823) for gtests failures in TestMultiplexInputStream.BlockingVsAsync. CLOSED TREE
Backed out changeset ebb1525ea063 (bug 1636823)
Backed out changeset 25455bb698d8 (bug 1636823)
Backed out changeset 930874501ab3 (bug 1636823)
Backed out changeset 5d46b483c59c (bug 1636823)
2020-06-01 15:55:33 +03:00
Andrea Marchesini ca598e4147 Bug 1636823 - nsMultiplexInputStream should not be blocking and nsIAsyncInputStream at the same time - part 4 - support Seek(END) correctly, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D77100
2020-06-01 12:16:45 +00:00
Andrea Marchesini 0a64e6bc6d Bug 1636823 - nsMultiplexInputStream should not be blocking and nsIAsyncInputStream at the same time - part 2 - remove the use of nsITellable interface for substreams, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D76902
2020-06-01 12:16:15 +00:00
Andrea Marchesini 24acc8cc48 Bug 1636823 - nsMultiplexInputStream should not be blocking and nsIAsyncInputStream at the same time - part 1 - implementation, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D76901
2020-06-01 12:15:57 +00:00
Dorel Luca 1413b0ea0d Backed out 4 changesets (bug 1636823) for build bustages in workspace/obj-build/dist/include/gtest/gtest.h. CLOSED TREE
Backed out changeset 52322ad2515e (bug 1636823)
Backed out changeset 2329c5e53493 (bug 1636823)
Backed out changeset a388fd40baeb (bug 1636823)
Backed out changeset 3019a2171aed (bug 1636823)
2020-06-01 14:39:31 +03:00
Andrea Marchesini acd3fa5467 Bug 1636823 - nsMultiplexInputStream should not be blocking and nsIAsyncInputStream at the same time - part 4 - support Seek(END) correctly, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D77100
2020-05-28 05:48:52 +00:00
Andrea Marchesini 699028c74a Bug 1636823 - nsMultiplexInputStream should not be blocking and nsIAsyncInputStream at the same time - part 2 - remove the use of nsITellable interface for substreams, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D76902
2020-06-01 10:31:46 +00:00
Andrea Marchesini fa799fe4a7 Bug 1636823 - nsMultiplexInputStream should not be blocking and nsIAsyncInputStream at the same time - part 1 - implementation, r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D76901
2020-06-01 10:34:52 +00:00
Kagami Sascha Rosylight 81428dbb30 Bug 1466532 - Part 2: Drop shortcut support in nsLocalFileWin r=froydnj
Native Windows apps (including but not limited to IE, old Edge, and Notepad) does not automatically resolve shell links (.lnk), so this patch follows their behavior.

Differential Revision: https://phabricator.services.mozilla.com/D73546
2020-05-28 16:59:56 +00:00
Kagami Sascha Rosylight 236de553f6 Bug 1466532 - Part 1: Drop nsIFile::followLinks r=froydnj,necko-reviewers,dragana
This drops `followLinks` as 1) it has been no-op on UNIX for a long time and 2) its Windows implementation never had a proper symlink support.

Differential Revision: https://phabricator.services.mozilla.com/D75569
2020-05-26 20:51:16 +00:00
Csoregi Natalia 38116effd4 Backed out changeset 8d0d9cb87686 (bug 1466532) for failures on test_file_protocol.js. CLOSED TREE 2020-05-26 01:46:49 +03:00
Kagami Sascha Rosylight b712a9333b Bug 1466532 - Part 1: Drop nsIFile::followLinks r=froydnj,necko-reviewers,dragana
This drops `followLinks` as 1) it has been no-op on UNIX for a long time and 2) its Windows implementation never had a proper symlink support.

Differential Revision: https://phabricator.services.mozilla.com/D75569
2020-05-25 18:55:59 +00:00
Kershaw Chang 5732714230 Bug 1539819 - P3: Some adjustments to make TRRServiceChannel work on socket process r=dragana,necko-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D68399
2020-05-18 20:18:08 +00:00
Aaron Klotz b392ecef67 Bug 1637452: Part 3 - Fix JNI includes in xpcom; r=froydnj
In this bug we're moving away from monolithic JNI headers to class-specific
headers so that we don't have to rebuild the world every time we make a change
to a JNI interface.

Differential Revision: https://phabricator.services.mozilla.com/D75366
2020-05-15 17:04:15 +00:00
Tom Tung 3d101cac5a Bug 1634267 - DOS device paths shouldn't be blocked by IsBlockedUNCPath; r=dom-workers-and-storage-reviewers,janv,froydnj
Bug 1536796 introduces "\\?\" prefix (DOS device specifier) to Windows file
paths. At the moment, the prefix is only prepended to the file paths that start
with a disk designator and a backslash.

On the other hands, IsBlockedUNCPath blocks file paths that start with "\\" in
Windows and that includes DOS device paths (the cases mentioned above).

Thus, this patch prevents DOS device paths from being treating as UNC paths in
IsBlockedUNCPath.

Differential Revision: https://phabricator.services.mozilla.com/D73621
2020-05-13 14:32:35 +00:00
Simon Giesecke 2a7a935311 Bug 1626570 - Improve handling of copying arrays in xpcom/. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D73624
2020-05-05 11:27:18 +00:00
Tom Tung ffb47f9d00 Bug 1626513 - Check whether the first letter is [A-Za-z] for StartsWithDiskDesignatorAndBackslash; r=dom-workers-and-storage-reviewers,janv,xpcom-reviewers,sg
Differential Revision: https://phabricator.services.mozilla.com/D72635
2020-04-28 15:57:10 +00:00
Valentin Gosu 0b2e4bafd9 Bug 1625749 - Make sure SlicedInputStream initializes all its members. r=nika
Differential Revision: https://phabricator.services.mozilla.com/D70919
2020-04-14 18:04:13 +00:00
Konstantin Morozov ea1011eb87 Bug 1629432 - converting integer literal to bool. r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D71181
2020-04-16 14:14:08 +00:00
Valentin Gosu b5c0442d4e Bug 1595886 - Use a temp string in readSegments r=dragana
Differential Revision: https://phabricator.services.mozilla.com/D68204

--HG--
extra : moz-landing-system : lando
2020-04-15 12:31:34 +00:00
Honza Bambas aa4bbee2f9 Bug 1627572 - Use an atomic bool for a quick-return check in FilePreferences::IsAllowedPath, r=michal
Differential Revision: https://phabricator.services.mozilla.com/D69976

--HG--
extra : moz-landing-system : lando
2020-04-07 11:18:54 +00:00
Eric Rahm 2bd348aaf4 Bug 1627390 - Add missing includes and namespaces to xpcom/io. r=xpcom-reviewers,sg
Differential Revision: https://phabricator.services.mozilla.com/D69656

--HG--
extra : moz-landing-system : lando
2020-04-07 19:09:47 +00:00
Andreas Farre 25ca8d7890 Bug 1620594 - Part 7: Remove TabGroup and SystemGroup. r=nika,bas
TabGroup never really made any difference in which thread something go
dispatched to. This was the intended use, but development of TabGroups
with abstract main threads never made it that far. The good thing is
that thish makes it safe to also remove to the SystemGroup and instead
switch all SystemGroup dispatches to dispatches to main thread.

Timers for setTimeout and workers were the sole users of wrapped and
throttled event targets, that those throttled queues have been moved
to the BrowsingContextGroup and are now accessed explicitly.

The SchedulerEventTarget has been removed, since there are no longer a
separate event target for every TaskCategory. Instead a
LabellingEventTarget has been added to DocGroup to handle the case
where an event is dispatched do DocGroup or when an AbstractThread is
created using a DocGroup. This means that we'll actually label more
events correctly with the DocGroup that they belong to.

DocGroups have also been moved to BrowsingContextGroup.

Depends on D67636

Differential Revision: https://phabricator.services.mozilla.com/D65936

--HG--
extra : moz-landing-system : lando
2020-04-07 15:17:47 +00:00
Tom Tung 4e96941288 Bug 1536796 - P2 - Handle disable string paring in nsLocalFileWin; r=sg,dom-workers-and-storage-reviewers,froydnj,janv
In the Windows API, the maximum length for a path is MAX_PATH in general.
However, the Windows API has many functions that also have Unicode versions to
permit an extended-length path for a maximum total path length of 32,767
characters. To specify an extended-length path, use the "\\?\" prefix.

A path component which ends with a dot is not allowed for Windows API. However,
using the "\\?\" prefix can also resolved this issue.

This patch aims to fix the issues which are mentioned above by prepending the
prefix to the path of nsLocalFile if mDisableStringParsing is set to true.

Differential Revision: https://phabricator.services.mozilla.com/D67014

--HG--
extra : moz-landing-system : lando
2020-04-01 07:51:05 +00:00
Tom Tung 4debe01abb Bug 1536796 - P1 - Introduce a flag (useDOSDevicePathSyntax) to nsILocalFileWin and a method (QM_NewLocalFile) to QuotaCommon; r=janv,dom-workers-and-storage-reviewers,froydnj
Differential Revision: https://phabricator.services.mozilla.com/D67873

--HG--
extra : moz-landing-system : lando
2020-04-01 07:26:27 +00:00
Chris Peterson b1f54173e8 Bug 1624776 - Replace MOZ_MUST_USE with [[nodiscard]] in xpcom. r=xpcom-reviewers,KrisWright
Also move MOZ_MUST_USE before function declarations' specifiers and return type. While clang and gcc's __attribute__((warn_unused_result)) can appear before, between, or after function specifiers and return types, the [[nodiscard]] attribute must precede the function specifiers.

Also remove MOZ_MUST_USE from nsIMemoryReporter's function pointer typedefs. [[nodiscard]] is an attribute of a function's declaration, not the function's type, and thus can't be applied to function pointers or lambdas.

Differential Revision: https://phabricator.services.mozilla.com/D68138

--HG--
extra : moz-landing-system : lando
2020-03-27 17:21:48 +00:00
Toshihito Kikuchi 3bc1f2c50b Bug 1605308 - Do not use ShellExecuteByExplorer when a downloaded file is an executable. r=Gijs,froydnj
This is one of the efforts to reduce usage of `ShellExecuteByExplorer`
(bug 1620335).

The purpose of using `ShellExecuteByExplorer` in the scenario to open a
downloaded file is to support applications which are not compatible with
the mitigation policies of our process.  When a downloaded file is an
executable, however, we prefer security to compatibility and in particular
we want to prevent binary planting on a user's download directory.

The proposed fix is to go to `ShellExecuteExW` straight if the target
file to launch is an executable.

Differential Revision: https://phabricator.services.mozilla.com/D66325

--HG--
extra : moz-landing-system : lando
2020-03-12 22:05:34 +00:00
Cosmin Sabou e7343ec23b Backed out changeset 01cef3030838 (bug 1605308) for build bustages on nsLocalFileWin.cpp
CLOSED TREE
2020-03-12 23:26:54 +02:00
Toshihito Kikuchi 156f519842 Bug 1605308 - Do not use ShellExecuteByExplorer when a downloaded file is an executable. r=Gijs,froydnj
This is one of the efforts to reduce usage of `ShellExecuteByExplorer`
(bug 1620335).

The purpose of using `ShellExecuteByExplorer` in the scenario to open a
downloaded file is to support applications which are not compatible with
the mitigation policies of our process.  When a downloaded file is an
executable, however, we prefer security to compatibility and in particular
we want to prevent binary planting on a user's download directory.

The proposed fix is to go to `ShellExecuteExW` straight if the target
file to launch is an executable.

Differential Revision: https://phabricator.services.mozilla.com/D66325

--HG--
extra : moz-landing-system : lando
2020-03-12 21:05:24 +00:00
Andrew McCreight 4babb2b5ab Bug 1609815 - Remove Web Replay C++ implementation. r=jgilbert,jandem,gbrown
Patch by bhackett and jlaster. Also reviewed by mccr8.

Differential Revision: https://phabricator.services.mozilla.com/D60197

--HG--
extra : moz-landing-system : lando
2020-02-27 17:39:15 +00:00
Ciure Andrei 00dd87f6f4 Backed out changeset d407a28318e6 (bug 1609815) for causing windows ming bustages CLOSED TREE
--HG--
extra : histedit_source : b2c748e31e0f6ba8fcf9960a336e0bbd361b07e6
2020-02-27 07:05:19 +02:00
Andrew McCreight b197e1f783 Bug 1609815 - Remove Web Replay C++ implementation. r=jgilbert,jandem,gbrown
Patch by bhackett and jlaster. Also reviewed by mccr8.

Differential Revision: https://phabricator.services.mozilla.com/D60197

--HG--
extra : moz-landing-system : lando
2020-02-27 04:43:48 +00:00
Tuan 3fadc6333d Bug 1614139 - Copy constructor of nsPipeInputStream should not be declared explicit. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D62735

--HG--
extra : moz-landing-system : lando
2020-02-13 16:09:09 +00:00
Simon Giesecke b50347f917 Bug 1611415 - Prefer using std::move over forget. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D60980

--HG--
extra : moz-landing-system : lando
2020-02-13 14:38:48 +00:00
shindli 91aa0518dd Backed out changeset 0c982bc69cb3 (bug 1611415) for causing build bustages in /builds/worker/workspace/build/src/obj-firefox/dist/include/nsCOMPtr CLOSED TREE 2020-02-12 20:13:29 +02:00
Simon Giesecke f604a47fa5 Bug 1611415 - Applied FixItHints from mozilla-non-std-move. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D60980

--HG--
extra : moz-landing-system : lando
2020-02-12 17:24:41 +00:00
Simon Giesecke 30984f7bab Bug 1613985 - Use default for equivalent-to-default constructors/destructors in xpcom. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D62543

--HG--
extra : moz-landing-system : lando
2020-02-12 11:13:33 +00:00
Nathan Froyd 4c03f99cfe Bug 1614361 - check GetPermissions call for failure; r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D62267

--HG--
extra : moz-landing-system : lando
2020-02-10 23:14:42 +00:00
Valentin Gosu 48199b9dc3 Bug 1482224 - Move nsLocalFileWin::Reveal to the background thread pool r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D61553

--HG--
extra : moz-landing-system : lando
2020-02-04 14:36:56 +00:00
Valentin Gosu 74f9a5e3d8 Bug 1608114 - Increase the priority of the TRR main thread events r=mayhemer
Differential Revision: https://phabricator.services.mozilla.com/D61306

--HG--
extra : moz-landing-system : lando
2020-01-31 08:46:44 +00:00
Kristen Wright 51dfe11781 Bug 1602986 - Get rid of NS_WARN_IF warnings in nsSegmentedBuffer::FreeOMT r=froydnj
At shutdown, the stream transport service is not available and `free(aPtr)` happens synchronously. These warnings will always trigger and produce a lot of logspam.

Differential Revision: https://phabricator.services.mozilla.com/D61217

--HG--
extra : moz-landing-system : lando
2020-01-29 10:03:34 +00:00
Gijs Kruitbosch 29de7f7b00 Bug 1596668, r=spohl
Differential Revision: https://phabricator.services.mozilla.com/D58206

--HG--
extra : moz-landing-system : lando
2020-01-11 16:04:19 +00:00
Toshihito Kikuchi 7b04cb1b04 Bug 1602726 - Fall back to ShellExecuteEx if ShellExecuteByExplorer fails. r=aklotz
It turned out that `ShellExecuteByExplorer` does not work on VDI such as Citrix
or Microsoft RemoteApp.  This patch adds a fallback to the original launching
code if `ShellExecuteByExplorer` fails.  This will be a temporary solution until
we find out a way to solve both interop issues `PreferSystem32Images` and VDI.

Differential Revision: https://phabricator.services.mozilla.com/D58054

--HG--
extra : moz-landing-system : lando
2020-01-08 16:41:03 +00:00
Eric Rahm 6012f510a8 Bug 1241518 - Part 2: Use Maybe instead of nsAutoPtr in nsInputStreamTee r=kmag
Differential Revision: https://phabricator.services.mozilla.com/D58283

--HG--
extra : moz-landing-system : lando
2020-01-07 00:06:02 +00:00
Eric Rahm 6d75492859 Bug 1241518 - Part 1: Remove unused nsAutoPtr.h includes in xpcom/ r=kmag
This removes various unused `#include "nsAutoPtr.h"` in `xpcom/`. Additionally
adds a few includes to the media stack.

Differential Revision: https://phabricator.services.mozilla.com/D58282

--HG--
extra : moz-landing-system : lando
2020-01-07 00:06:01 +00:00