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

Restored sites got deleted after page reload, when rewards page opened in multiple windows #3264

Closed
GeetaSarvadnya opened this issue Feb 7, 2019 · 7 comments · Fixed by brave/brave-core#1803

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Feb 7, 2019

Steps to Reproduce

  1. Clean profile 0.60.19
  2. Enable rewards through rewards panel
  3. Add any site to a-c table (Window 1)
  4. Right click on the site in a-c table and open it in new window ( Window2)
  5. Open Rewards page in Window2 through rewards panel
  6. Make sure site which is added in step 3 is available in a-c table in Window2
  7. Go to Window1, delete the site from a-c table
  8. Go to Window2 and see site deleted from there as well
  9. Click on Restore All on Window1 and see that site got restored and visible in a-c table
  10. Go to Window2 and see site is not visible in a-c table, Refresh the rewards page and see site is not
    restored
  11. Go to Window1 and reload the rewards page.
  12. Site which was restored got deleted after page reload

Actual result:

Restored sites got deleted after page reload, when rewards page opened in multiple windows

Expected result:

Restored sites should not get deleted

Reproduces how often:

Always

Brave version (brave://version info)

Brave 0.60.19 Chromium: 72.0.3626.81 (Official Build) beta (64-bit)
Revision ac8b982e05014492d1bd7d317628a4f22a97ffa0-refs/branch-heads/3626@{#796}
OS Windows 10

Reproducible on current release: No

  • 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

On single Rewards page it perfectly works fine, issue arises only when there are multiple rewards pages.

@brave/legacy_qa @NejcZdovc

@NejcZdovc NejcZdovc removed their assignment Feb 7, 2019
@GeetaSarvadnya GeetaSarvadnya added regression priority/P3 The next thing for us to work on. It'll ride the trains. labels Feb 8, 2019
@LaurenWags
Copy link
Member

Reproduced with

Brave 0.60.23 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Mac OS X

@LaurenWags
Copy link
Member

Could not reproduce with current released version:

Brave 0.59.35 Chromium: 72.0.3626.81 (Official Build) (64-bit)
Revision ac8b982e05014492d1bd7d317628a4f22a97ffa0-refs/branch-heads/3626@{#796}
OS Mac OS X

@LaurenWags
Copy link
Member

LaurenWags commented Feb 8, 2019

Additionally, using 0.60.23, I followed the STR from the issue and then also did the following after step 12:
13. Visited a new site in window1 for min amount of time. Verified it was added to the ac table on both windows. Note: % is acting as if the first site from step 3 is still visible, when it is not.
14. Visited a second new site in window2 for min amount of time. Verified it was added to the ac table on both windows. Note: % is acting as if the first site from step 3 is still visible, when it is not.
Here's what I see in a-c table and in publisher_info_db:
screen shot 2019-02-08 at 11 40 40 am
screen shot 2019-02-08 at 11 41 31 am
screen shot 2019-02-08 at 11 41 44 am
As you can see, github is in my database, but not displayed on my a-c table.

@LaurenWags
Copy link
Member

@GeetaSarvadnya can you check this one on latest 0.61.x?

@NejcZdovc NejcZdovc added this to the 0.63.x - Nightly milestone Mar 7, 2019
@NejcZdovc
Copy link
Contributor

this one was fixed with brave/brave-core#1803

@GeetaSarvadnya
Copy link
Author

@LaurenWags Issue is Reproducible on 0.61.51

Brave 0.61.51 Chromium: 73.0.3683.75 (Official Build) (64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Windows 10 OS Build 17134.523

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Mar 13, 2019

Verification passed on

Brave 0.63.5 Chromium: 73.0.3683.67 (Official Build) dev (64-bit)
Revision a83fd4f3207ae83412d329a9ca1239dd1e068345-refs/branch-heads/3683@{#760}
OS Windows 10 OS Build 17134.523

Verification passed on

Brave 0.63.14 Chromium: 73.0.3683.75 (Official Build) dev(64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Ubuntu 18.04 LTS

Verified passed with

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants