Commit Graph

57 Commits

Author SHA1 Message Date
Csoregi Natalia
8241a2637c Backed out changeset d609e38d7305 (bug 1809661) for failures on browser_tab_pickup_device_added_telemetry.js. CLOSED TREE 2023-02-27 23:21:07 +02:00
Sam Foster
41c9616e89 Bug 1809661 - Keep track of time spent with 0 tabs after a new device is added, and record a telemetry event with the time elapsed until new tabs are shown. r=kcochrane
* Keep track of the tab-pickup-container views in TabsSetupFlowManager and their visibility
* Fix visibilitychange handling in tab-pickup-container and add some tests
* Capture a timestamp when a device has been added and there are 0 tabs to show, with at least one visible tab-pickup-container view
* Record telemetry when there are > 0 tabs in these conditions

Differential Revision: https://phabricator.services.mozilla.com/D170526
2023-02-27 20:06:25 +00:00
Mark Banner
81e5d26ac3 Bug 1817183 - Update consumers of services/fxaccounts to import ES modules directly. r=markh,settings-reviewers,credential-management-reviewers,sgalich
Differential Revision: https://phabricator.services.mozilla.com/D170062
2023-02-17 16:34:43 +00:00
Cristian Tuns
b9ee007450 Backed out 6 changesets (bug 1816934, bug 1817182, bug 1817179, bug 1817183) for causing dt failures in browser_jsterm_autocomplete_null.js CLOSED TREE
Backed out changeset 17d4c013ed92 (bug 1817183)
Backed out changeset cfed8d9c23f3 (bug 1817183)
Backed out changeset 62fe2f589efe (bug 1817182)
Backed out changeset 557bd773fb85 (bug 1817179)
Backed out changeset 7f8a7865868b (bug 1816934)
Backed out changeset d6c1d4c0d2a0 (bug 1816934)
2023-02-17 10:51:33 -05:00
Mark Banner
c8baa7a866 Bug 1817183 - Update consumers of services/fxaccounts to import ES modules directly. r=markh,settings-reviewers,credential-management-reviewers,sgalich
Differential Revision: https://phabricator.services.mozilla.com/D170062
2023-02-17 09:01:45 +00:00
Mark Banner
18898557b3 Bug 1814931 - Convert services/sync imports to direct ES imports. r=markh,settings-reviewers,credential-management-reviewers,mconley,dimi
Differential Revision: https://phabricator.services.mozilla.com/D168837
2023-02-13 11:44:29 +00:00
Kelly Cochrane
03dfada2e1 Bug 1802889 - Resolve Fx View mobile promo message being incorrectly shown when signing out of current FxA device r=Gijs
There are two things I've added here:
  - The observers for when FxA devices are connected/disconnected were not added/removed as part of this update: https://phabricator.services.mozilla.com/D153069.
  - When a mobile device is the only synced device beyond the current one (desktop) and you remove the current device (desktop), then sign back in from Fx View Tab Pickup banner, `fxAccounts.device.recentDeviceList` only returns the mobile device for some reason (possibly due to device cache). This causes our checks for a secondary device to fail (as we now only have access to the mobile device from `recentDeviceList`, and we're assuming the one device we DO have access to is our current device - which is not the case). This is why Tab Pickup was incorrectly displaying the "Connect a mobile device" message. I've added a check at the start of `refreshDevices()` to manually refresh the device list (ignoring device cache) if the `recentDeviceList` doesn't contain a device with `isCurrentDevice` set to `true`. This is really a workaround for the caching stuff going on behind the scenes, but this does seem to fix things from our end.

Differential Revision: https://phabricator.services.mozilla.com/D165960
2023-01-06 13:11:01 +00:00
Sarah Clements
83cc18d749 Bug 1794610 - Change error state for password change r=sfoster,kcochrane,fluent-reviewers,flod
* Add new strings and add a new signed-out error type
* Update tests

Differential Revision: https://phabricator.services.mozilla.com/D162275
2022-12-06 14:30:39 +00:00
Mark Banner
17403914cb Bug 1795880 - Add an ESLint rule to enforce using static imports where possible. r=arai,geckoview-reviewers,m_kato
Differential Revision: https://phabricator.services.mozilla.com/D160131
2022-10-26 09:37:46 +00:00
Sammy Khamis
0c4a2fad3b Bug 1791851 - replace desktop tabs with the rust tabs engine r=teshaq,LougeniaBailey,markh,sfoster
Differential Revision: https://phabricator.services.mozilla.com/D157981
2022-10-25 20:03:23 +00:00
Hanna Jones
bc09027cd8 Bug 1791474 - show more specific message for locked primary password error r=sfoster,fluent-reviewers
* Updated the error message to use the text [here](https://www.figma.com/file/SE4xHgOW84yLiv7vFugm9R/Firefox-View-Stepping-Stone)
* Updated the existing password test to verify the message contents

Differential Revision: https://phabricator.services.mozilla.com/D159787
2022-10-24 18:29:53 +00:00
Sam Foster
0d9a297863 Bug 1795752 - Use fx-view rather than firefoxview as the entry point param for FxA signin/signup. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D159528
2022-10-18 00:22:26 +00:00
Cristian Tuns
39cbafb6a7 Backed out changeset d734d1d10478 (bug 1795752) for causing mochitest failures on browser_feature_callout.js CLOSED TREE 2022-10-17 17:17:49 -04:00
Sam Foster
1969a620bf Bug 1795752 - Use fx-view rather than firefoxview as the entry point param for FxA signin/signup. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D159528
2022-10-17 20:35:31 +00:00
Tim Giles
9fa180f7e9 Bug 1794474 - Force tabs sync when a user reloads Firefox View. r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D159401
2022-10-17 18:39:27 +00:00
Mark Hammond
21bd24ad69 Bug 1792550 - ensure the primary password is unlocked before signing in to sync. r=Mardak,sfoster
Differential Revision: https://phabricator.services.mozilla.com/D158719
2022-10-12 22:31:37 +00:00
Sarah Clements
afeab23b18 Bug 1785853 - Close device connected tab after user returns to Firefox View r=sfoster
* Add _closeDeviceConnectedTab to FirefoxViewHandler that's called by the TabSelect event handler
* Add test coverage

Differential Revision: https://phabricator.services.mozilla.com/D158031
2022-10-12 16:39:39 +00:00
Norisz Fay
f7de136a6d Backed out changeset 9b1e2e6ceb70 (bug 1785853) for causing mochitest failures on browser_setup_state.js CLOSED TREE 2022-10-12 17:50:02 +03:00
Sarah Clements
bf8716d193 Bug 1785853 - Close device connected tab after user returns to Firefox View r=sfoster
* Add _closeDeviceConnectedTab to FirefoxViewHandler that's called by the TabSelect event handler
* Add test coverage

Differential Revision: https://phabricator.services.mozilla.com/D158031
2022-10-12 13:26:41 +00:00
Sarah Clements
233f73e5bd Bug 1793754 - Fix sync error when user disconnects a device r=sfoster
* Only show sync error if user is not signed in and give them a way out when try again button is clicked

Differential Revision: https://phabricator.services.mozilla.com/D158972
2022-10-12 10:12:36 +00:00
Hanna Jones
92d8095a44 Bug 1789982 - handle case where sync is connected but user is logged out r=sclements
This patch adds logic to show an error when a user becomes disconnected due to changing their password on another device. We will now show the `sync-disconnected` error, which should direct the user to their sync settings where they can enter their new password/sign in again.

This patch also includes a test for this new error case, as well as some small changes to an existing test that started failing due to a slight mismatch between the UIState representation used in the test and the UIState we expect when a user logs out.

Differential Revision: https://phabricator.services.mozilla.com/D158043
2022-10-11 18:34:24 +00:00
Sarah Clements
e0825872b3 Bug 1792191 - Fix the Turn on syncing to continue error state r=sfoster
* Rename fxaSignedIn to fxaSignedInAndSyncEnabled to better reflect what its doing
* Re-enable the test_sync_disconnected_error test case and add another

Differential Revision: https://phabricator.services.mozilla.com/D158201
2022-10-07 10:04:35 +00:00
Csoregi Natalia
d301388c8f Backed out changeset d91a621a39be (bug 1785853) for multiple failures e.g.browser_setup_state.js CLOSED TREE 2022-10-03 19:48:44 +03:00
Sarah Clements
f13f2edf01 Bug 1785853 - Close device connected tab after user returns to Firefox View r=sfoster
* Add _closeDeviceConnectedTab to FirefoxViewHandler that's called by the TabSelect event handler
* Add test coverage

Differential Revision: https://phabricator.services.mozilla.com/D158031
2022-10-03 16:07:54 +00:00
Mark Banner
a49b8ef78f Bug 1792365 - Convert toolkit/modules consumers to use ES module imports directly. r=webdriver-reviewers,perftest-reviewers,geckoview-reviewers,extension-reviewers,preferences-reviewers,desktop-theme-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,robwu,Gijs,sgalich,bytesized,AlexandruIonescu,dao,m_kato
Differential Revision: https://phabricator.services.mozilla.com/D158094
2022-09-29 06:52:34 +00:00
Mark Banner
79b9557a71 Bug 1792398 - Enable ESLint rule 'strict' on mjs files as the directive is not necessary for modules. r=arai,pip-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D158115
2022-09-26 21:47:50 +00:00
Marian-Vasile Laza
b7f6194b8d Backed out changeset 0679274d6ed5 (bug 1792398) for causing bc failures on browser_sendQuery.js. CLOSED TREE 2022-09-26 22:53:00 +03:00
Mark Banner
5dcae1ed44 Bug 1792398 - Enable ESLint rule 'strict' on mjs files as the directive is not necessary for modules. r=arai,pip-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D158115
2022-09-26 18:51:57 +00:00
Narcis Beleuzu
95ae6b7d0f Backed out changeset 849faf753208 (bug 1792398) for bc failures on browser_sendQuery.js . CLOSED TREE 2022-09-26 20:44:40 +03:00
Mark Banner
ff212fab8b Bug 1792398 - Enable ESLint rule 'strict' on mjs files as the directive is not necessary for modules. r=arai,pip-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D158115
2022-09-26 16:51:10 +00:00
Kelly Cochrane
6a3fd20663 Bug 1791707 - Check for tablets when showing Fx View mobile promo r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D157936
2022-09-22 18:45:53 +00:00
Kelly Cochrane
bc0dc089e8 Bug 1790714 - Set Tab pickup open state pref to true when signing in r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D157857
2022-09-22 13:39:10 +00:00
Sam Foster
51a14bac68 Bug 1787619 - Handle primary password and sync login errors. r=Gijs,markh
* Show the error state when the Primary Password is locked and the user is signed-in
* Trigger a tab sync when the  user clicks the "try again" button from the setup error step.
  This will also result in an unlock prompt if the primary password is locked.

Differential Revision: https://phabricator.services.mozilla.com/D156360
2022-09-20 17:58:08 +00:00
Noemi Erli
9a3ab27e92 Backed out changeset 9239a20afb93 (bug 1787619) for causing failures in browser_setup_errors.js CLOSED TREE 2022-09-20 20:07:29 +03:00
Sam Foster
bb6c2d9701 Bug 1787619 - Handle primary password and sync login errors. r=Gijs,markh
* Show the error state when the Primary Password is locked and the user is signed-in
* Trigger a tab sync when the  user clicks the "try again" button from the setup error step.
  This will also result in an unlock prompt if the primary password is locked.

Differential Revision: https://phabricator.services.mozilla.com/D156360
2022-09-20 15:49:45 +00:00
Sam Foster
a78fec7c03 Bug 1789885 - Show the loading message only when we have no recent synced tabs data and no tab sync has completed yet r=Gijs,markh
* Remove the synced-tabs-not-ready state and instead make a waitingForTabs boolean the container can check when updating
* Refactor around the maybeUpdateUI so we either pass in a force-update param, or conditionally send
  the notification if the state changes
* Observe 'services.sync.tabs.changed' instead of the pref which gets updated as a side-effect of a
  tab sync
* Check recent tabs via SyncedTabs.jsm in TabsSetupFlowManager and queue up SyncedTabs.syncTabs() and refreshDevices() calls when the user signs in.

Differential Revision: https://phabricator.services.mozilla.com/D156997
2022-09-15 21:58:43 +00:00
Jared Hirsch
0ca0f19c16 Bug 1779384 - Persist collapsed state of Firefox View lists r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D156754
2022-09-14 17:18:35 +00:00
Sandor Molnar
e5de5b30e6 Backed out changeset dd71b6e79551 (bug 1779384) for causing browser-chrome failures in browser/components/firefoxview/tests/browser/browser_recently_closed_tabs.js CLOSED TREE 2022-09-14 04:59:04 +03:00
Jared Hirsch
e2d8d3f927 Bug 1779384 - Persist collapsed state of Firefox View lists r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D156754
2022-09-13 23:18:34 +00:00
Sarah Clements
395367fdea WIP Bug 1784363 - Update the fxa entrypoint to use fx-view r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D156081
2022-09-12 09:30:11 +00:00
Mike Conley
ae11f3c9d0 Bug 1785282 - Open 'Choose what to sync' dialog when users click 'Turn sync on in settings' in Firefox View. r=sclements
Differential Revision: https://phabricator.services.mozilla.com/D156578
2022-09-09 14:26:50 +00:00
Sam Foster
c99a8022bb Bug 1788735 - Fix fallback/default value for the services.sync.lastTabFetch pref getters. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D156218
2022-09-01 18:42:09 +00:00
Gijs Kruitbosch
4ef04e9049 Bug 1784085 - correct sync finishing topic identifier used in Firefox View so errors clear correctly, r=markh
Differential Revision: https://phabricator.services.mozilla.com/D155922
2022-08-30 09:32:56 +00:00
Gijs Kruitbosch
63a87376ea Bug 1784244 - do not surface sync errors in Firefox View if we previously synced successfully, r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D155845
2022-08-30 08:52:54 +00:00
Sam Foster
1eecd465c0 Bug 1784130 - Hide mobile promo when signed out. r=niklas
* Only show the mobile promo when tab syncing is enabled
* Toggle the promo & confirmation visibility in all setup states to ensure it gets hidden when we re-enter the sign-in to FxA state
* Add test coverage for signing out
* Ensure we always restore the mocks

Depends on D155427

Differential Revision: https://phabricator.services.mozilla.com/D154850
2022-08-26 05:37:49 +00:00
Butkovits Atila
acc81a64dd Backed out changeset d4db9e3554d5 (bug 1784130) for causing failures at browser_setup_state.js. 2022-08-25 23:07:58 +03:00
Sam Foster
c0c5dc6cc6 Bug 1784130 - Hide mobile promo when signed out. r=niklas
* Only show the mobile promo when tab syncing is enabled
* Toggle the promo & confirmation visibility in all setup states to ensure it gets hidden when we re-enter the sign-in to FxA state
* Add test coverage for signing out

Differential Revision: https://phabricator.services.mozilla.com/D154850
2022-08-25 19:12:39 +00:00
Sam Foster
9695132afd Bug 1786615 - Only show an error when sync isnt connected/enabled if we're currently signed in. r=markh
Differential Revision: https://phabricator.services.mozilla.com/D155427
2022-08-24 22:48:12 +00:00
Niklas Baumgardner
69a062bf44 Bug 1784969 - Force sync on Firefox View sync error or network offline action. r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D154828
2022-08-23 13:49:07 +00:00
Sarah Clements
8f51e55402 Bug 1784055 - Add error state for when sync is disconnected r=sfoster,fluent-reviewers,flod
* Add new error state strings and logic
* Add new test case

Differential Revision: https://phabricator.services.mozilla.com/D154675
2022-08-17 16:23:57 +00:00