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

searching via settings causes "Uncaught Error: Assertion failed: An associated control was expected for SETTINGS-SUBPAGE Manage sync, but was not found." and spinner doesn't stop animating #2757

Closed
GeetaSarvadnya opened this issue Dec 29, 2018 · 12 comments

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Dec 29, 2018

Description

Search Sync text doesn't find sync section in Chrome://settings page

Steps to Reproduce

  1. Go to Chrome://settings page
  2. Search a string 'Sync' or 'sync'
  3. Finds the sync section only for the first time ( Observe that the finder icon keeps rotating even after search found)
  4. Search a string 'Sync' or 'sync' again

Actual result:

Doesn't find the matching string, Finder icon keeps rotating but it never finds the sync section.

find sync string- issue
Console log error:

[5660:1840:1229/132354.759:ERROR:CONSOLE(8)] "Uncaught Error: Assertion failed: An associated control was expected for SETTINGS-SUBPAGE Manage sync, but was not found.", source: chrome://settings/crisper.js (8)
image

Expected result:

Should find the sync section everytime, finder icon should stop after it found search string

Reproduces how often:

Brave version (brave://version info)

Brave 0.59.13 Chromium: 72.0.3626.28 (Official Build) beta (64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Windows

Reproducible on current release: na

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

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? na
  • Is the issue reproducible on the latest version of Chrome? na

Additional Information

@brave/legacy_qa

@GeetaSarvadnya GeetaSarvadnya added this to the 1.x Backlog milestone Dec 29, 2018
@GeetaSarvadnya GeetaSarvadnya added this to Untriaged backlog in Sync via automation Dec 29, 2018
@LaurenWags
Copy link
Member

Reproduces on macOS

Brave 0.59.14 Chromium: 72.0.3626.28 (Official Build) beta(64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Mac OS X

@btlechowski
Copy link

Reproduces on Ubuntu 17.10

Brave 0.59.14 Chromium: 72.0.3626.28 (Official Build) beta(64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Linux

@kjozwiak
Copy link
Member

kjozwiak commented Jan 4, 2019

Seems like searching in brave://settings has generally regressed. When attempting to search using 0.59.18 Chromium: 72.0.3626.28 (Official Build) beta on macOS 10.14.2, the search spinner keeps animating even though it should find the phrase. This doesn't seem to affect 0.58.18 Chromium: 71.0.3578.98 (Official Build). Examples:

Example using 0.58.18 Chromium: 71.0.3578.98:

searchingrelease

0.59.18 Chromium: 72.0.3626.28:

searchingbeta

Also getting the same error as @GeetaSarvadnya when searching for strings like password and bookmarks:

[99039:775:0104/011913.349686:ERROR:CONSOLE(8)] "Uncaught Error: Assertion failed: An associated control was expected for SETTINGS-SUBPAGE Manage sync, but was not found.", source: chrome://settings/crisper.js (8)

@GeetaSarvadnya @btlechowski @LaurenWags @srirambv are you guys seeing the same thing?

@srirambv
Copy link
Contributor

srirambv commented Jan 4, 2019

@kjozwiak reproduced the same error on console and the spinner when search is done on Linux build as well
sync

@kjozwiak
Copy link
Member

kjozwiak commented Jan 4, 2019

@srirambv do you see the spinner happening when searching for other words like bookmark and password?

@srirambv
Copy link
Contributor

srirambv commented Jan 4, 2019

Yes for any search term the spinner shows up

@kjozwiak kjozwiak changed the title Search Sync text doesn't find sync section in Chrome://settings page searching via settings causes "Uncaught Error: Assertion failed: An associated control was expected for SETTINGS-SUBPAGE Manage sync, but was not found." and spinner doesn't stop animating Jan 4, 2019
@LaurenWags
Copy link
Member

Yes, I see the same @kjozwiak

@GeetaSarvadnya
Copy link
Author

@kjozwiak Tested again for any serach i am seeing the spinner. Getting assertion failed message in console.

@cezaraugusto
Copy link
Contributor

changing to feature/settings as this seems more like a settings bug and not specific to sync. please change back if I'm wrong.

@kjozwiak
Copy link
Member

kjozwiak commented Jan 4, 2019

changing to feature/settings as this seems more like a settings bug and not specific to sync. please change back if I'm wrong.

Agreed, might have been caused by adding Sync into the settings but affecting all of brave://settings. I moved it from the Sync project to the General project board. We should get this fixed for 0.59.x as it's a new regression. CCing @rebron

@rebron rebron added this to Untriaged Backlog in General Jan 8, 2019
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Jan 8, 2019
@rebron rebron moved this from Untriaged Backlog to P3, P4, & P5 Backlog in General Jan 8, 2019
@rebron rebron removed this from Untriaged backlog in Sync Jan 15, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@btlechowski
Copy link

Still reproducible:

Brave 0.62.16 Chromium: 73.0.3683.39 (Official Build) dev (64-bit)
Revision cc53b0e12fcaf42e4bab8d6c23bd4fb7aae99f6c-refs/branch-heads/3683@{#413}
OS Windows 7 Service Pack 1 Build 7601.24312

@rebron rebron moved this from P4 Backlog to P3 Backlog in General Mar 14, 2019
@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Apr 25, 2019

Unable to reproduce the issue. Verified on below versions:

0.63.48 - Not Reproducible
0.65.74 - Not Reproducible
0.66.24 - Not Reproducible
0.64.60 - Not Reproducible

General automation moved this from P3 Backlog to Completed Apr 25, 2019
@kjozwiak kjozwiak removed this from Completed in General May 7, 2019
@kjozwiak kjozwiak added closed/invalid and removed QA/Test-Plan-Specified QA/Yes bug feature/settings priority/P3 The next thing for us to work on. It'll ride the trains. regression labels May 7, 2019
@NejcZdovc NejcZdovc added this to the Dupe / Invalid / Not actionable milestone Jun 3, 2019
@bbondy bbondy removed this from the Dupe / Invalid / Not actionable milestone May 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants