Commit Graph
75 Commits
Author SHA1 Message Date
Sylvestre Ledru 6be5ec2fff Bug 2023419 - Remove Emacs/Vim modelines from toolkit/ r=emilio,mkaply,extension-reviewers,credential-management-reviewers,desktop-theme-reviewers,win-reviewers,webcompat-reviewers,core-sessionstore-reviewers,profiles-reviewers,ai-platform-reviewers,mconley,twisniewski,gstoll,jhirsch,rpl,dimi,akulyk
Differential Revision: https://phabricator.services.mozilla.com/D287874
2026-04-14 12:56:19 +00:00
agoloman ae99dfb565 Revert "Bug 2023419: apply code formatting via Lando" for causing bc failures @browser_sendQuery.js.
This reverts commit aa15b809d9.

Revert "Bug 2023419 - Remove Emacs/Vim modelines from toolkit/ - follow up r=emilio"

This reverts commit c659385796.

Revert "Bug 2023419 - Remove Emacs/Vim modelines from toolkit/ r=emilio,mkaply,extension-reviewers,credential-management-reviewers,desktop-theme-reviewers,win-reviewers,webcompat-reviewers,core-sessionstore-reviewers,profiles-reviewers,ai-platform-reviewers,mconley,twisniewski,gstoll,jhirsch,rpl,dimi,akulyk"

This reverts commit 1e55e9bf9b.
2026-04-13 13:06:21 +00:00
Sylvestre Ledru 1e55e9bf9b Bug 2023419 - Remove Emacs/Vim modelines from toolkit/ r=emilio,mkaply,extension-reviewers,credential-management-reviewers,desktop-theme-reviewers,win-reviewers,webcompat-reviewers,core-sessionstore-reviewers,profiles-reviewers,ai-platform-reviewers,mconley,twisniewski,gstoll,jhirsch,rpl,dimi,akulyk
Differential Revision: https://phabricator.services.mozilla.com/D287874
2026-04-13 11:30:51 +00:00
rperta f5848b1deb Revert "Bug 2023419: apply code formatting via Lando" for causing bc failures at browser_sendQuery.js and browser_console_to_mozlog.js
This reverts commit f8a9770c7d.

This reverts commit 47e16fbdd2.

This reverts commit 9f363abaef.
2026-03-20 06:37:54 +00:00
Sylvestre Ledru 9f363abaef Bug 2023419 - Remove Emacs/Vim modelines from toolkit/ r=emilio,mkaply,extension-reviewers,credential-management-reviewers,desktop-theme-reviewers,win-reviewers,webcompat-reviewers,profiles-reviewers,ai-ondevice-reviewers,mconley,twisniewski,gstoll,jhirsch,rpl,dimi,akulyk
replaced by .editorconfig
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D287874
2026-03-20 04:55:48 +00:00
Dave Townsend 6cd8d87df6 Bug 1915216: Add a method to asynchronously write the important data about the current profile to the INI file on disk. r=glandium,jhirsch,pehrsons,backup-reviewers,mconley
This adds an asynchronous method to lock the startup files using the same
lock that we use during normal startup.

The profile service then uses this lock to gate access to the profiles.ini
files adding a method to async flush the entire database or in the case
that the on-disk database has changed a way to mergwe in some properties
about the current profile into the on-disk version.

Differential Revision: https://phabricator.services.mozilla.com/D222662
2024-10-18 07:53:02 +00:00
Butkovits Atila 20cd2bfd08 Backed out changeset fe441360b591 (bug 1915216) for causing leaks at ThreadSafeWeakReference. CLOSED TREE 2024-10-17 19:11:29 +03:00
Dave Townsend 4d68ba5791 Bug 1915216: Add a method to asynchronously write the important data about the current profile to the INI file on disk. r=glandium,jhirsch,pehrsons,backup-reviewers,mconley
This adds an asynchronous method to lock the startup files using the same
lock that we use during normal startup.

The profile service then uses this lock to gate access to the profiles.ini
files adding a method to async flush the entire database or in the case
that the on-disk database has changed a way to mergwe in some properties
about the current profile into the on-disk version.

Differential Revision: https://phabricator.services.mozilla.com/D222662
2024-10-17 15:09:41 +00:00
Norisz Fay e8694852ee Backed out changeset 46f4c195d34a (bug 1915216) for causing async related xpcshell failures CLOSED TREE 2024-10-16 18:07:26 +03:00
Dave Townsend 75cb8ade57 Bug 1915216: Add a method to asynchronously write the important data about the current profile to the INI file on disk. r=glandium,jhirsch,pehrsons,backup-reviewers,mconley
This adds an asynchronous method to lock the startup files using the same
lock that we use during normal startup.

The profile service then uses this lock to gate access to the profiles.ini
files adding a method to async flush the entire database or in the case
that the on-disk database has changed a way to mergwe in some properties
about the current profile into the on-disk version.

Differential Revision: https://phabricator.services.mozilla.com/D222662
2024-10-16 08:32:25 +00:00
Dave Townsend 6d1522dd9a Bug 1892400: Add a method to send a command line to a running instance. r=jhirsch
Allows sending a command line after startup. Also allows blocking the remoting services
attempt to raise the application to the foreground if desired.

Differential Revision: https://phabricator.services.mozilla.com/D207976
2024-08-15 13:45:58 +00:00
Dave Townsend c201ff81d4 Bug 1892400: Remote service should live for the lifetime of the application. r=jhirsch
This lays the groundwork for being able to send remote messages at runtime.

Differential Revision: https://phabricator.services.mozilla.com/D207975
2024-08-15 13:45:58 +00:00
Dave Townsend c7c0e3294b Bug 1892400: Remove the startup token from the non-unix remote implementations. r=jhirsch,glandium
This simplifies the call signatures of all versions since this is only needed in unix.

Differential Revision: https://phabricator.services.mozilla.com/D216032
2024-08-15 13:45:57 +00:00
Dave Townsend 92efa10eef Bug 1892400: Windows remote client should send the intended command line. r=jhirsch,rkraesig
Currently the windows client sends the current process command line which is
potentially an issue since some of those arguments may have been consumed
already. It will also not support sending custom command lines after startup.

This also switches to sending the UTF-8 command line we already have since that
matches other platforms better and makes some arguments const.

Differential Revision: https://phabricator.services.mozilla.com/D216031
2024-08-15 13:45:57 +00:00
Dave Townsend 89ee8695ff Bug 1892400: Remove the unnecessary response and success arguments. r=jhirsch,glandium
A lot of this dates back to when the remoting service was a more fully featured
protocol, now it is just added complication.

Differential Revision: https://phabricator.services.mozilla.com/D207974
2024-08-15 13:45:56 +00:00
Sandor Molnar d7f53c7f93 Backed out 6 changesets (bug 1892400) for Gtest failures CLOSED TREE
Backed out changeset 25b37643e1fe (bug 1892400)
Backed out changeset 7c12bcdfd937 (bug 1892400)
Backed out changeset bf5862fa2a1d (bug 1892400)
Backed out changeset a0bb5ed531ce (bug 1892400)
Backed out changeset beb16099057e (bug 1892400)
Backed out changeset 82111a5fc831 (bug 1892400)
2024-08-14 11:53:24 +03:00
Dave Townsend 5e482208a0 Bug 1892400: Add a method to send a command line to a running instance. r=jhirsch
Allows sending a command line after startup. Also allows blocking the remoting services
attempt to raise the application to the foreground if desired.

Differential Revision: https://phabricator.services.mozilla.com/D207976
2024-08-14 06:57:53 +00:00
Dave Townsend a1046d62dd Bug 1892400: Remote service should live for the lifetime of the application. r=jhirsch
This lays the groundwork for being able to send remote messages at runtime.

Differential Revision: https://phabricator.services.mozilla.com/D207975
2024-08-14 06:57:52 +00:00
Dave Townsend 8b9c581455 Bug 1892400: Remove the startup token from the non-unix remote implementations. r=jhirsch,glandium
This simplifies the call signatures of all versions since this is only needed in unix.

Differential Revision: https://phabricator.services.mozilla.com/D216032
2024-08-14 06:57:51 +00:00
Dave Townsend bb48be5f26 Bug 1892400: Windows remote client should send the intended command line. r=jhirsch,rkraesig
Currently the windows client sends the current process command line which is
potentially an issue since some of those arguments may have been consumed
already. It will also not support sending custom command lines after startup.

This also switches to sending the UTF-8 command line we already have since that
matches other platforms better and makes some arguments const.

Differential Revision: https://phabricator.services.mozilla.com/D216031
2024-08-14 06:57:51 +00:00
Dave Townsend eb4badec80 Bug 1892400: Remove the unnecessary response and success arguments. r=jhirsch,glandium
A lot of this dates back to when the remoting service was a more fully featured
protocol, now it is just added complication.

Differential Revision: https://phabricator.services.mozilla.com/D207974
2024-08-14 06:57:51 +00:00
Dave Townsend 56b11c9d40 Bug 1792466: Ensure the startup lock directory exists each time we attempt to acquire the startup lock. r=mhowell
When one Firefox instance finishes starting up it releases the startup lock and
deletes the lock directory. Other instances that are waiting to startup are in
the meantime polling to acquire the lock but with the directory gone their next
attempt just throws a failure and so they assume they don't have the lock yet
and keep polling until they hit the five second timeout.

With this patch before each attempt to acquire the lock we make sure that the
lock directory exists first.

Differential Revision: https://phabricator.services.mozilla.com/D161075
2022-11-24 08:50:22 +00:00
Stanca Serban 7e58f49e05 Backed out 2 changesets (bug 1792466) for causing mch fails on test_mochitest_integration.py. CLOSED TREE
Backed out changeset 36cc09178d76 (bug 1792466)
Backed out changeset 418d1ede2ae3 (bug 1792466)
2022-11-17 14:55:21 +02:00
Dave Townsend 732b6ddba8 Bug 1792466: Ensure the startup lock directory exists each time we attempt to acquire the startup lock. r=mhowell
When one Firefox instance finishes starting up it releases the startup lock and
deletes the lock directory. Other instances that are waiting to startup are in
the meantime polling to acquire the lock but with the directory gone their next
attempt just throws a failure and so they assume they don't have the lock yet
and keep polling until they hit the five second timeout.

With this patch before each attempt to acquire the lock we make sure that the
lock directory exists first.

Differential Revision: https://phabricator.services.mozilla.com/D161075
2022-11-17 11:06:47 +00:00
Emilio Cobos Álvarez 6f872e3258 Bug 1767546 - Support XDG_ACTIVATION_TOKEN. r=stransky
Differential Revision: https://phabricator.services.mozilla.com/D159968
2022-11-08 12:43:24 +00:00
Sylvestre Ledru 9ffafce8cc Bug 1519636 - Reformat recent changes to the Google coding style r=andi,media-playback-reviewers,necko-reviewers,padenot,dragana
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D148375
2022-06-07 12:32:55 +00:00
Serge Guelton 51f25dcae7 Bug 1764407 - Cleanup toolkit/components/remote/ includes r=sylvestre
Differential Revision: https://phabricator.services.mozilla.com/D143530
2022-04-20 11:13:20 +00:00
ganguin 7a90e695a1 Bug 1661450 - 1/8 Guard X11 specific code with MOZ_X11 r=stransky
Add MOZ_X11 ifdefs on X11 specific code, mainly alongside GdkIsX11Display.

Differential Revision: https://phabricator.services.mozilla.com/D139526
2022-04-20 09:32:07 +00:00
criss 6d86efe708 Backed out 10 changesets (bug 1661450) for causing build bustages on nsWindow.cpp. CLOSED TREE
Backed out changeset 2c41d82de0c5 (bug 1661450)
Backed out changeset 5f58fcd7ac0b (bug 1661450)
Backed out changeset 62e56a6dcd22 (bug 1661450)
Backed out changeset 4b422ffa729f (bug 1661450)
Backed out changeset 6ca4705772da (bug 1661450)
Backed out changeset 031a6313459f (bug 1661450)
Backed out changeset 06ddf05e97d6 (bug 1661450)
Backed out changeset 4388b1b9aafd (bug 1661450)
Backed out changeset 600f9fd09fa6 (bug 1661450)
Backed out changeset cc5e8efe3ebf (bug 1661450)
2022-04-19 12:25:14 +03:00
ganguin 063d072ce3 Bug 1661450 - 1/8 Guard X11 specific code with MOZ_X11 r=stransky
Add MOZ_X11 ifdefs on X11 specific code, mainly alongside GdkIsX11Display.

Differential Revision: https://phabricator.services.mozilla.com/D139526
2022-04-19 08:35:26 +00:00
stransky 78058d5d7f Bug 1724242 [Linux] Use DBus remote when Firefox is built with --enable-dbus, r=glandium
Differential Revision: https://phabricator.services.mozilla.com/D124692
2021-09-28 16:47:02 +00:00
criss f71e4ba8e2 Backed out changeset 117f94e1e765 (bug 1724242) for causing bustages on Linux x64 asan. CLOSED TREE 2021-09-25 15:05:15 +03:00
stransky a639c69dc0 Bug 1724242 [Linux] Use DBus remote when Firefox is built with --enable-dbus, r=glandium
Differential Revision: https://phabricator.services.mozilla.com/D124692
2021-09-25 10:04:10 +00:00
Cristian Tuns c1653acb1f Backed out changeset 7cbf3890fe9c (bug 1724242) for causing build bustages. CLOSED TREE 2021-09-22 06:12:14 -04:00
stransky 71e33b3d90 Bug 1724242 [Linux] Use DBus remote when Firefox is built with --enable-dbus, r=glandium
Differential Revision: https://phabricator.services.mozilla.com/D124692
2021-09-22 08:10:34 +00:00
stransky 0cd595c18d Bug 1724679 [Linux] Check env variables for '1' to enable them, r=jhorak
Differential Revision: https://phabricator.services.mozilla.com/D122076
2021-08-11 07:28:57 +00:00
Jan Alexander Steffens (heftig) 9354866f9f Bug 1696845 - Use dlsym for gdk_*_display_get_type. r=stransky
Using `dlsym` for `gdk_wayland_display_get_type` is a cleaner solution
to bug 1696319, allowing running with a GTK that lacks the Wayland
backend.

Also adds a symmetric implementation for `gdk_x11_display_get_type`,
which should help running without X11.

Differential Revision: https://phabricator.services.mozilla.com/D107406
2021-03-11 14:32:53 +00:00
Narcis Beleuzu 7cd3bb34b4 Backed out changeset a4f4f640eaa7 (bug 1696845) for bustages on pangofc-fontmap.h 2021-03-10 04:49:51 +02:00
Jan Alexander Steffens (heftig) 0b6c0281d2 Bug 1696845 - Use dlsym for gdk_*_display_get_type. r=stransky
Using `dlsym` for `gdk_wayland_display_get_type` is a cleaner solution
to bug 1696319, allowing running with a GTK that lacks the Wayland
backend.

Also adds a symmetric implementation for `gdk_x11_display_get_type`,
which should help running without X11.

Differential Revision: https://phabricator.services.mozilla.com/D107406
2021-03-09 22:55:47 +00:00
Michael Hofmann 50397e50d8 Bug 1684365 - Add a null pointer check so that Firefox won't crash when it can't initialize mRemoteServer r=stransky
Firefox 84.0.1 crashes under Gentoo Linux if it is started in
Wayland mode and if it was compiled WITH Wayland support and
WITHOUT dbus support.

I traced down the problem to line 172 of
toolkit/components/remote/nsRemoteService.cpp:

   nsresult rv = mRemoteServer->Startup(mProgram.get(), mProfile.get());

mRemoteServer is NULL and Firefox crashes.

This patch adds a NULL pointer check before that line.

See:
- Mozilla's Bugzilla, bug 1684365
- https://bugs.gentoo.org/762035
- https://forums.gentoo.org/viewtopic-t-1126235.html

Differential Revision: https://phabricator.services.mozilla.com/D101536
2021-01-25 09:29:42 +00:00
Simon Giesecke 46908cfb51 Bug 1660470 - Add missing include directives/forward declarations. r=nika
Differential Revision: https://phabricator.services.mozilla.com/D87865
2020-11-23 16:21:38 +00:00
Eric Rahm 6bdf02842c Bug 1617334 - Remove nsAutoPtr usage from toolkit. r=gsvelto
Differential Revision: https://phabricator.services.mozilla.com/D63746
2020-02-22 14:50:52 +00:00
Martin Stransky 201b8155c9 Bug 1609134 Provide MOZ_DBUS_REMOTE env variable to use DBUS remote exclusively, r=jhorak
Differential Revision: https://phabricator.services.mozilla.com/D59872
2020-01-16 10:48:46 +00:00
Yuri 843752fbf8 Bug 1565597 - Mozilla Remoting implementation for macOS, r=mossop
Differential Revision: https://phabricator.services.mozilla.com/D56997
2019-12-13 09:20:24 +00:00
Noemi Erli 5f09bf8588 Backed out changeset 7c1213a74d28 (bug 1565597) for causing Linux bustages in RemoteUtils.h CLOSED TREE 2019-12-13 00:53:33 +02:00
Yuri 0541621400 Bug 1565597 - Mozilla Remoting implementation for macOS, r=mossop
Differential Revision: https://phabricator.services.mozilla.com/D56997
2019-12-12 22:40:10 +00:00
Gabriele Svelto f03cdb2b2d Bug 1600545 - Remove useless inclusions of header files generated from IDL files in modules/, netwerk/, parser/, security/, startupcache/, storage/, toolkit/, tools/, uriloader/, widget/, xpcom/ and xpfe/ r=Ehsan
The inclusions were removed with the following very crude script and the
resulting breakage was fixed up by hand. The manual fixups did either
revert the changes done by the script, replace a generic header with a more
specific one or replace a header with a forward declaration.

find . -name "*.idl" | grep -v web-platform | grep -v third_party | while read path; do
    interfaces=$(grep "^\(class\|interface\).*:.*" "$path" | cut -d' ' -f2)
    if [ -n "$interfaces" ]; then
        if [[ "$interfaces" == *$'\n'* ]]; then
          regexp="\("
          for i in $interfaces; do regexp="$regexp$i\|"; done
          regexp="${regexp%%\\\|}\)"
        else
          regexp="$interfaces"
        fi
        interface=$(basename "$path")
        rg -l "#include.*${interface%%.idl}.h" . | while read path2; do
            hits=$(grep -v "#include.*${interface%%.idl}.h" "$path2" | grep -c "$regexp" )
            if [ $hits -eq 0 ]; then
                echo "Removing ${interface} from ${path2}"
                grep -v "#include.*${interface%%.idl}.h" "$path2" > "$path2".tmp
                mv -f "$path2".tmp "$path2"
            fi
        done
    fi
done

Differential Revision: https://phabricator.services.mozilla.com/D55444
2019-12-06 09:17:57 +00:00
Dave Townsend b7ab48abcc Bug 1553781: When the remote service finds an instance to remote to it doesn't clean up the startup lock. r=glandium
This fixes the issue in a few redundant ways:
* nsProfileLock is made to properly clean itself up when destroyed.
* nsRemoteService makes sure the unlock when destroyed.
* nsAppRunner unlocks when a remote client has been found.

Differential Revision: https://phabricator.services.mozilla.com/D32360
2019-05-30 17:53:36 +00:00
Liang-Heng Chen 0cb4bb739f Bug 1539641 - Log and report storage access granted reason; r=Ehsan,chutten
Differential Revision: https://phabricator.services.mozilla.com/D26720
2019-04-12 00:38:57 +00:00
Sylvestre Ledru b31009da86 Bug 1519636 - Reformat recent changes to the Google coding style r=Ehsan
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D22514
2019-03-13 10:19:06 +01:00