Commit Graph
138 Commits
Author SHA1 Message Date
Keith Cirkel 0c5749c30d Bug 2029974 - Implement new stacking logic for popover=hint r=dom-core,farre,smaug
Differential Revision: https://phabricator.services.mozilla.com/D292397
2026-05-20 16:14:48 +00:00
Atila Butkovits 03cacc0e27 Revert "Bug 2029974 - Implement new stacking logic for popover=hint r=dom-core,farre,smaug" for causing leaks complaining about BaseProfilerMarkersPrerequisites.h.
This reverts commit f74dd68fbe.
2026-05-20 10:56:22 +00:00
Keith Cirkel f74dd68fbe Bug 2029974 - Implement new stacking logic for popover=hint r=dom-core,farre,smaug
Differential Revision: https://phabricator.services.mozilla.com/D292397
2026-05-20 09:03:45 +00:00
Leo Tenenbaum e883659858 Bug 1832869 - Only restore focus to previouslyFocusedElement if current focus is in dialog. r=dom-core,avandolder
Differential Revision: https://phabricator.services.mozilla.com/D289611
2026-04-01 21:40:31 +00:00
Sylvestre Ledru 9eb1a31a98 Bug 2023419 - Remove Emacs/Vim modelines from dom/ r=emilio,media-playback-reviewers,webidl,padenot
replaced by .editorconfig
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D287843
2026-03-16 21:02:32 +00:00
Keith Cirkel 372945d811 Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug
Differential Revision: https://phabricator.services.mozilla.com/D212580
2026-02-07 17:04:15 +00:00
Keith Cirkel d019676b9c Bug 2014549 - Refactor Optional<nsAString> to Maybe<nsAutoString> HTMLDialogElement r=dom-core,smaug
Differential Revision: https://phabricator.services.mozilla.com/D281977
2026-02-06 21:02:11 +00:00
Norisz Fay daf8354afc Revert "Bug 1867743 - Part 8: Implement accessible relations for hint popovers r=accessibility-platform-reviewers,Jamie" for causing reftest failures @ GlobalStyleSheetCache.cpp
This reverts commit aea2f63909.

Revert "Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug"

This reverts commit 27dc6421fb.
2026-02-06 00:35:00 +00:00
Keith Cirkel 27dc6421fb Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug
Differential Revision: https://phabricator.services.mozilla.com/D212580
2026-02-05 21:31:03 +00:00
agoloman 9e0669996c Revert "Bug 1867743 - Part 8: Implement accessible relations for hint popovers r=accessibility-platform-reviewers,Jamie" for causing ba failures @browser_relations_domain_popoverinvokerisdetails.js.
This reverts commit b72f9faaf4.

Revert "Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug"

This reverts commit fdb65c5972.
2026-02-03 17:45:29 +00:00
Keith Cirkel fdb65c5972 Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug
Differential Revision: https://phabricator.services.mozilla.com/D212580
2026-02-03 16:01:32 +00:00
Serban Stanca f0c9c81272 Revert "Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug" for causing linting opt failures.
This reverts commit 1714552e61.
2026-02-02 18:46:10 +00:00
Keith Cirkel 1714552e61 Bug 1867743 - Part 7: Implement popover=hint r=dom-core,smaug
Differential Revision: https://phabricator.services.mozilla.com/D212580
2026-02-02 17:59:22 +00:00
Keith Cirkel 6d5ada44aa Bug 1867743 - Part 6: Refactor popover methods to no longer hardcode to Auto mode r=dom-core,smaug
The methods CloseEntirePopoverList, HidePopoverStackUntil, AutoPopoverList,
GetTopmostAutoPopover, and IsAutoPopover all either take a popover list (of hint
or auto) or have equivalent methods (e.g. HintPopoverList). In order to
introduce popover=hint, this change refactors these methods to take a
PopoverAttributeState which determines the "opened in mode" to filter the
popover list by, ensuring that these methods can act on either Auto or Hint
popovers.

This change has no behavioural changes - everything should function the
same - but it is preparatory work for popover=hint.

Differential Revision: https://phabricator.services.mozilla.com/D276372
2025-12-15 22:56:49 +00:00
Keith Cirkel c27e0af2ec Bug 1990014 - Re-order dialog attr changed steps so that cleanup steps always occur r=smaug,dom-core
This is causing assertion failures due to the Dialog not being removed
from the open dialogs list when it is not open.

Differential Revision: https://phabricator.services.mozilla.com/D265771
2025-10-01 08:49:32 +00:00
Keith Cirkel 90c4ed44a7 Bug 1968987 - Part 2 - Set ToggleEvent source on Popover/Dialog r=smaug,dom-core
This uses the implicit "invoker" from each element to set the
ToggleEvent Source, so that toggle events dispatched from Dialogs &
Popovers include the `source` attribute.

Differential Revision: https://phabricator.services.mozilla.com/D265435
2025-09-30 16:33:25 +00:00
agoloman 654ef74068 Revert "Bug 1968987: apply code formatting via Lando" for causing wpt failures @details-toggle-source.html.
This reverts commit 4c94fd990e.

Revert "Bug 1968987 - Part 2 - Set ToggleEvent source on Popover/Dialog r=smaug,dom-core"

This reverts commit f322cfc77f.

Revert "Bug 1968987 - Part 1 - Add Source to ToggleEvent, no longer generate. r=smaug,dom-core,webidl,saschanaz"

This reverts commit 7df5d3019b.
2025-09-30 16:25:03 +00:00
Keith Cirkel f322cfc77f Bug 1968987 - Part 2 - Set ToggleEvent source on Popover/Dialog r=smaug,dom-core
This uses the implicit "invoker" from each element to set the
ToggleEvent Source, so that toggle events dispatched from Dialogs &
Popovers include the `source` attribute.

Differential Revision: https://phabricator.services.mozilla.com/D265435
2025-09-30 15:00:28 +00:00
Serban Stanca 9ecc18eebf Revert "Bug 1968987 - fix linting failure in details-toggle-source.html." for causing wpt failures in details-toggle-source.html .
This reverts commit 37b12c48c7.

This reverts commit a9742e4fb6.

This reverts commit abd05d12c3.

This reverts commit d251e1c53c.
2025-09-23 12:37:58 +00:00
Keith Cirkel abd05d12c3 Bug 1968987 - Part 2 - Set ToggleEvent source on Popover/Dialog r=smaug,dom-core
This uses the implicit "invoker" from each element to set the
ToggleEvent Source, so that toggle events dispatched from Dialogs &
Popovers include the `source` attribute.

Differential Revision: https://phabricator.services.mozilla.com/D265435
2025-09-23 08:45:08 +00:00
Alexandru Marc 4dd97b58ef Revert "Bug 1968987 - Part 2 - Set ToggleEvent source on Popover/Dialog r=smaug,dom-core" for causing build bustages @ ToggleEvent.cpp
This reverts commit f66dd32b15.

Revert "Bug 1968987 - Part 1 - Add Source to ToggleEvent, no longer generate. r=smaug,dom-core,webidl,saschanaz"

This reverts commit 89ee599ac8.
2025-09-22 16:47:49 +00:00
Keith Cirkel f66dd32b15 Bug 1968987 - Part 2 - Set ToggleEvent source on Popover/Dialog r=smaug,dom-core
This uses the implicit "invoker" from each element to set the
ToggleEvent Source, so that toggle events dispatched from Dialogs &
Popovers include the `source` attribute.

Differential Revision: https://phabricator.services.mozilla.com/D265435
2025-09-22 15:27:39 +00:00
Keith Cirkel 76027b3aad Bug 1987440 - Tidy up HTMLDialogElement.cpp r=smaug,dom-core
This mostly adds code comments, slightly reorganises some code (in
non-user impacting ways) and removes some TODOs which are now done.

Differential Revision: https://phabricator.services.mozilla.com/D264097
2025-09-16 08:49:49 +00:00
Sandor Molnar d77b410bd9 Revert "Bug 1987440 - Tidy up HTMLDialogElement.cpp r=smaug,dom-core" for causing wpt failures @ /html/semantics/the-button-element/command-and-commandfor/on-dialog-behavior.html
This reverts commit 7bf002d547.
2025-09-11 17:24:38 +00:00
Keith Cirkel 7bf002d547 Bug 1987440 - Tidy up HTMLDialogElement.cpp r=smaug,dom-core
This mostly adds code comments, slightly reorganises some code (in
non-user impacting ways) and removes some TODOs which are now done.

Differential Revision: https://phabricator.services.mozilla.com/D264097
2025-09-11 10:02:34 +00:00
Serban Stanca f01d101e1c Revert "Bug 1987440 - Tidy up HTMLDialogElement.cpp r=smaug,dom-core" for causing wpt failures.
This reverts commit 8ce5433353.
2025-09-11 02:12:28 +00:00
Keith Cirkel 8ce5433353 Bug 1987440 - Tidy up HTMLDialogElement.cpp r=smaug,dom-core
This mostly adds code comments, slightly reorganises some code (in
non-user impacting ways) and removes some TODOs which are now done.

Differential Revision: https://phabricator.services.mozilla.com/D264097
2025-09-11 00:41:26 +00:00
Emilio Cobos Álvarez 791917d8af Bug 1976665 - Sort headers in dom/. r=dom-core,media-playback-reviewers,webrtc-reviewers,dom-storage-reviewers,padenot,janv,smaug,jesup,jib
Most changes are automated. Relevant bit is dom/.clang-format. I had to
manually disable clang-format in a few places where importing external
files and so on that aren't self-contained, and fix a couple includes /
forward declarations.

Differential Revision: https://phabricator.services.mozilla.com/D256800
2025-08-05 11:47:34 +00:00
Olli Pettay 0b460dc40d Bug 1923880, ParentNode.moveBefore(), r=edgar,emilio
iframe tests neeeded tweak because the changes for about:blank haven't landed quite yet.
Render blocking and such aren't supported, so the .ini files are still there.
Selection test seemed to be very chromium specific.

Differential Revision: https://phabricator.services.mozilla.com/D255463
2025-07-21 16:33:44 +00:00
iulian moraru d9d5a440a7 Revert "Bug 1923880, ParentNode.moveBefore(), r=edgar,emilio" for causing non-unified build bustages on nsGenericHTMLFrameElement.cpp.
This reverts commit feac8d52fc.
2025-07-21 10:49:53 +00:00
Olli Pettay feac8d52fc Bug 1923880, ParentNode.moveBefore(), r=edgar,emilio
iframe tests neeeded tweak because the changes for about:blank haven't landed quite yet.
Render blocking and such aren't supported, so the .ini files are still there.
Selection test seemed to be very chromium specific.

Differential Revision: https://phabricator.services.mozilla.com/D255463
2025-07-21 09:37:38 +00:00
Keith Cirkel 28201964af Bug 1950359 - Implement request-close invoker command r=dom-core,edgar
This Implements the new `request-close` invoker command, as specified
recently in https://github.com/whatwg/html/pull/11045

Differential Revision: https://phabricator.services.mozilla.com/D256032
2025-07-04 15:43:49 +00:00
Keith Cirkel 3e6e9a0dab Bug 1975525 - Pass button value when using close command r=dom-core,edgar
Differential Revision: https://phabricator.services.mozilla.com/D256031
2025-07-04 15:43:47 +00:00
Keith Cirkel 818ce65d6b Bug 1970331 - Drop "Auto" value from command&commandfor r=emilio,devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D254685
2025-06-26 10:34:08 +00:00
Sandor Molnar e053dddd84 Revert "Bug 1970331 - Drop "Auto" value from command&commandfor r=emilio" for causing perma dt failures @ browser_dbg-event-breakpoints.js.js
This reverts commit e3bc176127.
2025-06-23 20:48:50 +00:00
Keith Cirkel e3bc176127 Bug 1970331 - Drop "Auto" value from command&commandfor r=emilio
Differential Revision: https://phabricator.services.mozilla.com/D254685
2025-06-23 19:01:59 +00:00
Keith Cirkel 24964c182e Bug 1966738 - Part 1: Relocate logic for adding a CloseWatcher to the window r=smaug
This allows us to add more capabilities to this routine (such as
notifying other classes that there is an active CloseWatcher) without
needing to add too much duplication to the codebase.

Differential Revision: https://phabricator.services.mozilla.com/D249647
2025-06-23 08:14:19 +00:00
Lando c3c6824fd2 Bug 1950338: apply code formatting via Lando
# ignore-this-changeset
2025-06-12 15:01:02 +00:00
Keith Cirkel d78e18b0a5 Bug 1950338 - Part 6: Rename invoker functions to command functions r=dom-core,masayuki
Renames:
- IsValidInvokeAction->IsValidCommandAction
- HandleInvokeInternal->HandleCommandInternal
- InvokeAction->CommandAction

Differential Revision: https://phabricator.services.mozilla.com/D252516
2025-06-12 15:00:47 +00:00
agoloman 025ccdad29 Revert "Bug 1950338: apply code formatting via Lando" for causing build bustages @CommandEvent.cpp.
This reverts commit d33dcda2c6.

Revert "Bug 1950338 - Part 8: Rename invoketarget->commandfor, invokeaction->command r=webidl,devtools-reviewers,nchevobbe,smaug"

This reverts commit 0fe8930d32.

Revert "Bug 1950338 - Part 7: Move command&commandfor logic to HTMLButtonElement r=smaug"

This reverts commit 5a9255cd0b.

Revert "Bug 1950338 - Part 6: Rename invoker functions to command functions r=dom-core,masayuki"

This reverts commit 76f840a55a.

Revert "Bug 1950338 - Part 5: Remove InvokeEvent, migrate .invoker to CommandEvent .source r=webidl,devtools-reviewers,nchevobbe,emilio,smaug"

This reverts commit f7d2727969.

Revert "Bug 1950338 - Part 4: Move invokeTargetElement, invokeAction to HTMLButtonElement IDL r=webidl,smaug"

This reverts commit a7a025a843.

Revert "Bug 1950338 - Part 3: Expose CommandEvent in commandfor pref; move from chrome-webidl to webidl r=webidl,emilio"

This reverts commit eb133a45e6.

Revert "Bug 1950338 - Part 2: Move invokers meta ini to correct directory r=dom-core,sefeng"

This reverts commit face866adb.

Revert "Bug 1950338 - Part 1: Rename `dom.element.invokers` flag to `dom.element.commandfor` r=webidl,devtools-reviewers,nchevobbe,smaug"

This reverts commit 532bb887b5.
2025-06-12 14:53:31 +00:00
Lando d33dcda2c6 Bug 1950338: apply code formatting via Lando
# ignore-this-changeset
2025-06-12 12:57:45 +00:00
Keith Cirkel 76f840a55a Bug 1950338 - Part 6: Rename invoker functions to command functions r=dom-core,masayuki
Renames:
- IsValidInvokeAction->IsValidCommandAction
- HandleInvokeInternal->HandleCommandInternal
- InvokeAction->CommandAction

Differential Revision: https://phabricator.services.mozilla.com/D252516
2025-06-12 12:57:30 +00:00
Keith Cirkel 6ad6db9d44 Bug 1966907 - Return early instead of asserting window is present r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D249771
2025-05-19 11:15:04 +00:00
Sylvestre Ledru 8ae54ea4f9 Bug 1519636 - Reformat recent changes to the Google coding style r=geckoview-reviewers,profiler-reviewers,win-reviewers,dom-storage-reviewers,nalexander,gstoll,tcampbell,janv,julienw
Updated with clang-format version 19.1.7 (taskcluster-DYvBxDZJRVqTi8E7pTSJAQ)

Differential Revision: https://phabricator.services.mozilla.com/D249880
2025-05-17 19:11:13 +00:00
Jan-Niklas Jaeschke b3debf824d Bug 1963845 - Use Span for EnumTable. r=emilio,smaug
This patch makes the usage of `EnumTable`s safer
by using `Span`s instead of relying on an empty
element at the end.
Also the name has been changed from `EnumTable`
to `EnumTableEntry` to be more precise.

Differential Revision: https://phabricator.services.mozilla.com/D247453
2025-05-06 08:28:23 +00:00
pstanciu 7d79502fbe Revert "Bug 1963845 - Use Span for EnumTable. r=emilio,smaug" for Build Bustages on HTMLSharedListElement.cpp
This reverts commit eebbccf7b9.
2025-05-05 13:48:07 +00:00
Jan-Niklas Jaeschke eebbccf7b9 Bug 1963845 - Use Span for EnumTable. r=emilio,smaug
This patch makes the usage of `EnumTable`s safer
by using `Span`s instead of relying on an empty
element at the end.
Also the name has been changed from `EnumTable`
to `EnumTableEntry` to be more precise.

Differential Revision: https://phabricator.services.mozilla.com/D247453
2025-05-05 12:49:25 +00:00
Keith Cirkel 265b3e9f14 Bug 1963847 - Use ElementState::OPEN in Open() over GetBoolAttr() r=smaug
See: https://phabricator.services.mozilla.com/D235504#inline-1362631

Differential Revision: https://phabricator.services.mozilla.com/D247458
2025-05-02 12:55:06 +00:00
Steve Fink cd64d0fb7d Revert "Bug 1963847 - Use ElementState::OPEN in Open() over GetBoolAttr() r=smaug" for asserting at dom/html/HTMLDialogElement.cpp:287
This reverts commit 527fa0b956.
2025-05-02 11:25:53 +00:00
Keith Cirkel 3dc321db74 Bug 1963839 - Remove dom.element.dialog.toggle_events.enabled flag r=smaug
Differential Revision: https://phabricator.services.mozilla.com/D247452
2025-05-02 10:41:39 +00:00