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

Browser crash when attempting to tweet after tipping #5516

Closed
btlechowski opened this issue Aug 2, 2019 · 34 comments · Fixed by brave/brave-core#8444
Closed

Browser crash when attempting to tweet after tipping #5516

btlechowski opened this issue Aug 2, 2019 · 34 comments · Fixed by brave/brave-core#8444

Comments

@btlechowski
Copy link

btlechowski commented Aug 2, 2019

After tipping, clicking on Tweet may crash the browser

Steps to Reproduce

  1. Clean install on staging
  2. Enable Rewards
  3. Claim grant or fund wallet another way
  4. Open publisher page (https://www.reddit.com/user/jsadler-brave)
  5. Use inline tipping
  6. Click tip amount
  7. Quickly click another tip amount (repeat few times)
  8. Immediately click SEND MY TIP
  9. Immediately click Tweet

Note: This is an Intermittent issue, if browser does not crash, start from step 5.

Actual result:

Browser Crash. Needs to run the browser again.

Windows 7 Uploaded Crash Report ID 7c1b0025c7afe8b9 (Local Context: fe8dec3f-846c-494d-8c7f-6305e621f7a0)
Ubuntu 18.04 LTS Crash Report ID d165028fa3f61bfa

Expected result:

No crash

Reproduces how often:

Intermittent issue, around 20% repro rate

Brave version (brave://version info)

Brave 0.68.114 Chromium: 76.0.3809.87 (Official Build) beta (64-bit)
Revision 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Ubuntu 18.04 LTS
Brave 0.68.114 Chromium: 76.0.3809.87 (Official Build) beta (64-bit)
Revision 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Windows 7 Service Pack 1 (Build 7601.24494)

Reproduced on staging and production. Also reproduced by tipping youtube publisher.

cc @NejcZdovc @brave/legacy_qa

@btlechowski
Copy link
Author

Still reproducible

Brave 0.68.120 Chromium: 76.0.3809.87 (Official Build) beta (64-bit)
Revision 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Ubuntu 18.04 LTS

Uploaded Crash Report ID fa3223c63ab8d96c (Local Context: Brave)
Crash report uploaded on Friday, August 9, 2019 at 1:05:16 AM

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Aug 16, 2019
@masparrow
Copy link

@NejcZdovc is it possible to get access to the crash reports cited above?

@masparrow
Copy link

masparrow commented Sep 25, 2019

@btlechowski hi - a quick question - when it works (doesn't crash) what does the 'Tweet' button do for you? For me, nothing happens after clicking other than it dismisses the banner.

Also, did you use a VPN to get the grant claim? If so which one please, as there may be a subtle timing issue involved.

@btlechowski
Copy link
Author

btlechowski commented Sep 30, 2019

@masparrow re grant, your don't really need the grant, you just need funds to tip

I reproduced the issue with Tweet button. This is not as it used to be before. Maybe it is related to the crash.
You should see something like that:
image

Does not reproduce on Twitter(inline tipping), youtube and regular page.

edit: I logged #6223. You are very likely blocked on this issue.

@NejcZdovc
Copy link
Contributor

@btlechowski can you please send crash id or terminal logs

@btlechowski
Copy link
Author

The crash ids are in the description.

I will repaste them for convenience:
Windows 7 Uploaded Crash Report ID 7c1b0025c7afe8b9 (Local Context: fe8dec3f-846c-494d-8c7f-6305e621f7a0)
Ubuntu 18.04 LTS Crash Report ID d165028fa3f61bfa
Uploaded Crash Report ID fa3223c63ab8d96c (Local Context: Brave)

@NejcZdovc
Copy link
Contributor

@btlechowski thank you, I missed it.

@masparrow
Copy link

@btlechowski @LaurenWags with the fix from PR 3592 (which resolves #6223) I have been unable to reproduce the crash from this issue. I've burned through 60 BAT (and I'm a cheap tipper ;) )and every time the Tweet page loads as expected. I'd recommend retesting this issue once #6223 is resolved (slated for 0.72 nightly ATM).

@LaurenWags
Copy link
Member

@masparrow if you would like, let us know once the fix for #6223 has been merged to Nightly (looks like it's still open atm) and we can check this on Nightly for you.

@masparrow
Copy link

@LaurenWags #6223 PR merged :) TIA!

@NejcZdovc NejcZdovc removed this from the 0.72.x - Nightly milestone Oct 3, 2019
@LaurenWags
Copy link
Member

@btlechowski I have not been able to reproduce with the following version. Can you please retest as well?

Brave 0.72.72 Chromium: 77.0.3865.90 (Official Build) nightly (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)

@btlechowski
Copy link
Author

btlechowski commented Oct 8, 2019

I am still able to reproduce the crash. I was testing on staging with grant.

Brave 0.72.77 Chromium: 77.0.3865.90 (Official Build) nightly (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Ubuntu 18.04 LTS

Uploaded Crash Report ID 0e2953da93e8455a (Local Context: Brave)

I will try to get a recording tomorrow. Maybe it will help.

@masparrow
Copy link

@btlechowski A recording would be great - just in case some subtle order of events or timing is key to reproducing.

@btlechowski
Copy link
Author

btlechowski commented Oct 8, 2019

I managed to record it.

I used staging env with a grant. I was connected to Belgium through VPN.

I also noticed that if crash does not reproduce, there is no need to start from step 1. It is OK to start from step 5.

5516

Uploaded crash id for this crash: 0e594ec66abf309d

@btlechowski
Copy link
Author

@kjozwiak The crash report was not generated. Probably due to #4296.

@LaurenWags
Copy link
Member

@kjozwiak could not reproduce issue, updated #5516 (comment) with my notes. We should probably get a follow up issue logged for Win/Linux if this is still occurring there 👍

@btlechowski
Copy link
Author

After internal discussion it was decided to reopen this issue and mark it as OS/Linux and OS/Windows

@btlechowski
Copy link
Author

Still crashing

Brave 1.18.74 Chromium: 87.0.4280.101 (Official Build) (64-bit)
Revision 9407c80213cda69c2b7abcb4fa8e3f74488f4956-refs/branch-heads/4280@{#1807}
OS Ubuntu 18.04 LTS

@LaurenWags
Copy link
Member

I reproduced on Linux as well today

Brave	1.18.74 Chromium: 87.0.4280.101 (Official Build) (64-bit)
Revision	9407c80213cda69c2b7abcb4fa8e3f74488f4956-refs/branch-heads/4280@{#1807}
OS	Linux

Moving to Rewards Global board for triage cc @marshall

@zenparsing
Copy link

Possibly related: #15142

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 16, 2021

Verification passed on


Brave | 1.24.65 Chromium: 90.0.4430.72 (Official Build) dev (64-bit)
-- | --
Revision | b6172ef8d07ef486489a4b11b66b2eaeed50d132-refs/branch-heads/4430@{#1233}
OS | Windows 10 OS Version 2004 (Build 19041.867)

  • Verified the STR from the description and ensured there is no crash when tweet after tipping

Verification passed on

Brave 1.24.70 Chromium: 90.0.4430.72 (Official Build) beta (64-bit)
Revision b6172ef8d07ef486489a4b11b66b2eaeed50d132-refs/branch-heads/4430@{#1233}
OS Ubuntu 18.04 LTS

Verified the STR from the description. No crash.
The twitter login was not shown. Logged #15565

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