Closed Bug 1814943 Opened 1 year ago Closed 1 year ago

Safari - Don't list resources for import when those resources won't actually bring things over

Categories

(Firefox :: Migration, enhancement)

enhancement

Tracking

()

RESOLVED WONTFIX

People

(Reporter: kpatenio, Unassigned)

References

(Depends on 1 open bug)

Details

We want to return resource objects only if a resource has actual data inside. The Safari migrator has its own implementation of getResources that tries to get various bits of information from a Safari profile. It seems that the migrator tries to find:

  • history
  • bookmarks
  • reading list

All these resources are however plists, which are not fully supported anymore in modern versions of Safari. Therefore there may be some overlap with Bug 1236916 (also see this comment in that bug).

Severity: -- → N/A
Depends on: 1802219
No longer depends on: 1814941

It turns out we don't want to / can't do this given the upcoming design for the Migration Wizard. The spec calls for the Migration Wizard to request permission for reading the Safari folder after the user has selected which resources to import. This makes it impossible for us to determine if valid resources exist in the Safari folder (beyond what we currently do by checking for the existence of files). I've spoken to UX and PM about this, and the decision is to make Safari special - we won't check for empty resources for Safari, and then run the permission step after resource selection.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.