-
Notifications
You must be signed in to change notification settings - Fork 879
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
Set Brave as a default browser w/o system UI on Windows #11125
Conversation
2f6015f
to
4d73efa
Compare
4d73efa
to
431efb0
Compare
431efb0
to
ed615d6
Compare
Verification PASSED on
Verification PASSED on
|
protected: | ||
~DefaultBrowserWorker() override; | ||
|
||
+ BRAVE_DEFAULT_BROWSER_WORKER_H |
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.
these should always go in public
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.
ha! I actually asked Simon to move it before private:
(which he did), but missed the part that this class have protected:
(kinda rare in Chromium). Should be more precise with wording 🙂 #11125 (comment)
Resolves brave/brave-browser#19466
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
,npm run lint
,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Test 1
Make default
button and this is set as a default browser immediately w/o launching system UITest 2
Set as a default
and check browser is set as a default browser w/o system UINOTE: This feature is only available from Win10 RS2. Older browser will work like chromium.