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

Web view crash when tearing/attaching the tabs back #9972

Closed
srirambv opened this issue Jul 12, 2017 · 4 comments
Closed

Web view crash when tearing/attaching the tabs back #9972

srirambv opened this issue Jul 12, 2017 · 4 comments

Comments

@srirambv
Copy link
Collaborator

srirambv commented Jul 12, 2017

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

  • Describe the issue you encountered:
    Web view crash when tearing/attaching the tabs back

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

  • Brave Version (revision SHA):
    Originally found on

Brave 0.17.15
rev 39009b8
Muon 4.1.8

Confirmed happening on while testing for (#9865)

Brave 0.17.9
rev 2606974
Muon 4.1.4
  • Steps to reproduce:

    1. Clean install the 0.17.x build
    2. Visit a site in multiple tabs
    3. Tear off one of them, new window has webview crash (0.17.15)
    4. Tear off one of the tab into window, reattach it to the parent window, webview crashes (0.17.9)
  • Actual result:
    Web view crash when tearing/attaching the tabs back

  • Expected result:
    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?
    Possibly yes

  • Can this issue be consistently reproduced?
    No happens at random times. Following the same steps as mentioned in other builds doesn't cause the crash

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    0.17.9
    image
    0.17.15
    image

  • Any related issues:
    cc: @bridiver

@srirambv
Copy link
Collaborator Author

Still happening on a clean profile on 0.18;24. Found while testing tab page
pin

@srirambv
Copy link
Collaborator Author

cc: @bbondy @bsclifton

@bsclifton
Copy link
Member

@srirambv is this still an issue? I know @bbondy had a lot of recent fixes with tear off tabs (which should have landed in 0.18.25)

@luixxiul luixxiul added the needs-info Another team member needs information from the PR/issue opener. label Sep 9, 2017
@srirambv
Copy link
Collaborator Author

Verified from 0.18.29-0.18.32 with steps mention in #9972 (comment), no crash when tearing off tabs.

@luixxiul luixxiul removed the needs-info Another team member needs information from the PR/issue opener. label Sep 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants