Duncan McIntosh
f1eee99f1d
Bug 2035732 - Don't check whether pinning is allowed before checking if the browser is pinned. r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D298355
2026-05-08 16:20:46 +00:00
Harshit
75c6fdd3e5
Bug 2035660 - Condition the IOpenWith api flag based on windows versions. r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D297956
2026-05-05 00:56:26 +00:00
Nicholas Rishel
5d6644eba8
Bug 2033172 - Add option to request pin to taskbar, without waiting for the request. r=cdupuis
...
Differential Revision: https://phabricator.services.mozilla.com/D295124
2026-04-18 02:30:40 +00:00
Sylvestre Ledru
bbcc7547a3
Bug 2023419 - Remove Emacs/Vim modelines from browser/ r=emilio,mkaply,webcompat-reviewers,extension-reviewers,credential-management-reviewers,fxview-reviewers,sessionstore-reviewers,tabbrowser-reviewers,desktop-theme-reviewers,win-reviewers,omc-reviewers,migration-reviewers,home-newtab-reviewers,ai-frontend-reviewers,browser-installer-reviewers,profiles-reviewers,ip-protection-reviewers,mconley,sfoster,twisniewski,thecount,flozia,gstoll,sthompson,mimi,kpatenio,rpl,dimi,mossop,akulyk,aborondo
...
Differential Revision: https://phabricator.services.mozilla.com/D287835
2026-04-17 22:55:57 +00:00
Antonio Borondo
27d957a99e
Bug 2022767 - Focus the set default browser button in Windows Settings from Firefox r=cdupuis,Jamie,nika
...
Differential Revision: https://phabricator.services.mozilla.com/D290084
2026-04-10 06:52:12 +00:00
Nika Layzell
5f0f9dd22a
Bug 2020817 - Part 4: Annotate & clean up nsACString callers of BeginReading/Data, r=xpcom-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,toolkit-telemetry-reviewers-rotation,layout-reviewers,dom-worker-reviewers,gfx-reviewers,webgpu-reviewers,necko-reviewers,dom-core,smaug,handyman,emilio,pehrsons,kershaw,timhuang,edenchuang,padenot,bradwerth
...
This is a large number of largely mechanical changes. When possible, the code
was called to directly call `.get()` instead, performing the assertions
manually. In many places PromiseFlatCString() calls were added to dynamically
copy the strings into null terminated buffers if the string was not null
terminated.
Differential Revision: https://phabricator.services.mozilla.com/D286158
2026-03-25 08:23:04 +00:00
Harshit Sohaney
0270278573
Bug 2019409 - Provide an api to launch the open with dialog on windows with a fallback to ms-settings. r=nrishel,omc-reviewers,aminomancer,drubino
...
Differential Revision: https://phabricator.services.mozilla.com/D287228
2026-03-20 19:39:24 +00:00
rperta
93d1074012
Revert "Bug 2023419 - Remove Emacs/Vim modelines from browser/ r=emilio,mkaply,webcompat-reviewers,fxview-reviewers,sessionstore-reviewers,tabbrowser-reviewers,desktop-theme-reviewers,win-reviewers,migration-reviewers,home-newtab-reviewers,ai-frontend-reviewers,mconley,twisniewski,thecount,flozia,gstoll,sthompson" for causing node related failures
...
This reverts commit 3ae5875024 .
2026-03-16 20:46:05 +00:00
Sylvestre Ledru
3ae5875024
Bug 2023419 - Remove Emacs/Vim modelines from browser/ r=emilio,mkaply,webcompat-reviewers,fxview-reviewers,sessionstore-reviewers,tabbrowser-reviewers,desktop-theme-reviewers,win-reviewers,migration-reviewers,home-newtab-reviewers,ai-frontend-reviewers,mconley,twisniewski,thecount,flozia,gstoll,sthompson
...
replaced by .editorconfig
# ignore-this-changeset
Differential Revision: https://phabricator.services.mozilla.com/D287835
2026-03-16 20:06:36 +00:00
Antonio Borondo
07d3102cfa
Bug 1908298 - Fix IsDefaultBrowser for MSIX builds r=cdupuis
...
Differential Revision: https://phabricator.services.mozilla.com/D285970
2026-03-09 07:33:41 +00:00
Timothy Nikkel
884a9c4296
Bug 2009131. Use async_notify and bilinear filtering for image Draw/GetFrame calls. r=gfx-reviewers,aosmond
...
We always want to user aync notifications. The filtering controls how we draw if we use a pre-existing surface that has the wrong size, but in this case that is unlikely to happen due to the sync decode flag, but it would make the most sense if it did. We should update this to use best practices in case this code gets copied (which is exactly how this issue came up).
Differential Revision: https://phabricator.services.mozilla.com/D278253
2026-01-20 13:55:33 +00:00
Duncan McIntosh
ef8dd2aa3e
Bug 2003933 - Replace the C++ implementation of createWindowsIcon with a JavaScript implementation. r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D275002
2026-01-19 19:38:01 +00:00
serge-sans-paille
7662b3abdc
Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch
...
It is replaced by a standard-conforming cast to (void) which is
understood the same by the compiler.
In some cases this could be replaced by `[[maybe_unused]]` but for
expression whose value is ignored, this requires storing to a temporary
variable first, so not going that way.
One notable change is that marking an object of type already_AddRefed as
unused used to implicitly leak memory. This is now made explicit.
Most of the heavy lifting was done through:
$ git grep -l '\<Unused\> <<' | xargs sed -i -e 's/Unused << /(void)/g'
$ git grep -l mozilla/Unused.h | xargs sed -i -e '/mozilla\/Unused.h/ d'
Differential Revision: https://phabricator.services.mozilla.com/D268706
2025-10-21 19:36:01 +00:00
Cristina Horotan
8a1b1970b7
Revert "Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch" for causing Gtest failures on AlreadyAddRefed.h
...
This reverts commit e98adca292 .
2025-10-21 16:16:29 +00:00
serge-sans-paille
e98adca292
Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch
...
It is replaced by a standard-conforming cast to (void) which is
understood the same by the compiler.
In some cases this could be replaced by `[[maybe_unused]]` but for
expression whose value is ignored, this requires storing to a temporary
variable first, so not going that way.
One notable change is that marking an object of type already_AddRefed as
unused used to implicitly leak memory. This is now made explicit.
Most of the heavy lifting was done through:
$ git grep -l '\<Unused\> <<' | xargs sed -i -e 's/Unused << /(void)/g'
$ git grep -l mozilla/Unused.h | xargs sed -i -e '/mozilla\/Unused.h/ d'
Differential Revision: https://phabricator.services.mozilla.com/D268706
2025-10-21 15:05:17 +00:00
Cristina Horotan
aab924e90d
Revert "Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch" for causing build bustage at nsBrowserApp.cpp
...
This reverts commit fd0ffa193c .
2025-10-21 10:44:13 +00:00
serge-sans-paille
fd0ffa193c
Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch
...
It is replaced by a standard-conforming cast to (void) which is
understood the same by the compiler.
In some cases this could be replaced by `[[maybe_unused]]` but for
expression whose value is ignored, this requires storing to a temporary
variable first, so not going that way.
One notable change is that marking an object of type already_AddRefed as
unused used to implicitly leak memory. This is now made explicit.
Most of the heavy lifting was done through:
$ git grep -l '\<Unused\> <<' | xargs sed -i -e 's/Unused << /(void)/g'
$ git grep -l mozilla/Unused.h | xargs sed -i -e '/mozilla\/Unused.h/ d'
Differential Revision: https://phabricator.services.mozilla.com/D268706
2025-10-21 09:44:11 +00:00
Narcis Beleuzu
e1e6bbb1c9
Revert "Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch" for causing bustages on Document.cpp
...
This reverts commit b23c15e927 .
2025-10-21 06:34:58 +00:00
serge-sans-paille
b23c15e927
Bug 1994397 - Remove obsolete Unused.h header r=emilio,tschuster,geckoview-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,cookie-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,permissions-reviewers,profiles-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,firefox-svg-reviewers,layout-reviewers,timhuang,gstoll,TravisLong,bytesized,mstange,karlt,robwu,pehrsons,m_kato,jari,jhirsch
...
It is replaced by a standard-conforming cast to (void) which is
understood the same by the compiler.
In some cases this could be replaced by `[[maybe_unused]]` but for
expression whose value is ignored, this requires storing to a temporary
variable first, so not going that way.
One notable change is that marking an object of type already_AddRefed as
unused used to implicitly leak memory. This is now made explicit.
Most of the heavy lifting was done through:
$ git grep -l '\<Unused\> <<' | xargs sed -i -e 's/Unused << /(void)/g'
$ git grep -l mozilla/Unused.h | xargs sed -i -e '/mozilla\/Unused.h/ d'
Differential Revision: https://phabricator.services.mozilla.com/D268706
2025-10-21 06:01:36 +00:00
serge-sans-paille
418a8f52a1
Bug 1994393 - Remove unused inclusion of mozilla/UniquePtrExtensions.h r=emilio
...
Differential Revision: https://phabricator.services.mozilla.com/D268683
2025-10-15 19:49:46 +00:00
agoloman
68648b85d8
Revert "Bug 1985098 - Replace createWindowsIcon with nsIShellService.createIcon and implement it for Linux. r=nrishel,mossop" for causing xpc failures @test_createIcon.js.
...
This reverts commit d6e686af50 .
2025-09-02 22:26:17 +00:00
Duncan McIntosh
d6e686af50
Bug 1985098 - Replace createWindowsIcon with nsIShellService.createIcon and implement it for Linux. r=nrishel,mossop
...
Differential Revision: https://phabricator.services.mozilla.com/D263033
2025-09-02 21:45:48 +00:00
Nicholas Rishel
74292bb77c
Bug 1972537 - Part 2: Prefer 256x256 for the SourceSurface used to generate Windows ICO images. r=dmcintosh
...
This resolves scaling issues seen with images generated from SVGs.
Differential Revision: https://phabricator.services.mozilla.com/D260554
2025-08-15 22:01:11 +00:00
Duncan McIntosh
1c7f55d8d8
Bug 1976020 - Part 2: Remove nsIWindowsShellService.getTaskbarTabShortcutPath and nsIWindowsShellService.getTaskbarTabPins. r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D256311
2025-07-24 22:59:14 +00:00
Duncan McIntosh
ddd2725a59
Bug 1976020 - Part 1: Use relative paths when pinning and unpinning shortcuts. r=nrishel
...
This removes the dependency on getTaskbarTabShortcutPath when unpinning,
freeing the way to remove it entirely.
Differential Revision: https://phabricator.services.mozilla.com/D257175
2025-07-24 22:59:11 +00:00
Duncan McIntosh
809b0fea6b
Bug 1974880 - Implement a function to delete shortcuts and remove them from the shortcut log. r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D256310
2025-07-24 22:59:07 +00:00
Duncan McIntosh
c9701ad186
Bug 1938967 - Part 2: Use relative paths within Taskbar Tabs to create shortcuts. r=nrishel,fluent-reviewers,bolsson
...
Differential Revision: https://phabricator.services.mozilla.com/D254747
2025-07-17 17:30:38 +00:00
Duncan McIntosh
3644a62b47
Bug 1938967 - Part 1: Allow shortcuts to be located in subdirectories of the Start menu or desktop. r=nrishel
...
This is intended for Taskbar Tabs, since it will allow all shortcuts for
taskbar tabs to be placed into a single folder. It also removes the
limit of ten shortcuts in a given folder, which matches the
uninstaller's behaviour; notwithstanding an INI parsing bug, it will
always terminate.
The uninstaller is already able to handle this; no changes are needed
there.
Differential Revision: https://phabricator.services.mozilla.com/D252417
2025-07-17 17:30:35 +00:00
Alexandru Marc
93fa021ab9
Revert "Bug 1938967 - Part 2: Use relative paths within Taskbar Tabs to create shortcuts. r=nrishel,fluent-reviewers,bolsson" for causing bc failures @ browser_createWindowsShortcut.js
...
This reverts commit 76c91e29a4 .
Revert "Bug 1938967 - Part 1: Allow shortcuts to be located in subdirectories of the Start menu or desktop. r=nrishel"
This reverts commit 17f3a92d47 .
2025-07-14 23:34:48 +00:00
Duncan McIntosh
76c91e29a4
Bug 1938967 - Part 2: Use relative paths within Taskbar Tabs to create shortcuts. r=nrishel,fluent-reviewers,bolsson
...
Differential Revision: https://phabricator.services.mozilla.com/D254747
2025-07-14 17:24:59 +00:00
Duncan McIntosh
17f3a92d47
Bug 1938967 - Part 1: Allow shortcuts to be located in subdirectories of the Start menu or desktop. r=nrishel
...
This is intended for Taskbar Tabs, since it will allow all shortcuts for
taskbar tabs to be placed into a single folder. It also removes the
limit of ten shortcuts in a given folder, which matches the
uninstaller's behaviour; notwithstanding an INI parsing bug, it will
always terminate.
The uninstaller is already able to handle this; no changes are needed
there.
Differential Revision: https://phabricator.services.mozilla.com/D252417
2025-07-14 17:24:56 +00:00
Francesco Lodolo
8e18c07c8c
Bug 1969793 - Correct path to setDesktopBackground.ftl in nsWindowsShellService, r=saschanaz
...
Differential Revision: https://phabricator.services.mozilla.com/D252151
2025-06-02 15:04:42 +00:00
jilvin
0e104cbe54
Bug 1902439 - Checks return of nsIOutputStream::Write() to NS_OK in WriteBitmap() to handle failures better. r=mpohle,nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D234318
2025-05-20 17:37:48 +00:00
Francesco Lodolo (:flod)
88fdda4cc7
Bug 1955747 - Migrate strings in shellservice.properties to Fluent, r=nrishel,eemeli
...
Differential Revision: https://phabricator.services.mozilla.com/D242977
2025-04-02 05:42:44 +00:00
Eric Chen
67bb7296ec
Bug 1955385 - revise MSIX code path for CheckCurrentProcessAUMIDForTesting r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D242588
2025-03-23 19:07:20 +00:00
Eric Chen
192c0c6787
Bug 1950734 - Make MSIX builds skip the AUMID altering steps in taskbar pinning as it's unnecessary r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D239827
2025-03-19 20:41:18 +00:00
Cristina Horotan
bb136e0d12
Backed out changeset c7a895ce1e23 (bug 1950734) for causing bc failures at browser_processAUMID.js. CLOSED TREE
2025-03-13 07:31:36 +02:00
Eric Chen
3fa85c5c7a
Bug 1950734 - Make MSIX builds skip the AUMID altering steps in taskbar pinning as it's unnecessary r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D239827
2025-03-12 23:45:49 +00:00
Eric Chen
d4bc2bcaee
Bug 1949263 - prevent MSIX Firefox from calling PollAppsFolderForShortcut r=nrishel
...
Differential Revision: https://phabricator.services.mozilla.com/D239824
2025-03-03 15:34:20 +00:00
Nicholas Rishel
7034f58767
Bug 1932795 - Fix debug build assert by moving SourceSurface::GetFrame call to main thread during nsWindowsShellService::CreateWindowsIcon. r=nalexander,win-reviewers,rkraesig
...
Differential Revision: https://phabricator.services.mozilla.com/D229849
2024-11-27 21:56:01 +00:00
Nicholas Rishel
57ab7f1ebd
Bug 1930048 - Fix MinGW compile errors. r=mpohle,win-reviewers,gstoll
...
Tagalong: Clean up include order and remove unnecessary includes.
Differential Revision: https://phabricator.services.mozilla.com/D229033
2024-11-14 20:45:22 +00:00
Max Christian Pohle
846bcd126a
Bug 1923448 - Generalized Windows Pinning, r=nrishel,gfx-reviewers,nical
...
Differential Revision: https://phabricator.services.mozilla.com/D224975
2024-11-12 22:48:55 +00:00
Nicholas Rishel
2a8da6a369
Bug 1893458 - Verify Start Menu shortcuts created at runtime have propagated to shell:appsfolder before attempting to pin them. r=win-reviewers,rkraesig
...
Differential Revision: https://phabricator.services.mozilla.com/D227131
2024-11-07 21:51:23 +00:00
Stanca Serban
1cefb44ccb
Backed out changeset 2fe294eb4a06 (bug 1923448) for causing nsWindowsShellService.cpp. CLOSED TREE
2024-11-07 12:37:21 +02:00
Max Christian Pohle
423ab686ff
Bug 1923448 - Generalized Windows Pinning, r=nrishel,gfx-reviewers,nical
...
Differential Revision: https://phabricator.services.mozilla.com/D224975
2024-11-07 09:54:52 +00:00
Nicholas Rishel
085457c06f
Bug 1926151 - Remove Desktop and Public Desktop from folders a pinnable shortcut can exist in. r=bhearsum
...
We prioritize using `ITaskbarManager` now, which requires a shortcut exist in a location visible from `shell:appsfolder`. `IPinnedList3` can use shortcuts from Desktop and Public Desktop (and potentially other locations) but there's no reason to check for these cases since we have already created the shortcut for `ITaskbarManager`. The rare case of failing to create as shortcut for `ITaskbarManager` is not worth accounting for given `IPinnedList3` is not supported in newer versions of Windows.
Differential Revision: https://phabricator.services.mozilla.com/D227134
2024-10-31 19:53:56 +00:00
Nicholas Rishel
51fef936bb
Bug 1926151 - Pre: Rename Matching to Pinnable to better convey usage of hasPinnableShortcut. r=bhearsum,firefox-desktop-core-reviewers
...
Differential Revision: https://phabricator.services.mozilla.com/D227133
2024-10-31 19:53:55 +00:00
serge-sans-paille
262f78c07f
Bug 1922838 - Replace ArrayLength, ArrayEnd and MOZ_ARRAY_LENGTH by standard alternative r=glandium,necko-reviewers,jgilbert,application-update-reviewers,media-playback-reviewers,credential-management-reviewers,anti-tracking-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,bytesized,janv,dimi,daisuke,karlt,gstoll,canaltinova,timhuang
...
Namely std::size, std::end and std::size. This drops C support for
MOZ_ARRAY_LENGTH but it wasn't used anyway.
Differential Revision: https://phabricator.services.mozilla.com/D224611
2024-10-28 08:21:19 +00:00
Alexandru Marc
a5eb905bf4
Backed out changeset d92f391b3b0c (bug 1922838) for backing out bug 1915351
2024-10-25 16:42:33 +03:00
serge-sans-paille
07101ac24d
Bug 1922838 - Replace ArrayLength, ArrayEnd and MOZ_ARRAY_LENGTH by standard alternative r=glandium,necko-reviewers,jgilbert,application-update-reviewers,media-playback-reviewers,credential-management-reviewers,anti-tracking-reviewers,places-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,bytesized,janv,dimi,daisuke,karlt,gstoll,canaltinova,timhuang
...
Namely std::size, std::end and std::size. This drops C support for
MOZ_ARRAY_LENGTH but it wasn't used anyway.
Differential Revision: https://phabricator.services.mozilla.com/D224611
2024-10-24 09:06:01 +00:00