Ensure that Variant 2's "Select All" checkbox only checks items that are visible.
Categories
(Firefox :: Migration, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox113 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Right now it's checking both hidden and visible checkboxes: https://searchfox.org/mozilla-central/rev/82828dba9e290914eddd294a0871533875b3a0b5/browser/components/migration/content/migration-wizard.mjs#695-697
It should only check the visible ones. That way, we don't accidentally confuse the parent process with a request to migrate things for which there are no resources.
At the same time, we should also make the MigrationWizardParent more resilient to this, where it will choose to ignore any attempts to migrate resources that it doesn't support.
Assignee | ||
Comment 1•2 years ago
|
||
STR:
- Make sure
browser.migrate.content-modal.enabled
istrue
andbrowser.migrate.content-modal.import-all.enabled
is true - Open the Migration Wizard (about:preferences, choose "Import Data" from the General section)
- Choose any browser from the list (I chose Safari on macOS)
- Expand the details section for the various resource types, and click "Select All" to de-select everything
- Click "Select All" again to reselect all of the checkboxes
- Click "Import"
- Depending on whether or not you've already given permission to the migrator, you might have to get through the permission flow, in which case, do that.
ER:
Once the migration begins, we should see just the checked items were imported from. The "Done" button is enabled.
AR:
"Form autofill data" gets listed even though it wasn't selected as a resource type, and shows a progress spinner. The "Cancel" button is disabled. See screenshot.
Assignee | ||
Comment 2•2 years ago
|
||
Assignee | ||
Comment 3•2 years ago
|
||
This also adds some belt-and-suspenders checks here so that even if the Migration Wizard
somehow gets confused and sends resource types that the migrator can't import from, they'll
just get ignored rather than having a forever-progress-spinner for that resource type.
Updated•2 years ago
|
Comment 5•2 years ago
|
||
bugherder |
Description
•