-
Notifications
You must be signed in to change notification settings - Fork 900
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Import open windows and tabs from legacy Brave #981
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me. can you squash three commits into one?
The patch issue is addressed in efe72e2#diff-6b0d3600d50944ab08b6145722eaa7c2
@darkdh Squashed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++, also did a local test and it works as expected
I have tested and confirmed that this PR builds and works as expected on macOS and Linux, and I'm waiting on a Windows build so I can test there too. Once I've verified this is working on all platforms, I'll merge and request uplift to 0.57.x. |
Confirmed this PR builds and manual test plan passes on Windows. |
🎉 |
Great job, @garrettr! 😄 👍 |
Import open windows and tabs from legacy Brave
Import open windows and tabs from legacy Brave
Import open windows and tabs from legacy Brave
Awesome, thanks for getting this done and merged @bsclifton @garrettr 👍 |
Partial resolution brave/brave-browser#998. Only supports importing windows and tabs from legacy Brave, which is sufficient to support the goal of auto-migrating users from Muon to brave-core in 0.57.x.
Submitter Checklist:
npm test brave_unit_tests && npm test brave_browser_tests
) ongit rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
1. Manual import
Clear the Brave Development user data directory, then:
npm start
chrome://settings/importData
Brave (old)
Open windows and tabs
is presentOpen windows and tabs
If you select all import options, instead of only
Open windows and tabs
, all of the tabs should appear identical or nearly identical to what was visible when you quit Muon (because all of the session data was also imported).2. Automatic/first-run import
Clear the Brave Development user data directory, then run the Brave binary with the
--upgrade-from-muon
flag, e.g../src/out/Debug/Brave\ Browser\ Development.app/Contents/MacOS/Brave\ Browser\ Development --enable-logging --v=0 --disable-brave-update --upgrade-from-muon
.Once loaded, this should be nearly indistinguishable from the previous Muon session. The first run experience window should remain in the foreground of all of the imported windows.
Reviewer Checklist: