зеркало из https://github.com/mozilla/gecko-dev.git
00de7dca50
This fixes two bugs. The first is that when the firefox profile migrator doesn't know which profile to migrate it attempts to fall back to another profile. I think this was intended to be the default but in bug 1322797 I ended up making it the current profile, which is the profile we're restoring into now. I think at this stage the profile directory doesn't even exist so things go wrong. Changing to use the actual default works but.... When the profile migrator UI doesn't know what profile to migrate from it uses the default profile. So if the profile you're actually trying to restore is not the default we'll effectively throw its data into the archive and replace it with data from the default profile. I'm inclined to say that if the migrator does not know what profile to migrate from it should error at that point for safety. Why would the profile migrator not know what profile to migrate from? Because of a long-standing text encoding problem. In C++ profile names are encoded in UTF8. But we try to pass them to JS through an IDL parameter of type ACString. This does no UTF8 decoding and so JS recieves an incorrect name if the name includes non-ascii characters and so can't find the profile. This patch fixes the IDL parameter to AUTF8String which does the decoding correctly and so JS gets the name correctly. We should probably think about whether just passing the nsIToolkitProfile object to the migrator is a better choice here. Differential Revision: https://phabricator.services.mozilla.com/D26250 --HG-- extra : moz-landing-system : lando |
||
|---|---|---|
| .. | ||
| content | ||
| tests | ||
| .eslintrc.js | ||
| 360seProfileMigrator.jsm | ||
| ChromeMigrationUtils.jsm | ||
| ChromeProfileMigrator.jsm | ||
| ESEDBReader.jsm | ||
| EdgeProfileMigrator.jsm | ||
| FirefoxProfileMigrator.jsm | ||
| IEProfileMigrator.jsm | ||
| MSMigrationUtils.jsm | ||
| MigrationUtils.jsm | ||
| ProfileMigrator.jsm | ||
| SafariProfileMigrator.jsm | ||
| components.conf | ||
| jar.mn | ||
| moz.build | ||
| nsIBrowserProfileMigrator.idl | ||
| nsIEHistoryEnumerator.cpp | ||
| nsIEHistoryEnumerator.h | ||
| nsWindowsMigrationUtils.h | ||