1163644 - (edge-migration) [meta] Data importers/migrators for Microsoft Edge browser

Status: NEW

(Firefox :: Migration, task)

Description

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Our import wizard on Windows currently doesn't show Microsoft Edge/Spartan as an option. It should really be in there, potentially at the very top.

Comment 1

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 39 and 40 from discussion on release-drivers about possible uplift of support for Windows 10.

Comment 2

[Marco Bonardo [:mak]]

So, rumors say Win10 could be released in july, users will take at least one month before building data they might be interested into migrating (basically they have to use Edge for at least one month and decide they don't like it), that pushes the date to August, more realistically September.
Having a migrator for Firefox 41 (released on 22nd of September) should cover vast majority of use-cases.

Comment 3

[Bryan Clark (DevTools PM) [:clarkbw]]

(In reply to Marco Bonardo [::mak] from comment #2)
> So, rumors say Win10 could be released in july, users will take at least one
> month before building data they might be interested into migrating
> (basically they have to use Edge for at least one month and decide they
> don't like it), that pushes the date to August, more realistically September.
> Having a migrator for Firefox 41 (released on 22nd of September) should
> cover vast majority of use-cases.

I'll circle back with Philipp and Dolske, but I agree that the data importer seems like a feature we can hold off on until we assume our users have some data and are migrating over.

Comment 4

[Matthew N. [:MattN]]

At first glance, the bookmarks from Edge/Spartan are shared with IE and stored in the same directory in the profile folder.

History, cookies, and settings are the other data types we currently import and I didn't look much at them yet.

Comment 5

[Masatoshi Kimura [:emk]]

Are you looking at the older build? Edge has different bookmarks from what IE has for me.

Comment 6

[Matthew N. [:MattN]]

I'm running the version with Windows 10 TP.

Comment 7

[Masatoshi Kimura [:emk]]

Please try with the latest "Insider" Preview (no longer TP). The latest build is 10130.

Comment 8

[Makoto Kato [:m_kato]]

Spartan's favorites is under Spartan APPX's data folder.  I don't know the package folder name is kept even if RTM.

%HOME%/AppData/Local/Packages/Microsoft.Windows.Spartan_cw5n1h2txyewy/AC/Spartan/User/Default/Favorites

Comment 9

[Jim Mathies [:jimm]]

Edge is a universal app so it acts a lot like metro apps do. I'd bet that this directory can be difference per machine: 'Microsoft.Windows.Spartan_cw5n1h2txyewy'

IMO we shouldn't waste any time on this bug. There are far more serious issues on win10 to deal with.

Comment 10

[Justin Dolske [:Dolske]]

Changing to P3 after discussion -- this will be important to do eventually, but it's not critical to have immediately for the Windows 10 release. Worst case until this is we import a slightly-stale version of the user's IE data (if they've been using Edge, and if it stays separate).

Comment 11

[Liz Henry (:lizzard) (relman/hg->git project)]

OK. Glad to know this isn't going to block release. Marking wontfix for 39.

Comment 12

[Masatoshi Kimura [:emk]]

A documented way to get a folder for AppContainer (or a Metro app or a universal app) is the GetAppContainerFolderPath function:
https://msdn.microsoft.com/en-us/library/windows/desktop/hh448543%28v=vs.85%29.aspx
The DeriveAppContainerSidFromAppContainerName function will return an app container sid from an AppContainerName (or a "Moniker") such as "Microsoft.Windows.Spartan_cw5n1h2txyewy":
https://msdn.microsoft.com/en-us/library/windows/desktop/hh448541%28v=vs.85%29.aspx
NetworkIsolationEnumAppContainers will enumerate all AppContainers on the system:
https://msdn.microsoft.com/en-us/library/windows/desktop/hh447479%28v=vs.85%29.aspx

Comment 13

[:Gijs (out for now; he/him)]

http://blogs.windows.com/bloggingwindows/2015/06/19/upcoming-changes-to-windows-10-insider-preview-builds/ indicates they're changing the app ID and people will lose all the stuff they don't actively back up and restore (and they're only providing instructions for restoring favorites).

Comment 14

[Masatoshi Kimura [:emk]]

It means that they can't change the app ID lightly (otherwise users will lose favorites and other settings again). So we can use the new app ID as a stable import source.

Comment 15

[Marco Bonardo [:mak]]

We need at least bookmarks migration for Firefox 41, provided the strings are approved to land in Aurora in bug 1179346.

Comment 16

[Edwin Wong [:edwong]]

fyi - the passwords team (rchtara/rittme) is taking password specific importing off our backlog.

Comment 18

[Justin Dolske [:Dolske]]

I've made this a metabug, and split out the different migrator types into subbugs since they are at least somewhat independent of each other. (Feel free to continue to use this one for discussing shared issues.)

Comment 19

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

I don't see a reason to track this meta bug going forward. I am hoping that devs/Sheila/RelMan team will track individual bugs targeting specific releases.

Comment 20

[Sylvestre Ledru [:Sylvestre]]

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information

Comment 21

[Matthew N. [:MattN]]

(In reply to Sylvestre Ledru [:sylvestre] from comment #20)
> Moving to p3 because no activity for at least 1 year(s).
> See
> https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.
> md#how-do-you-triage for more information

Those definitions don't reflect when a component isn't resourced. Moving back to P2.