-
Notifications
You must be signed in to change notification settings - Fork 936
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
Default shield advanced view to TRUE for existing users. (uplift to 0.69.x) #3186
Conversation
NOTE: not ready to be merged; there is a failing browser test on WINDOWS only... investigating... |
Ready for consideration 😄 Above issue fixed |
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.
@bsclifton mind taking a look at some of these failures?
13:30:54 [33367/40121] CXX obj/brave/browser/ui/ui/brave_web_ui_controller_factory.o
13:30:54 [33368/40121] CXX obj/brave/browser/browser_process/brave_profile_prefs.o
13:30:54 FAILED: obj/brave/browser/browser_process/brave_profile_prefs.o
13:31:03 [33405/40121] CXX obj/brave/browser/browser_process/brave_content_browser_client.o
13:31:03 ninja: build stopped: subcommand failed.
12:55:42 [21825/37908] CXX obj/chrome/browser/ui/ui/sad_tab.o
12:55:42 [21826/37908] CXX obj/brave/browser/browser_process/brave_profile_prefs.o
12:55:42 FAILED: obj/brave/browser/browser_process/brave_profile_prefs.o
12:55:52 [21863/37908] CXX obj/brave/browser/browser_process/brave_content_browser_client.o
12:55:52 ninja: build stopped: subcommand failed.
13:54:31 [27495/37908] CXX obj/brave/browser/browser_process/brave_profile_prefs.o
13:54:31 FAILED: obj/brave/browser/browser_process/brave_profile_prefs.o
13:54:32 [27508/37908] CXX obj/brave/browser/ui/ui/brave_new_tab_button.o
13:54:32 ninja: build stopped: subcommand failed.
491cc41
to
be2967c
Compare
Rebased - ready for approval / merge (after CI runs) |
@bsclifton looks like the following checks have failed under
|
be2967c
to
414cff6
Compare
Problem found + fixed - when uplifting, I missed the removal of the check in |
Default shield advanced view to TRUE for existing users.
414cff6
to
aa112c7
Compare
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.
Uplift into 0.69.x
approved after deliberating with @brave/uplift-approvers. Both CI/Jenkins
and CI/Travis
passed without any issues or failures. Please make sure that the correct labels are added and the associated issue is moved into the correct milestone.
Fixes brave/brave-browser#5582
Uplift of #3154
Approved, please ensure that before merging:
After you merge: