Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Webview crash when visiting https://jsfiddle.net/bkf50r8v/13/ #7885

Closed
srirambv opened this issue Mar 25, 2017 · 3 comments · Fixed by brave/muon#172
Closed

Webview crash when visiting https://jsfiddle.net/bkf50r8v/13/ #7885

srirambv opened this issue Mar 25, 2017 · 3 comments · Fixed by brave/muon#172

Comments

@srirambv
Copy link
Collaborator

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Webview crash when visiting https://jsfiddle.net/bkf50r8v/13/

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version (revision SHA):
    Brave 0.14.0
    rev ffa6f8f

  • Steps to reproduce:

    1. Do a clean install of 0.14.0 Preview 5
    2. Visit https://jsfiddle.net/bkf50r8v/13/
    3. Web view crashes with frown face
  • Actual result:
    Web view crashes with frown face

  • Expected result:
    Web view should not crash

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    No

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    webrender crash

  • Any related issues:
    cc: @darkdh @bridiver

@bridiver
Copy link
Collaborator

is this a regression? If it's not I'm not sure that I would label it high impact.

@darkdh
Copy link
Member

darkdh commented Mar 27, 2017

exactly save behavior as #6522 but different dll
It was ole32.dll and now it is ws2_32.dll

Unhandled exception at 0x00007FFDC2BE7788 (KernelBase.dll) in brave.exe: 0xC06D007E: Module not found (parameters: 0x0000002C1091D000).

@srirambv
Copy link
Collaborator Author

is this a regression?

Yes it is. Worked fine in previous releases. Missed adding the label, update now. I have seen this sad face crash on some video streaming sites as well. Not consistently but one or two out of ten attempts.

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