-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Webview crashes when ad's enabled #2870
Comments
not sure what you mean by a "webview" crash. We don't have webviews in brave-core. Did Brave crash? The "actual result" is a bit confusing to me as well because it shows |
oops, closed by accident ;) |
The blank screen with the frown face when you visit |
got it, which crash id is it? The most recent one is not uploaded so there is no server crash id |
The most recent one wasn't for the webview. Not sure how it got captured. All the ones between |
Not seeing this while testing on macOS. I've viewed each ad I've received for the past hour or so, but no crashes experienced.
|
@srirambv so which tab crashes? The tab that was open or the new tab that opens from clicking on the notification? These crashes are not all the same and don't look ads related, I think this might be an upstream issue that just happens to get triggered by creating a new tab this way |
Looks like the tab that was open since the crash reports are mostly |
@bridiver That profile where i had these crashes, new tab opened up blank each time. But i couldn't reproduce the same behaviour on a new profile. As for |
And I would say @srirambv that we should probably just ignore the case for iab.com for now since actually users won't use that to trigger. Only the forbes case would be something an actual user would encounter. |
@GeetaSarvadnya can you upload the crash reports? That's the only way I can view them |
@bridiver Click on |
@GeetaSarvadnya did you enable crash reporting? I saw in the screenshot that it was disabled, but there is a delay so usually clicking on the link and then restarting the browser will send it |
@GeetaSarvadnya also your STR doesn't seem to match your screenshot. You have more open tabs than the STR would create |
I'm not able to repro either of the STRs on master. Can someone else verify? |
@bridiver I cannot reproduce on master |
@bridiver I have enabled |
@bridiver #2870 (comment) - To get Crash reports i have closed and re-opened browser multiple times and multiple BTW I just tried reproducing the issue on Latest dev build 0.60.16 - Issue is reproducible, but i am not getting Uploaded crash report id's. @srirambv It would be nice, if you can reverify the issue again on latest build and share your inputs. |
@GeetaSarvadnya close and reopen Brave and it should send the reports |
@srirambv Can you please retest and let us know if you can reproduce using latest master |
@tmancey I am unable to build on master right now. I can give it a try on the latest dev build and update here |
@srirambv If you could try once brave/brave-core#1645 is merged to master (0.62.x dev) that would be great |
Running from source causes the active page to have a webview crash irrespective of it being a brave page or a webpage.
I am not seeing any ads on running |
@srirambv is retesting as Brave Core was on wrong branch |
Manually switched branch on |
We found this issue was caused due to Git Bash. @srirambv can you please link this ticket to your new ticket and retest using Command Prompt. |
Unable to reproduce the issue when launching via command prompt. Closing as fixed. |
Description
Webview crashes when ad's enabled
Steps to Reproduce
0.60.9
Actual result:
Expected result:
There shouldn't be any webview crash
Reproduces how often:
Easy
Brave version (brave://version info)
Reproducible on current release:
Yes on dev build
Website problems only:
N/A
N/A
Additional Information
Issue reproduced by @GeetaSarvadnya
cc: @bridiver @jsecretan @brave/legacy_qa
The text was updated successfully, but these errors were encountered: