Commit Graph

160 Commits

Author SHA1 Message Date
Emily McMinn
efeb1ce68d Bug 1938446 - Land illustrations & animations for Tab Groups callouts r=omc-reviewers,pdahiya
Differential Revision: https://phabricator.services.mozilla.com/D233445
2025-01-13 18:48:32 +00:00
Punam Dahiya
8050d3e240 Bug 1939918 - Remove newtab utils dependancy in aboutwelcome and asrouter karma unit tests r=omc-reviewers,mconley,emcminn
Differential Revision: https://phabricator.services.mozilla.com/D233428
2025-01-13 17:52:42 +00:00
Cosmin Sabou
efae93e17d Backed out 2 changesets (bug 1938446, bug 1919598) for causing failures on browser_tabnotificationbox_switch_tabs and conflicts. CLOSED TREE
Backed out changeset be28df34cc76 (bug 1938446)
Backed out changeset ffa630d286bb (bug 1919598)
2025-01-11 01:58:14 +02:00
Emily McMinn
b2b5a8709e Bug 1938446 - Land illustrations & animations for Tab Groups callouts r=omc-reviewers,pdahiya
Differential Revision: https://phabricator.services.mozilla.com/D233445
2025-01-10 21:50:48 +00:00
negin
34caedb5e0 Bug 1919598 - Land Firefox View Discoverability treatment-b in mc- r=omc-reviewers,fluent-reviewers,emcminn,accessibility-frontend-reviewers,nstroud
Differential Revision: https://phabricator.services.mozilla.com/D229198
2025-01-10 21:31:18 +00:00
Cosmin Sabou
05bc4c0400 No bug - correct some typos in manifest files. a=typo-fix
CLOSED TREE
2025-01-10 23:43:19 +02:00
Punam Dahiya
f52a594a1a Bug 1866170 - Give about:welcome and asrouter their own try-runner.js test job runner scripts r=home-newtab-reviewers,omc-reviewers,mviar,mconley,ahal
Differential Revision: https://phabricator.services.mozilla.com/D233111
2025-01-09 22:20:59 +00:00
Punam Dahiya
750029ed7c Bug 1899027 - Move asrouter tests from newtab to asrouter r=home-newtab-reviewers,omc-reviewers,mconley,mviar
Differential Revision: https://phabricator.services.mozilla.com/D233024
2025-01-08 17:37:31 +00:00
Meg Viar
81a9ec1500 Bug 1937625 - Strengthen embedded browser element flags in Messaging System r=omc-reviewers,nalexander,nika,negin
Enhancements for the embedded browser component in AboutWelcome:

- Improve the speed at which the browser starts by predicting the `remoteType` the load is going to finish in
- Set `maychangeremoteness` attribute to true to allow changing remoteness if user navigates to a different URL that requires this (such as an `about:` page, though with the current set up `about:` pages won't load for security reasons).
- Set `nodefaultsrc` attribute to true to avoid loading a default URL if `src` is blank

Differential Revision: https://phabricator.services.mozilla.com/D232425
2025-01-08 00:14:40 +00:00
Mike Conley
4647ce1c30 Bug 1937234 - Add signed out avatar variants. r=skhamis,emilio,omc-reviewers,pdahiya
These variants are controlled via the avatarIconVariant variable of the
fxaButtonVisibility feature.

Differential Revision: https://phabricator.services.mozilla.com/D232128
2024-12-20 16:07:56 +00:00
Emilio Cobos Álvarez
becf088418 Bug 1929400 - Don't use :has() to hide BookmarksBarButton message. r=aminomancer,desktop-theme-reviewers,omc-reviewers,dao
This is faster, no point in making customize mode slower for something
that almost never shows up anyways.

Differential Revision: https://phabricator.services.mozilla.com/D228104
2024-12-19 14:32:52 +00:00
Sammy Khamis
6a8f7f80ad Bug 1936922: Cache response from listAttachedOAuthClients to prevent unnecessary network calls r=markh,omc-reviewers,aminomancer
Differential Revision: https://phabricator.services.mozilla.com/D232006
2024-12-17 23:09:53 +00:00
kpatenio
c8197f60d5 Bug 1920235 - add ability to access shadowDOM for callout selectors. r=omc-reviewers,aminomancer
The intent of this patch is to add support for feature callout anchors situated in the shadowDOM. For instance, for the Review Checker migration project, we want to add callout messages on the feature's sidebar icon. The actual parsing of the selector itself should be credited to @aminomancer; I'm authoring this patch to make sure we can get the change landed and tested.

I also verified the change with a few selectors via about:asrouter. For instance, if you wish to test with the actual Review Checker sidebar icon, you should have the prefs `sidebar.revamp` and `browser.shopping.experience2023.integratedSidebar` set to true. After enabling RC in the sidebar settings, add the following anchor to any callout: `"#sidebar-main > sidebar-main::%shadow% .tools-and-extensions::%shadow% moz-button[view='viewReviewCheckerSidebar']"`.

Lastly, I added a test in `browser/components/asrouter/tests/browser/browser_feature_callout_panel.js` to ensure `%shadow%` works as intended.

Differential Revision: https://phabricator.services.mozilla.com/D231337
2024-12-13 02:56:50 +00:00
Emily McMinn
b5d1cfafd6 Bug 1931659 - Add currentTabsOpen context to nthTabClosed trigger; as well as a similar nthTabOpened trigger r=omc-reviewers,pdahiya,dao
Differential Revision: https://phabricator.services.mozilla.com/D231144
2024-12-12 17:14:23 +00:00
Florian Quèze
a41844c552 Bug 1931901 - remove leftover test references to legacy scalar APIs, r=chutten,geckoview-reviewers,extension-reviewers,home-newtab-reviewers,robwu,ohall,mconley.
Differential Revision: https://phabricator.services.mozilla.com/D229929
2024-12-10 10:31:58 +00:00
Mike Conley
e06d4179af Bug 1932859 - Prevent auto-closing of the AppMenu / FxA panels when generically clicking the FxA menu message CTA. r=pdahiya,omc-reviewers
The change in CustomizableUI is required because the routine wasn't designed to handle
custom elements with shadow roots. The routine's job is to walk up the ancestry of
the click event and determine if any of the parents have certain characteristics
that would indicate that the event should not close the panel, and without this
change it would bail out as soon as it hit the shadow root if the event's original
target was inside the custom element.

In this case, the element enclosing the fxa-menu-message is the "appMenu-fxa-menu-message"
toolbaritem with `closemenu="none"`, which this modification now lets
CustomizableUI detect.

Differential Revision: https://phabricator.services.mozilla.com/D230560
2024-12-09 21:16:19 +00:00
Tooru Fujisawa
0f29352379 Bug 1934048 - Update obsoleve comments about JSM in browser/components/asrouter/tests/unit/ASRouter.test.js. r=mconley
Differential Revision: https://phabricator.services.mozilla.com/D230566
2024-12-05 00:04:59 +00:00
Punam Dahiya
3ce35fab35 Bug 1923726 - Update NimbusRolloutMessageProvider with recent messaging rollouts r=negin,omc-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D230813
2024-12-02 23:02:59 +00:00
Logan Rosen
d24c9d9ff4 Bug 1857834 - auto-formatting with Prettier v3 r=linter-reviewers,webdriver-reviewers,perftest-reviewers,search-reviewers,devtools-reviewers,sync-reviewers,reusable-components-reviewers,profiler-reviewers,dom-storage-reviewers,android-reviewers,firefox-ai-ml-reviewers,hjones,mcheang,mstange,sparky,janv,nchevobbe,tarek,Standard8,markh
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D230598
2024-11-29 15:18:25 +00:00
Meg Viar
48ab3860bf Bug 1923868 - Support setting Firefox as default based on installer attribution campaign r=firefox-desktop-core-reviewers ,nalexander,omc-reviewers,pdahiya
This patch adds an startup idle task that sets the browser as default if an attribution campaign id of "set_default_browser" is present on first run. This works supports an upcoming experiment where users will have the option to "download as default" via [[ https://www.mozilla.org/en-US/firefox/new/ | the stub installer marketing page ]].

Differential Revision: https://phabricator.services.mozilla.com/D225212
2024-11-25 18:14:28 +00:00
Sebastian Hengst
4d78eba78c Bug 1922355 - adjust mochitest expectations for macOS 14.70 x86-64. r=jmaher,tabbrowser-reviewers,dao
Differential Revision: https://phabricator.services.mozilla.com/D229944
2024-11-25 16:28:39 +00:00
Jason Prickett
86f8c07bf1 Bug 1928467 - Add ability for bookmarks toolbar button to be removed after unenrollment r=mviar,omc-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D229380
2024-11-22 20:32:07 +00:00
Alexandru Marc
f8117504e2 Backed out changeset a2435b289727 (bug 1923868) for causing bc failures @ browser_aboutwelcome_multistage_languageSwitcher.js. CLOSED TREE 2024-11-22 19:14:57 +02:00
Meg Viar
cf5467d380 Bug 1923868 - Support setting Firefox as default based on installer attribution campaign r=firefox-desktop-core-reviewers ,nalexander,omc-reviewers,pdahiya
This patch adds an startup idle task that sets the browser as default if an attribution campaign id of "set_default_browser" is present on first run. This works supports an upcoming experiment where users will have the option to "download as default" via [[ https://www.mozilla.org/en-US/firefox/new/ | the stub installer marketing page ]].

Differential Revision: https://phabricator.services.mozilla.com/D225212
2024-11-22 14:29:15 +00:00
Adi
9d9552bf95 Backed out changeset 893b697b0ad7 (bug 1923868) for causing bc failures @nsMacDockSupport.mm. CLOSED TREE 2024-11-22 05:40:59 +02:00
Adi
1553bea548 Backed out changeset 12efd90d2c57 (bug 1928467) for causing bc failures @browser_bookmarks_bar_button.js. 2024-11-22 05:38:11 +02:00
Jason Prickett
89c974a0bc Bug 1928467 - Add ability for bookmarks toolbar button to be removed after unenrollment r=mviar,omc-reviewers
This patch adds the functionality of the fxms_bmb_button surface to be removed when a user has been unenrolled from an experiment that used said feature.

Differential Revision: https://phabricator.services.mozilla.com/D229380
2024-11-22 02:40:22 +00:00
Meg Viar
12908e2cd7 Bug 1923868 - Support setting Firefox as default based on installer attribution campaign r=firefox-desktop-core-reviewers ,nalexander,omc-reviewers,pdahiya
This patch adds an startup idle task that sets the browser as default if an attribution campaign id of "set_default_browser" is present on first run. This works supports an upcoming experiment where users will have the option to "download as default" via [[ https://www.mozilla.org/en-US/firefox/new/ | the stub installer marketing page ]].

Differential Revision: https://phabricator.services.mozilla.com/D225212
2024-11-21 20:54:40 +00:00
hanna alemu
f2b065e5bb Bug 1825381 - Create experimental set to default prompt-style spotlight and enable showing via Nimbus r=mviar,omc-reviewers,firefox-desktop-core-reviewers ,mossop,aminomancer
Differential Revision: https://phabricator.services.mozilla.com/D225699
2024-11-20 18:04:46 +00:00
Jason Prickett
e0cf13dab7 Bug 1928465 - Add asrouter trigger listener for button click r=mviar,omc-reviewers
This patch adds an `elementClick` trigger listener that can be used like so:

```
trigger: {
        id: "elementClicked",
        params: ["element-id"],
}
```

Differential Revision: https://phabricator.services.mozilla.com/D228399
2024-11-20 02:58:51 +00:00
Beth Rennie
83f3481e28 Bug 1929116 - Update NimbusEnrollment.schema.json and store new Firefox Labs fields in saved enrolments r=chumphreys,omc-reviewers,aminomancer
Differential Revision: https://phabricator.services.mozilla.com/D228723
2024-11-19 22:18:43 +00:00
Sandor Molnar
847fd3bab7 Bug 1775852 - Disable browser_asrouter_group_userprefs.js on linux1804-64-qr asan & debug & opt configuration. r=intermittent-reviewers,jmaher DONTBUILD
Differential Revision: https://phabricator.services.mozilla.com/D228117
2024-11-06 01:24:02 +00:00
Dave Townsend
4ea798a326 Bug 1894239: Enable MOZ_SELECTABLE_PROFILES by default. r=jhirsch,omc-reviewers,niklas,emcminn,mconley
This also removes some tests of `MOZ_SELECTABLE_PROFILES` from `browser/components/profiles` files
where the test is pointless because that test is excluded from the build if `MOZ_SELECTABLE_PROFILES`
is unset.

Differential Revision: https://phabricator.services.mozilla.com/D227372
2024-11-05 12:59:04 +00:00
Stanca Serban
7bfaea64ba Backed out changeset 101063793364 (bug 1894239) for causing mochitests failures in browser_sentence_case_strings.js. CLOSED TREE 2024-11-04 20:39:11 +02:00
Dave Townsend
09c0a011d8 Bug 1894239: Enable MOZ_SELECTABLE_PROFILES by default. r=jhirsch,omc-reviewers,niklas,emcminn,mconley
This also removes some tests of `MOZ_SELECTABLE_PROFILES` from `browser/components/profiles` files
where the test is pointless because that test is excluded from the build if `MOZ_SELECTABLE_PROFILES`
is unset.

Differential Revision: https://phabricator.services.mozilla.com/D227372
2024-11-04 16:20:41 +00:00
Goloman Adrian
43172c6f10 Backed out changeset 4aeabb1fca7f (bug 1894239) for causing bc failures @browser_parsable_css.js. CLOSED TREE 2024-11-04 17:11:24 +02:00
Dave Townsend
658587a8e9 Bug 1894239: Enable MOZ_SELECTABLE_PROFILES by default. r=jhirsch,omc-reviewers,niklas,emcminn,mconley
This also removes some tests of `MOZ_SELECTABLE_PROFILES` from `browser/components/profiles` files
where the test is pointless because that test is excluded from the build if `MOZ_SELECTABLE_PROFILES`
is unset.

Differential Revision: https://phabricator.services.mozilla.com/D227372
2024-11-04 13:19:59 +00:00
Goloman Adrian
a536e48199 Backed out changeset 07b4dffd9109 (bug 1894239) for causing bc failures @browser_all_files_referenced.js. CLOSED TREE 2024-11-04 14:50:12 +02:00
Dave Townsend
d23817862a Bug 1894239: Enable MOZ_SELECTABLE_PROFILES by default. r=jhirsch,omc-reviewers,niklas,emcminn,mconley
This also removes some tests of `MOZ_SELECTABLE_PROFILES` from `browser/components/profiles` files
where the test is pointless because that test is excluded from the build if `MOZ_SELECTABLE_PROFILES`
is unset.

Differential Revision: https://phabricator.services.mozilla.com/D227372
2024-11-04 11:15:32 +00:00
Goloman Adrian
8d0a2fbb16 Backed out changeset 6539a7d0d9ca (bug 1894239) for causing xpcshell failures at test_selectable_profile_launch.js. CLOSED TREE 2024-11-01 20:43:36 +02:00
Dave Townsend
0085ce80d2 Bug 1894239: Enable MOZ_SELECTABLE_PROFILES by default. r=jhirsch,omc-reviewers,niklas,emcminn,mconley
This also removes some tests of `MOZ_SELECTABLE_PROFILES` from `browser/components/profiles` files
where the test is pointless because that test is excluded from the build if `MOZ_SELECTABLE_PROFILES`
is unset.

Differential Revision: https://phabricator.services.mozilla.com/D227372
2024-11-01 17:08:27 +00:00
Cristian Tuns
8f3ca1449e Backed out changeset 7df0f0dd73a1 (bug 1894239) for causing bc failures in /browser_appmenu.js CLOSED TREE 2024-11-01 12:48:39 -04:00
Dave Townsend
52f716131a Bug 1894239: Enable MOZ_SELECTABLE_PROFILES by default. r=jhirsch,omc-reviewers,niklas,emcminn,mconley
This also removes some tests of `MOZ_SELECTABLE_PROFILES` from `browser/components/profiles` files
where the test is pointless because that test is excluded from the build if `MOZ_SELECTABLE_PROFILES`
is unset.

Differential Revision: https://phabricator.services.mozilla.com/D227372
2024-11-01 14:38:45 +00:00
hanna alemu
6d1493ff8d Bug 1899747 - Add option to extend feature callout page_event_listener to all windows r=omc-reviewers,mviar,aminomancer
Differential Revision: https://phabricator.services.mozilla.com/D213239
2024-10-30 20:56:29 +00:00
Mike Conley
398b96c057 Bug 1926585 - Append the right menu identifier to the utm_content param for FxAMenuMessage. r=pdahiya,omc-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D226650
2024-10-24 16:06:45 +00:00
Emily McMinn
f3bbb3366b Bug 1918536 - CREATE_NEW_SELECTABLE_PROFILE Special Message Action && test message r=jhirsch,omc-reviewers,pdahiya
Differential Revision: https://phabricator.services.mozilla.com/D222980
2024-10-17 18:18:59 +00:00
hanna alemu
c1335b0cb7 Bug 1892548 - Remove faulty test r=omc-reviewers,mviar
Removes the react_to_trigger test that causes intermittent failures on different platforms. The method that renders the infobar message (showInfoBarMessage) and the defaultBrowserCheck trigger are covered by other tests separately.

Differential Revision: https://phabricator.services.mozilla.com/D225532
2024-10-15 19:52:11 +00:00
Mike Conley
8f93af2686 Bug 1920799 - Add tests for the new menu messaging surface for the FxA CTA message. r=pdahiya,omc-reviewers,home-newtab-reviewers,nbarrett
To manually test the message, make sure to set
`browser.newtabpage.activity-stream.asrouter.devtoolsEnabled` to `true`,
and then visit about:asrouter. Scroll down to the message with ID
FXA_ACCOUNTS_APPMENU_PROTECT_BROWSING_DATA, and click "Show".

This will default to opening the AppMenu with the message. You can open
it for the PXI menu by changing `testingTriggerContext` in the test
message definition from "app_menu" to "pxi_menu".

Differential Revision: https://phabricator.services.mozilla.com/D224790
2024-10-15 16:15:09 +00:00
Mike Conley
dd4373aecb Bug 1920799 - Add a new messaging surface to the AppMenu and PXI menus for describing the value of signing into an FxA. r=pdahiya,Gijs,desktop-theme-reviewers,omc-reviewers,home-newtab-reviewers,fluent-reviewers,hjones,skhamis,nbarrett
Developing tests in a later patch in this series.

Differential Revision: https://phabricator.services.mozilla.com/D223409
2024-10-15 16:15:09 +00:00
Chris H-C
724b00c304 Bug 1920562 - Remove JS uses of Services.telemetry.setEventRecordingEnabled r=florian,extension-reviewers,settings-reviewers,pip-reviewers,credential-management-reviewers,search-reviewers,devtools-reviewers,sync-reviewers,sessionstore-reviewers,omc-reviewers,migration-reviewers,firefox-desktop-core-reviewers ,urlbar-reviewers,sfoster,nchevobbe,valentin,Gijs,dimi,lina,mconley,pdahiya,willdurand
Differential Revision: https://phabricator.services.mozilla.com/D223833
2024-10-08 20:14:42 +00:00