Skip to content
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

Selecting "Restore settings to their original defaults" broken #2708

Closed
michaellmonaghan opened this issue Dec 23, 2018 · 5 comments · Fixed by brave/brave-core#2130
Closed
Assignees
Labels
bug feature/global-settings Settings at browser level independent of shields settings priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/include

Comments

@michaellmonaghan
Copy link

michaellmonaghan commented Dec 23, 2018

Test plan

See https://github.com/brave/brave-core/pull/2130/files

Description

"Restore settings to their original defaults" does nothing. Appears to load indefinitely.

Steps to Reproduce

  1. Open settings
  2. Click "Restore settings to their original defaults"

Actual result:

Settings not cleared and the following shown indefinitely.
image

Expected result:

Settings would be cleared.

Reproduces how often:

Reproducible on both machines I use brave on.

Brave version (brave://version info)

0.58.16 Chromium: 71.0.3578.98 (Official Build) (64-bit)]

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    not tested

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
    no

  • Is the issue reproducible on the latest version of Chrome?
    no

Additional Information

@srirambv srirambv added bug feature/global-settings Settings at browser level independent of shields settings priority/P4 Planned work. We expect to get to it "soon". QA/Yes release-notes/include labels Dec 25, 2018
@srirambv srirambv added this to the 1.x Backlog milestone Dec 25, 2018
@srirambv
Copy link
Contributor

Issue reproduced on 0.58.18 as well.

@rebron rebron added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P4 Planned work. We expect to get to it "soon". labels Jan 8, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@rebron
Copy link
Collaborator

rebron commented Mar 14, 2019

cc: @mkarolin can you take a peek when you get a chance?

@LaurenWags
Copy link
Member

Issue reproduces with

Brave 0.62.33 Chromium: 73.0.3683.75 (Official Build) beta(64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Mac OS X

@jasonrsadler
Copy link

jasonrsadler commented Mar 31, 2019

When reset pushed, get in console:

[88589:775:0331/105815.341086:ERROR:CONSOLE(123)] 
"Uncaught TypeError: Cannot set property 'checked' of undefined", source: 
chrome://settings/reset_page/reset_profile_dialog.js (123)

@mkarolin mkarolin self-assigned this Apr 1, 2019
mkarolin added a commit to brave/brave-core that referenced this issue Apr 2, 2019
Fixes brave/brave-browser#2708

Instead of patching the html/js of the reset_profile_dialog, introduced
Brave behavior that hides/unchecks/disables the checkbox we don't want
seen along with the entire footer of the dialog since it contains
nothing else.
mkarolin added a commit to brave/brave-core that referenced this issue Apr 11, 2019
Fixes brave/brave-browser#2708

Instead of patching the html/js of the reset_profile_dialog, introduced
Brave behavior that hides/unchecks/disables the checkbox we don't want
seen along with the entire footer of the dialog since it contains
nothing else.
mkarolin added a commit to brave/brave-core that referenced this issue Apr 16, 2019
Fixes brave/brave-browser#2708

Instead of patching the html/js of the reset_profile_dialog, introduced
Brave behavior that hides/unchecks/disables the checkbox we don't want
seen along with the entire footer of the dialog since it contains
nothing else.
@bsclifton bsclifton added this to the 0.66.x - Nightly milestone Apr 23, 2019
@srirambv
Copy link
Contributor

srirambv commented May 13, 2019

Verification passed on

Brave 0.64.75 Chromium: 74.0.3729.131 (Official Build) (64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Linux

Verification PASSED on macOS 10.14.4 x64 using the following build:

Brave 0.64.75 Chromium: 74.0.3729.131 (Official Build) (64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Mac OS X
  • ensured that the Learn more under the modal opens https://support.brave.com/hc/en-us/articles/360017903152-How-do-I-reset-Brave-settings-to-default-
  • ensured that the modal is dismissed when clicking on Cancel
  • ensured that the modal is dismissed after clicking on Reset settings
  • ensured that bookmarks, history and passwords are not being cleared
  • ensured that sessions/cookies are being cleared (made sure I was logged out)
  • installed several extensions and ensured they were disabled when clicking on Reset settings (made sure they were appeared disabled under brave://extensions/)
  • ensured that search engines, pinned tabs, and on startup are being cleared

Verification passed on

Brave 0.64.75 Chromium: 74.0.3729.131 (Official Build) (64-bit)
Revision 518a41c1fa7ce1c8bb5e22346e82e42b4d76a96f-refs/branch-heads/3729@{#954}
OS Windows 10 OS Build 17134.523
  • Verified reset settings modal is closed after clicking on reset button
  • Verified pinned tabs, search engine and new tab settings are reset
  • Verified that bookmarks, history and passwords are not being cleared

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/global-settings Settings at browser level independent of shields settings priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants