Commit Graph

935414 Commits

Author SHA1 Message Date
Timothy Nikkel
888868356b Bug 1923790. Skip checking for perspective in GetResultingTransformMatrix when we already know there is no perspective. r=jwatt
Differential Revision: https://phabricator.services.mozilla.com/D225170
2024-10-14 10:43:06 +00:00
Timothy Nikkel
72090a3700 Bug 1923790. Skip a matrix PostTranslate in GetResultingTransformMatrix when it's useless. r=gfx-reviewers,layout-reviewers,lsalzman,emilio
Differential Revision: https://phabricator.services.mozilla.com/D225169
2024-10-14 10:43:06 +00:00
Jan de Mooij
9e3fce8f78 Bug 1924279 - Add missing ReportOutOfMemory call to selectforgc testing function. r=jonco
This bug was introduced 12 years ago but we're now catching this with new assertions.

Differential Revision: https://phabricator.services.mozilla.com/D225476
2024-10-14 10:42:04 +00:00
Jan de Mooij
725fc75c54 Bug 1924062 - Throw exception for asm.js module functions in disnative testing function. r=arai
A function marked `isAsmJSNative` is either the outer module function or an actual
exported asm.js function. The module function can't be disassembled.

Differential Revision: https://phabricator.services.mozilla.com/D225357
2024-10-14 10:38:58 +00:00
Emilio Cobos Álvarez
87184f51bb Bug 1924240 - Simpler parent / child setup in nsCocoaWindow.mm. r=mac-reviewers,bradwerth
Try seems happy with this.

Differential Revision: https://phabricator.services.mozilla.com/D225425
2024-10-14 10:32:25 +00:00
Luca Greco
d48990e9b9 Bug 1921483 - Explicitly cover expected behavior on clicking the blocklist info url in the install failure dialog. r=willdurand
Differential Revision: https://phabricator.services.mozilla.com/D225384
2024-10-14 10:30:17 +00:00
kycn
f946c39e9f Bug 1907615 - Collect full history from the parent process on SHIP on Android. r=geckoview-reviewers,owlish
Collecting the full history updates from the parent process until the partial history colletion is introduced for SHIP on Android.

Differential Revision: https://phabricator.services.mozilla.com/D221710
2024-10-14 10:26:35 +00:00
Titouan Thibaud
0d9eb3c796 Bug 1887651 - Run baseline profile generation for Fenix on Firebase TestLab and integrate it into Nightly apks r=android-reviewers,releng-reviewers,firefox-build-system-reviewers,nalexander,jcristau,rsainani
Differential Revision: https://phabricator.services.mozilla.com/D221635
2024-10-14 10:08:15 +00:00
Jan Varga
31af6ee2f0 Bug 1866402 - Further reduce code duplication in QuotaManager::OpenStorageDirectory and QuotaManager::OpenClientDirectory; r=dom-storage-reviewers,jari
Differential Revision: https://phabricator.services.mozilla.com/D223710
2024-10-14 10:06:14 +00:00
Jan Varga
61a43c79f8 Bug 1866402 - Reduce code duplication in QuotaManager::OpenStorageDirectory and QuotaManager::OpenClientDirectory; r=dom-storage-reviewers,jari
Differential Revision: https://phabricator.services.mozilla.com/D223460
2024-10-14 10:06:14 +00:00
Jan Varga
9b838294ef Bug 1866402 - Implement initialization of entire persistent storage; r=dom-storage-reviewers,jari
IndexedDB still calls QuotaManager::EnsurePersistentOriginIsInitializedInternal
from its idle database maintenance. However, we want to get rid of that so the
method can be made private. The idle database maintenance should only rely on
QuotaManager::OpenStorageDirectory method to do all necessary initialization.
Before we can do that, we need a method dedicated for initialization of entire
persistent storage.

Differential Revision: https://phabricator.services.mozilla.com/D195653
2024-10-14 10:06:14 +00:00
William Durand
d510f68c17 Bug 1923268 - Update soft/hard block strings. r=rpl,fluent-reviewers,android-reviewers,bolsson,delphine,amejiamarmol
Differential Revision: https://phabricator.services.mozilla.com/D225047
2024-10-14 10:00:26 +00:00
teoxoy
29021a05f3 Bug 1918739 - Update doc comment for wgpu_server_instance_request_adapter. r=webgpu-reviewers,ErichDonGubler
Differential Revision: https://phabricator.services.mozilla.com/D225188
2024-10-14 09:54:48 +00:00
pstanciu
55abcb2995 Backed out changeset 875fb891b6a9 (bug 1922563) for causing build bustages on MathMLElement.cpp CLOSED TREE 2024-10-14 15:11:22 +03:00
serge-sans-paille
6f7cb9dc13 Bug 1923084 - Fix type of gAppData to be the same in declaration and definition. r=firefox-desktop-core-reviewers ,mossop,win-reviewers,gstoll
Differential Revision: https://phabricator.services.mozilla.com/D224753
2024-10-14 09:34:23 +00:00
serge-sans-paille
03a70f29be Bug 1923085 - Make sure nsThreadUtils.h is included before definition of Task r=xpcom-reviewers,emilio
Task inner structure contains an #ifdef from a variable defined in
nsThreadUtils.h.
Otherwise we may end up with different compilation unit have different
vtable (odr violation).

Differential Revision: https://phabricator.services.mozilla.com/D224754
2024-10-14 09:34:23 +00:00
moz-wptsync-bot
b70e9b16a1 Bug 1924292 - [wpt-sync] Update web-platform-tests to c7076ab2397dd3dcf0bb93dc304d05be04cd2a47, a=testonly
MANUAL PUSH: wpt sync bot

wpt-head: c7076ab2397dd3dcf0bb93dc304d05be04cd2a47
wpt-type: landing
2024-10-14 09:15:18 +00:00
Nina Satragno
15a2b2c340 Bug 1924153 [wpt PR 48579] - [fenced-frames] Migrate WebAuthn test, a=testonly
Automatic update from web-platform-tests
[fenced-frames] Migrate WebAuthn test

Migrate a fenced frames WebAuthn test to run on headless chrome with
chromedriver instead of the legacy content shell runner. The legacy
content shell runner will stop supporting the WebAuthn testdriver API,
soon.

Fix the test to set up the virtual authenticator environment on the top
frame instead, since testdriver should be invoked from the same window
hosting the testharness.

Bug: 372169469
Change-Id: I6e1b296fe86ee4087da2a5a63868e18a6b440984
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5920770
Auto-Submit: Nina Satragno <nsatragno@chromium.org>
Reviewed-by: Liam Brady <lbrady@google.com>
Commit-Queue: Liam Brady <lbrady@google.com>
Cr-Commit-Position: refs/heads/main@{#1367535}

--

wpt-commits: 5611c150825562522da2d6be67b929e399211c35
wpt-pr: 48579
2024-10-14 09:15:17 +00:00
Jonathan Lee
465b68f1b6 Bug 1924243 [wpt PR 48582] - [wptrunner] Cycle testdriver event loop when testharness isn't loaded, a=testonly
Automatic update from web-platform-tests
[wptrunner] Cycle testdriver event loop when testharness isn't loaded (#48582)

Workaround for https://crbug.com/340662810 that seems to work
without an increase in test runtime or regressions
(https://crrev.com/c/5698330/9).
--

wpt-commits: 280d04eaabed08d430d12c8af971b0b56f32310f
wpt-pr: 48582
2024-10-14 09:15:16 +00:00
moz-wptsync-bot
4999dfc21a Bug 1924180 [wpt PR 48581] - Update wpt metadata, a=testonly
wpt-pr: 48581
wpt-type: metadata
2024-10-14 09:15:16 +00:00
Keith Cirkel
7d86e06ac7 Bug 1924180 [wpt PR 48581] - Rename & refactor invoker commands to reflect new spec decisions., a=testonly
Automatic update from web-platform-tests
Rename & refactor invoker commands to reflect new spec decisions.

Invoker commands now live solely on HTMLButtonElement, and have been
dropped from HTMLInputElement. This was discussed and resolved + minuted
in Jul 18 WHATNOT meeting:
https://github.com/whatwg/html/issues/10471#issuecomment-2237135334)

Words must be separated with dashes. This was discussed and resolved +
minuted in Jul 18 WHATNOT meeting:
https://github.com/whatwg/html/issues/10471#issuecomment-2237135334)

"Custom" commands now require a double dash (`--`) prefix. This was
discussed & resolved + minuted in the Jul 25 OpenUI meeting:
https://github.com/openui/open-ui/issues/969#issuecomment-2251153158

In addition, CommandEvent#invoker is now renamed to CommandEvent#source
as discussed and resolved + minuted in Jul 18 WHATNOT meeting:
https://github.com/whatwg/html/issues/10471#issuecomment-2237135334)

This CL updates the tests & implementation to follow these changes. The
changes can also be found in the spec PR which reflects this updated
status:

https://github.com/whatwg/html/pull/9841

Bug: 40284894
Change-Id: Iefa91a1736867239a21f8717aee1fcaaa54fdfd0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5839601
Reviewed-by: Mason Freed <masonf@chromium.org>
Reviewed-by: Luke <lwarlow@igalia.com>
Commit-Queue: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1367705}

--

wpt-commits: 5df7a2c02bb50efa0368910fd49242c08def7a5d
wpt-pr: 48581
2024-10-14 09:15:15 +00:00
moz-wptsync-bot
d5d341f2ca Bug 1924174 [wpt PR 48580] - Update wpt metadata, a=testonly
wpt-pr: 48580
wpt-type: metadata
2024-10-14 09:15:14 +00:00
Mason Freed
a14287b74e Bug 1924174 [wpt PR 48580] - Add implicit anchor relationship for popover invokers, a=testonly
Automatic update from web-platform-tests
Add implicit anchor relationship for popover invokers

When PopoverAnchorRelationships is enabled, include popover invokers
as "implicit anchor elements" for anchor positioning.

This also adds a test of the behavior when nesting <select> within
other top layer elements, and vice versa.

Bug: 364669918
Change-Id: Ie1d362a41e3aec72d62dbfe378e65ea9a262c324
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5846489
Auto-Submit: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Joey Arhar <jarhar@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1367631}

--

wpt-commits: 39246a878f6b2e8cd4387896d05318f9e0e1e85a
wpt-pr: 48580
2024-10-14 09:15:13 +00:00
Anne van Kesteren
979a5b58a7 Bug 1924145 [wpt PR 48577] - Shadow trees: more involved dispatch tests, a=testonly
Automatic update from web-platform-tests
Shadow trees: more involved dispatch tests

In particular, this adds tests for when the target get moved out of the shadow tree or into a shadow tree by an event listener.

Co-authored-by: Chris Dumez <cdumez@apple.com>
--

wpt-commits: b330bd718452c3dc9b7f6826ba62d51bf81d5f4d
wpt-pr: 48577
2024-10-14 09:15:12 +00:00
github-actions[bot]
e1ca63ce89 Bug 1924075 [wpt PR 48576] - Sync interfaces/ with @webref/idl 3.55.2, a=testonly
Automatic update from web-platform-tests
Sync interfaces/ with @webref/idl 3.55.2 (#48576)

Co-authored-by: wpt-pr-bot <wpt-pr-bot@users.noreply.github.com>
--

wpt-commits: 1fc69a2cdff44f94964faddebcccf024a9c8b6f0
wpt-pr: 48576
2024-10-14 09:15:11 +00:00
Cathie Chen
e17b6f4527 Bug 1923930 [wpt PR 48565] - Fix check failure at LineBreaker::HandleTextForFastMinContent, a=testonly
Automatic update from web-platform-tests
Fix check failure at LineBreaker::HandleTextForFastMinContent

The DCHECK_EQ fails, because sometimes the decoration width of elements
with `box-decoration-break` is zero, so `position_` will be
equal to `indent`, and the value of `should_break_at_first_opportunity`
will be false. `has_cloned_box_decorations_` is true. So the DCHECK_EQ
fails.

Bug: 362530623
Change-Id: I4523c16089326340c4bab88ae3b674ccb0bdd9ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5873209
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Auto-Submit: cathie chen <cchen@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1367328}

--

wpt-commits: f28c47aeda5fe8c5722e176ba97a3980e507c0c1
wpt-pr: 48565
2024-10-14 09:15:11 +00:00
Kent Tamura
3b713ff653 Bug 1924012 [wpt PR 48575] - css-shapes: Fix flakiness of shape-image-029.html, a=testonly
Automatic update from web-platform-tests
css-shapes: Fix flakiness of shape-image-029.html

Wait until 'support/left-half-rectangle.png' loaded.

Bug: 372252514, 372756068
Change-Id: I106ab5e4de34be91f4365129011aec58f19c936a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5923354
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Auto-Submit: Kent Tamura <tkent@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1367293}

--

wpt-commits: 4630a18e4df986614b1a68b277e70caba40dd66b
wpt-pr: 48575
2024-10-14 09:15:10 +00:00
Shiyi Zou
f2b959d38c Bug 1923968 [wpt PR 48568] - webnn: utilize constant nodes in DirectML backend, a=testonly
Automatic update from web-platform-tests
webnn: utilize constant nodes in DirectML backend

This CL utilizes `DML_GRAPH_NODE_TYPE_CONSTANT` introduced in
`DML_FEATURE_LEVEL_6_2` for constant scalar inputs, which allows CPU
memory to be directly consumed during graph compilation.

Change-Id: I468e600e0b1dae59a0ab017ef7fe86b5e0e00c59
Bug: 366284850
Cq-Include-Trybots: luci.chromium.try​:win11-blink-rel, mac14.arm64-blink-rel, mac14-blink-rel, mac15.arm64-blink-rel, mac15-blink-rel, linux-blink-rel
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5859935
Reviewed-by: Rafael Cintron <rafael.cintron@microsoft.com>
Commit-Queue: Shiyi Zou <shiyi.zou@intel.com>
Reviewed-by: ningxin hu <ningxin.hu@intel.com>
Auto-Submit: Shiyi Zou <shiyi.zou@intel.com>
Cr-Commit-Position: refs/heads/main@{#1367255}

--

wpt-commits: 8903062b181d2fdf406e7b74083f13af59012c15
wpt-pr: 48568
2024-10-14 09:15:09 +00:00
moz-wptsync-bot
ab865d9d9f Bug 1923911 [wpt PR 48564] - Update wpt metadata, a=testonly
wpt-pr: 48564
wpt-type: metadata
2024-10-14 09:15:08 +00:00
Andreu Botella
d6045f1237 Bug 1923911 [wpt PR 48564] - [line-clamp] Hide from paint non-fixed-height abspos after clamp, a=testonly
Automatic update from web-platform-tests
[line-clamp] Hide from paint non-fixed-height abspos after clamp

With the `continue: collapse` variant of `line-clamp`, absolutely
positioned boxes that are after the clamp point in the layout tree
should be hidden from paint. However, abspos with an indefinite block
size were not always being hidden.

The reason for this was that for such abspos,
`OutOfFlowLayoutPart::CalculateOffset` performs an initial layout,
which can end up becoming the final layout of the abspos, which does
not happen for the fixed-height abspos that we had tested previously.
As it turns out, that layout was not checking the `NodeInfo`'s
`is_hidden_for_paint` field to determine whether to hide the fragment.
This change fixes that.

Bug: 40336192
Change-Id: I5876c762d0d9957ea5785660c73528b83de99eb9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5923076
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Andreu Botella <abotella@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1367195}

--

wpt-commits: 0ca2856475e2d067f7f2523ffb30de965c28d14a
wpt-pr: 48564
2024-10-14 09:15:07 +00:00
moz-wptsync-bot
a9419b1e43 Bug 1923975 [wpt PR 48570] - Update wpt metadata, a=testonly
wpt-pr: 48570
wpt-type: metadata
2024-10-14 09:15:06 +00:00
Jonathan Lee
d38d68fc9f Bug 1923975 [wpt PR 48570] - [reporting][wpt] Isolate stashed reports, a=testonly
Automatic update from web-platform-tests
[reporting][wpt] Isolate stashed reports

Several `reporting/` tests use the same wptserve stash entry (report ID
`d0d517bf-891b-457a-b970-8b2b2c81a0bf` [0]). If at least two such tests
are unlucky enough to run in parallel, the first one to poll `report.py`
will cannibalize all of the reports and get too many, while the others
will get too few.

Fix this by regenerating different IDs for all but one of the tests. For
the external WPTs that embed on `middle-frame.https.sub.html`, plumb the
report ID to `same-origin-report.https.sub.html` as a query parameter,
where it can be substituted into the `Reporting-Endpoints` response
header [1].

Continue using hardcoded IDs instead of randomly generated ones so that
reports can be matched with their tests.

[0]: https://source.chromium.org/search?q=d0d517bf-891b-457a-b970-8b2b2c81a0bf%20f:wpt%20&ss=chromium%2Fchromium%2Fsrc
[1]: https://web-platform-tests.org/writing-tests/server-pipes.html#sub

Bug: 338470458
Test: run_web_tests.py external/wpt/reporting wpt_internal/reporting --fully-parallel -j 36
Change-Id: I46d4d9de954d853615b067725a136a65a6df622b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5922396
Reviewed-by: Ian Clelland <iclelland@chromium.org>
Commit-Queue: Jonathan Lee <jonathanjlee@google.com>
Cr-Commit-Position: refs/heads/main@{#1367116}

--

wpt-commits: 3e0163f8505cac8a369aab811179cc9dc0f60ebe
wpt-pr: 48570
2024-10-14 09:15:05 +00:00
serge-sans-paille
430ae418b9 Bug 1922563 - Reduce memory foot print of mozilla::dom::MathMLElement::MapGlobalMathMLAttributesInto()::values r=emilio
Use some kind of prefix table to avoid storing redundant string. Also
Avoid storing an array of enum when there's a direct mapping between
enum values and loop index.

This result in a reduction of 352 byte in the resulting binary, so this
may not be worth the effort, but it was great fun to write that patch.

Differential Revision: https://phabricator.services.mozilla.com/D224494
2024-10-14 09:04:25 +00:00
Julian Descottes
25e6bc6da0 Bug 1886518 - [wdspec] Add wdspec tests for image cache network requests r=webdriver-reviewers,Sasha
Depends on D224505

Cross browser tests are added for basic image cache scenarios.
An additional mozilla specific wdspec test is added to cover scenarios where the behavior is implementation defined.

Differential Revision: https://phabricator.services.mozilla.com/D224506
2024-10-14 08:51:49 +00:00
Julian Descottes
b21e6d226e Bug 1886518 - [remote] Add cached resource test for images and scripts r=webdriver-reviewers,Sasha
Differential Revision: https://phabricator.services.mozilla.com/D224505
2024-10-14 08:51:48 +00:00
Mugurell
df1da866ca Bug 1917818 - part 2 - Improve browser layout with top toolbar updates when the keyboard is animated r=android-reviewers,skhan
Previously when the keyboard would be showing the entire screen will first resize to the end height
after which the keyboard will be animated over an empty content.
With these changes we postpone the resize to after the keyboard animation is finished to ensure a
smooth animation over the web content instead of over empty space.

Differential Revision: https://phabricator.services.mozilla.com/D223586
2024-10-14 08:27:42 +00:00
Mugurell
5c92bf47ef Bug 1917818 - part 1 - Synchronize showing browser's bottom toolbar on top of the keyboard r=android-reviewers,sfamisa
Previously when the keyboard would be showing / hiding the entire screen will first resize to
the end height after which the keyboard will be animated.
This meant just one reflow for the engine view but a choppy layout animation.

With the new changes we'll ensure the bottom toolbar is animated in sync with the keyboard
and the engine view is resized only once
- before hiding the keyboard to ensure that it will then nicely reveal the web content
- after showing the full keyboard to ensure it gradually covered web content with no UI issue

Differential Revision: https://phabricator.services.mozilla.com/D223585
2024-10-14 08:27:42 +00:00
Mugurell
a5dba92c22 Bug 1911042 - part 2 - Synchronize showing the search toolbar on top of the keyboard r=android-reviewers,sfamisa
Differential Revision: https://phabricator.services.mozilla.com/D223155
2024-10-14 08:27:41 +00:00
Mugurell
855fe61c73 Bug 1911042 - part 1 - Add a way to set up manually handling persistent insets for any Window r=android-reviewers,sfamisa
Differential Revision: https://phabricator.services.mozilla.com/D223154
2024-10-14 08:27:41 +00:00
iorgamgabriel
069a996e02 Bug 1923826 - [Menu Redesign] Separate the content descriptions for each menu bottom sheet. r=android-reviewers,boek
Differential Revision: https://phabricator.services.mozilla.com/D225187
2024-10-14 08:03:06 +00:00
Mirko Brodesser
fbb6e5903e Bug 1921008: part 6) Add javascript-url navigation via form-submission tests for Trusted Types. r=tschuster
Differential Revision: https://phabricator.services.mozilla.com/D223596
2024-10-14 07:58:48 +00:00
Mirko Brodesser
f72d5dd15a Bug 1921008: part 5) Clarify click-loop-breaking mechanism in <trusted-types-navigation.html>. r=tschuster
Before, <navigation-support.html> didn't correctly break the loop, as it
didn't use hashes.

Differential Revision: https://phabricator.services.mozilla.com/D223754
2024-10-14 07:58:48 +00:00
Mirko Brodesser
b463000704 Bug 1921008: part 4) Remove checking trusted-types as effective directive in <trusted-types-navigation.html>. r=tschuster
It was confusing.

Differential Revision: https://phabricator.services.mozilla.com/D223753
2024-10-14 07:58:47 +00:00
Mirko Brodesser
58e7ffb45b Bug 1921008: part 3) Remove event listener when desired event was received in <trusted-types-navigation.html>. r=tschuster
Previously, the event listeners of a previous test run would stay,
contributing to the set of assertions of the current test run.

Differential Revision: https://phabricator.services.mozilla.com/D223595
2024-10-14 07:58:47 +00:00
Mirko Brodesser
2fb0476410 Bug 1921008: part 2) Add assertions to <trusted-types-navigation.html>. r=tschuster
Before, the sub-tests passed without any assertions, hampering checking
whether the tests were indeed correct.

Differential Revision: https://phabricator.services.mozilla.com/D223594
2024-10-14 07:58:46 +00:00
Mirko Brodesser
e19a3bfef8 Bug 1921008: part 1) Factor duplicated code from <navigation-support.html> and <navigation-report-only-support.html> out. r=tschuster
The factored out code will be extended in the next part.

Differential Revision: https://phabricator.services.mozilla.com/D223593
2024-10-14 07:58:46 +00:00
Mugurell
94dd56c7ef Bug 1923638 - Ignore toolbars when configuring the engine view if in a PWA r=android-reviewers,skhan
Differential Revision: https://phabricator.services.mozilla.com/D225202
2024-10-14 07:53:06 +00:00
Mugurell
300a59a411 Bug 1923205 - part 2 - Avoid dynamic behavior for the engine view when in a PWA after screen rotation r=android-reviewers,skhan
Differential Revision: https://phabricator.services.mozilla.com/D225201
2024-10-14 07:53:06 +00:00
Mugurell
a99b12ee72 Bug 1923205 - part 1 - Allow PWAs to remove dynamic behavior for the engine view r=android-reviewers,skhan
The important change (and only change needed to fix the bug) is to remove `EngineViewClippingBehavior`
if already set which would update how the engine view is displayed to account for the dynamic toolbars
if in PWAs we don't show toolbars.

Since we want the same functionality for configuring toolbars and the engine view when toolbars
should be hidden in a PWA or when entering fullscreen and also when toolbars should be shown again
we'll use now same methods in both places which encapsulate this potentially brittle code.

Differential Revision: https://phabricator.services.mozilla.com/D225063
2024-10-14 07:53:05 +00:00
Tarek Ziadé
5f85c36e4f Bug 1924078 - Remove custom warning and custom tabs r=vazish,fluent-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D225347
2024-10-14 07:48:01 +00:00