- Jul 29, 2024
-
-
ffxbld authored
Differential Revision: https://phabricator.services.mozilla.com/D217937
-
- Jul 26, 2024
-
-
Rob Wu authored
Differential Revision: https://phabricator.services.mozilla.com/D217822
-
- Jul 25, 2024
-
-
Rob Wu authored
Differential Revision: https://phabricator.services.mozilla.com/D217450
-
Rob Wu authored
Differential Revision: https://phabricator.services.mozilla.com/D217449
-
- Jul 27, 2024
-
-
Thomas Wisniewski authored
Differential Revision: https://phabricator.services.mozilla.com/D217757
-
- Jul 26, 2024
-
-
Jiaxun Yang authored
Port D197690 for MIPS. Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> Original Revision: https://phabricator.services.mozilla.com/D217672 Differential Revision: https://phabricator.services.mozilla.com/D217779
-
- Jul 24, 2024
-
-
Emilio Cobos Álvarez authored
-
- Jul 23, 2024
-
-
Emilio Cobos Álvarez authored
In the "regressing" bug, I: * renamed zero-scroll-offset.html -> zero-scroll-offset-001.html * created zero-scroll-offset-002.html, which is effectively the same test but with an horizontal offset too. It seems the test I based this off was flaky on Linux. For now, update the expectations to deal with them. Needs more investigation to properly fix tho, but this flaky is not a _real_ regression. Differential Revision: https://phabricator.services.mozilla.com/D217512
-
- Jul 22, 2024
-
-
Emilio Cobos Álvarez authored
Bug 1905426 - Refine zero-scroll-offset scroll anchoring heuristic to match blink. r=hiro, a=dmeehan See link in the comment. We were checking both offsets, but blink only checks one. In this case, the page has a table with `overflow-x: auto` and `overflow-y: hidden`. They implement a custom virtual scrolling on the vertical axis, and scroll-anchoring interacts poorly with it. Adapt zero-scroll-offset.html to test for this. Differential Revision: https://phabricator.services.mozilla.com/D217201
-
Neil Deakin authored
Bug 1904593, don't clear the autofill state when modified by script, r=dom-core,credential-management-reviewers,dimi,sefeng, a=dmeehan It turns out that many sites are modifying the field state immediately after a value has been autofilled, typically to format the value such as add spaces between parts of a credit card number Differential Revision: https://phabricator.services.mozilla.com/D215113
-
- Jul 25, 2024
-
-
ffxbld authored
Differential Revision: https://phabricator.services.mozilla.com/D217740
-
John Schanck authored
Original Revision: https://phabricator.services.mozilla.com/D216847 Differential Revision: https://phabricator.services.mozilla.com/D217737
-
Ryan VanderMeulen authored
-
Masayuki Nakano authored
Bug 1906727 - Make `HTMLEditor::SplitAncestorStyledInlineElementsAt` check attr value type first r=peterv, a=dmeehan Differential Revision: https://phabricator.services.mozilla.com/D215990
-
- Jul 24, 2024
-
-
Kershaw Chang authored
Bug 1909666 - Re-enable network.http.http2.enabled.deps and not send SETTINGS_NO_RFC7540_PRIORITIES, r=necko-reviewers,jesup, a=dmeehan Differential Revision: https://phabricator.services.mozilla.com/D217584
-
- Jul 19, 2024
-
-
Steve Fink authored
Bug 1895951 - Switch IndexedDB to newer ArrayBuffer (or view) APIs. r=dom-storage-reviewers,sfink,jari, a=dmeehan The patch has been authored by Steve Fink, Jari Jalkanen and Jan Varga. Differential Revision: https://phabricator.services.mozilla.com/D211140
-
- Jul 03, 2024
-
-
Jan Varga authored
Bug 1906083 - Add a bool template argument to TypedArray::ProcessData to enable support for large types arrays; r=peterv, a=dmeehan Currently, there isn't a straightforward and safe method to verify the length before invoking TypedArray::ProcessData. If the length isn't checked and exceeds INT32_MAX, it triggers a release assert. The addition of the AllowLargeTypedArrays argument offers a mechanism to validate the length within the processor, allowing for an error to be returned if necessary. Differential Revision: https://phabricator.services.mozilla.com/D215478
-
- Jul 24, 2024
-
-
John Schanck authored
Differential Revision: https://phabricator.services.mozilla.com/D217596
-
Mike Hommey authored
A change in the latest version of binutils broke building pixman for arm. The binutils change: https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=226749d5a6ff0d5c607d6428d6c81e1e7e7a994b This change was committed upstream in https://gitlab.freedesktop.org/pixman/pixman/-/commit/865e6ce00bb79a6b925ed4c2c436e1533e4472aa and will be in the next release. Original Revision: https://phabricator.services.mozilla.com/D217517 Differential Revision: https://phabricator.services.mozilla.com/D217540
-
- Jul 22, 2024
-
-
Emilio Cobos Álvarez authored
Bug 1907789 - Fix some edge cases when handling removal / reconstruction of table captions and col groups. r=dholbert, a=dmeehan Being the last caption is not enough to remove an anonymous table (and table wrapper). Differential Revision: https://phabricator.services.mozilla.com/D216587
-
- Jul 19, 2024
-
-
Harshit Sohaney authored
Bug 1907783 - Use the new clear on shutdown branch to prevent losing pending shutdown items on unexpected shutdown. r=pbz,places-reviewers,Standard8, a=dmeehan Differential Revision: https://phabricator.services.mozilla.com/D216616
-
- Jul 23, 2024
-
-
Alexandre Lissy authored
Differential Revision: https://phabricator.services.mozilla.com/D217227
-
Vincent Hilla authored
Original Revision: https://phabricator.services.mozilla.com/D217233 Differential Revision: https://phabricator.services.mozilla.com/D217452
-
- Jul 19, 2024
-
-
Masayuki Nakano authored
Bug 1897255 - Make `nsIFrame::PeekOffsetForLine` won't cross editing host boundary unless the callers allows that r=emilio, a=dmeehan The problems in the testcase are, `nsIFrame::PeekOffsetForLine` returns a frame for any content node outside the editing host and `nsIFrame::GetLastLeaf` returns a native anonymous subtree node if the frame is for the native anonymous subtree root like `<input>` or `<textarea>`. For fixing the former, the methods need to check whether found frame's editable state **and** whether the editable node is an inclusive descendant of the editing host. However, there are complicated cases with inline editing hosts and elements whose `contenteditable` is set to `false` and this is a new regression for ESR 128. Therefore, we need to fix this without any behavior changes as far as possible. Therefore, this patch basically checks only whether the editing state of found one is editable and/or whether the one is an inclusive descendant of the editing host to avoid `Selection` moves outside the editing host. The reaming complicated cases should be handled in new bugs which blocks bug 1873155. For fixing the latter, `nsIFrame::GetLastLeaf` needs to check the given frame's content is a native anonymous subtree root or not. Differential Revision: https://phabricator.services.mozilla.com/D216371
-
- Jul 18, 2024
-
-
Emilio Cobos Álvarez authored
We rely on positioning ancestors before descendants to get the right position. This was very implicitly enforced before the regressing bug, by reframing on position changes (due to the order StickyScrollContainer::AddFrame was called, ancestors before children). In order to support dynamic sticky changes without reframing, we need to handle the case of an existing ancestor switching position to sticky dynamically. Luckily we already have the right data-structure for that, so the change is rather trivial. Write a testharness test for this because APZ does get the position right, so it's more future-proof against regressions to just test the raw layout values. Differential Revision: https://phabricator.services.mozilla.com/D216858
-
Emilio Cobos Álvarez authored
Remove one totally unused function, and some useless virtual keywords. Differential Revision: https://phabricator.services.mozilla.com/D216860
-
- Jul 19, 2024
-
-
Nika Layzell authored
Bug 1834864 - Select BCG more consistently during COOP+COEP process switches, r=smaug,tabbrowser-reviewers,mak, a=dmeehan Previously it was possible to bypass specific BCG selection based on cross-origin isolated status if the site was allowed to load file URIs using enterprise policies, which could lead to a crash. This patch changes the behaviour such that BCG selection now happens correctly. The site will still not be cross-origin isolated due to being loaded into a file content process. Differential Revision: https://phabricator.services.mozilla.com/D217007
-
- Jul 12, 2024
-
-
Jon Coppeard authored
There are two issues here. The first is that there unexpected marking work at the start of sweeping after entering from the mark phase without yielding. We previously called assertNoMarkingWork() after markUntilBudgetExhausted() in the marking phase so something since then must have added it. As far as I can tell this must be the conditional call to collectNurseryFromMajorGC(), where a post barrier for a pointer cleared during finalization (e.g. for Maps in mapObject::sweepAfterMinorGC) ends up marking something. I'm not sure such barriers are necessary, but for now the safest thing to do is to move this nursery collection to the start of the slice so that it happens before we drain the mark stack. The second issue is that we check the budget and conditionally yield if we enter from the marking state. The comment above this code states that this is not safe since we have not yet started sweeping a sweep group. This check was added in bug 1865383 but was not the main part of the fix. I think we should remove this. I wasn't able to come up with a test case to reproduce this. Differential Revision: https://phabricator.services.mozilla.com/D215925
-
- Jul 23, 2024
-
-
donal meehan authored
-
Beth Rennie authored
Original Revision: https://phabricator.services.mozilla.com/D216601 Differential Revision: https://phabricator.services.mozilla.com/D217304
-
- Jul 22, 2024
-
-
Kershaw Chang authored
Original Revision: https://phabricator.services.mozilla.com/D217324 Differential Revision: https://phabricator.services.mozilla.com/D217337
-
Greg Stoll authored
Original Revision: https://phabricator.services.mozilla.com/D215958 Differential Revision: https://phabricator.services.mozilla.com/D217336
-
Maxx Crawford authored
Bug 1909256 - Update FeatureManifest weatherLocationSearch to use setPref instead of fallbackPref a=RyanVM Original Revision: https://phabricator.services.mozilla.com/D217289 Differential Revision: https://phabricator.services.mozilla.com/D217294
-
Julian Descottes authored
Bug 1909281 - Mark Page.authenticate test as FAIL/PASS due to different values for use_redirect_for_retries a=dmeehan Only required for 128 release and ESR, next releases only have one event for auth attempts. Differential Revision: https://phabricator.services.mozilla.com/D217306
-
Kershaw Chang authored
Differential Revision: https://phabricator.services.mozilla.com/D217025
-
- Jul 15, 2024
-
-
Alexandre Lissy authored
Bug 1899516 - Warn user when missing unprivileged user namespace r=firefox-desktop-core-reviewers ,fluent-reviewers,Gijs,flod, a=dmeehan, l10n=bolsson Differential Revision: https://phabricator.services.mozilla.com/D212024
-
- Jul 18, 2024
-
-
Jan de Mooij authored
Differential Revision: https://phabricator.services.mozilla.com/D214959
-
- Jul 17, 2024
-
-
smayya authored
Bug 1908115 - disable network.auth.use_redirect_for_retries for release. r=necko-reviewers,kershaw, a=dmeehan Differential Revision: https://phabricator.services.mozilla.com/D216816
-
- Jul 04, 2024
-
-
Paul Bone authored
Differential Revision: https://phabricator.services.mozilla.com/D212231
-
Paul Bone authored
Differential Revision: https://phabricator.services.mozilla.com/D212230
-