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

Wrong page is shown in rewards panel when on YT #1660

Closed
btlechowski opened this issue Oct 17, 2018 · 8 comments · Fixed by brave/brave-core#712
Closed

Wrong page is shown in rewards panel when on YT #1660

btlechowski opened this issue Oct 17, 2018 · 8 comments · Fixed by brave/brave-core#712

Comments

@btlechowski
Copy link

Description

Steps to Reproduce

  1. Clean profile
  2. Enable rewards and claim grant
  3. Open google.com in new tab and wait till Send a tip is shown in rewards panel (reload if needed)
  4. Open youtube.com in another tab and wait till Send a tip is shown in rewards panel
  5. Switch to google.com tab and show rewards panel
  6. Switch to youtube.com tab and open a video
  7. Quickly switch back to google tab and wait for youtube page to load
  8. Switch to youtube tab and open rewards panel

Actual result:

Wrong page(google.com instead of youtube.com) is shown in rewards panel.
The page shown in the rewards panel will be used for tipping(google.com instead of youtube.com)
Reloading the page will not fix the issue.
image

Recording starts at step 5.
rewards_fail

Expected result:

The proper page is shown in rewards panel
image

Reproduces how often:

always

Brave version (brave://version info)

Verification passed on

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Windows 7 x64

cc @kjozwiak @LaurenWags @srirambv @GeetaSarvadnya @mandar-brave @NejcZdovc

@LaurenWags
Copy link
Member

Reproduced on macOS

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Mac OS X

screen shot 2018-10-17 at 1 49 49 pm

@NejcZdovc
Copy link
Contributor

@LaurenWags @btlechowski this will be fixed with this one #1517

@kjozwiak kjozwiak added this to the 1.x Backlog milestone Oct 17, 2018
@kjozwiak
Copy link
Member

As per @NejcZdovc, this will be fixed by #1517 which is inside the 1.x Backlog. Moved this issue into the same milestone.

@NejcZdovc
Copy link
Contributor

closing as dupe, because it has the same problem then #1517

@NejcZdovc NejcZdovc modified the milestones: 1.x Backlog, Dupe / Invalid / Not actionable Oct 18, 2018
@kjozwiak kjozwiak added closed/duplicate Issue has already been reported and removed QA/Yes feature/rewards labels Oct 19, 2018
@btlechowski
Copy link
Author

Reproduced in 0.55.20 on Ubuntu 17.10. #1517 did not fix the issue.

Brave 0.55.20 Chromium: 70.0.3538.67 (Official Build) (64-bit)
Revision 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS Linux

image

@btlechowski btlechowski reopened this Oct 23, 2018
@btlechowski btlechowski added feature/rewards QA/Yes and removed closed/duplicate Issue has already been reported labels Oct 23, 2018
@btlechowski btlechowski modified the milestones: Dupe / Invalid / Not actionable, 1.x Backlog Oct 23, 2018
@NejcZdovc NejcZdovc modified the milestones: 1.x Backlog, 0.57.x - Dev Oct 23, 2018
@kjozwiak
Copy link
Member

Here's STR that I've been using to reproduce the above:

  • launch 0.55.20 Chromium: 70.0.3538.67 using a brand new profile
  • visit https://www.youtube.com/channel/UCCatR7nWbYrkVXdxXb4cGXw
  • click on the BAT icon in the URL and click on Join Rewards
  • once the wallet has been created, open a new tab and load brave.com
  • click on the BAT icon in the URL and you should see brave.com
  • switch back to the YT tab (Datadash) and click on the BAT icon in the URL

You'll get the following:

screen shot 2018-10-23 at 12 31 04 am

@NejcZdovc NejcZdovc self-assigned this Oct 23, 2018
@NejcZdovc NejcZdovc changed the title Sometimes wrong page is shown in rewards panel Wrong page is shown in rewards panel when on YT Oct 23, 2018
@NejcZdovc
Copy link
Contributor

So I did a fix where when we switch tabs we don't preserve tabs. So now the expected behaviour is that when you switch back to https://www.youtube.com/channel/UCCatR7nWbYrkVXdxXb4cGXw tab panel should show wallet summary. This will be fully fixed when we add support for media publishers in #1282

NejcZdovc added a commit to brave/brave-core that referenced this issue Oct 24, 2018
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 26, 2018

Tried above 2 STR's, issue is fixed and working fine on:

Brave 0.57.3 Chromium: 70.0.3538.67 (Official Build) dev (64-bit)
Revision 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS Windows

Verified passed with

Brave 0.56.7 Chromium: 70.0.3538.77 (Official Build) beta(64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Mac OS X

Verification Passed on

Brave 0.56.8 Chromium: 70.0.3538.77 (Official Build) (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Linux

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