| Summary: | Release WebGLLayer earlier in ~GraphicsContextGLOpenGL | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Kenneth Russell <kbr> | ||||
| Component: | WebGL | Assignee: | Kenneth Russell <kbr> | ||||
| Status: | RESOLVED WONTFIX | ||||||
| Severity: | Normal | CC: | dino, ews-watchlist, graouts, justin_fan, kondapallykalyan, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Bug Depends on: | 210151 | ||||||
| Bug Blocks: | 210153, 210222 | ||||||
| Attachments: |
|
||||||
|
Description
Kenneth Russell
2020-04-08 14:40:44 PDT
Created attachment 395871 [details]
Patch
Committed r259764: <https://trac.webkit.org/changeset/259764> All reviewed patches have been landed. Closing bug and clearing flags on attachment 395871 [details]. Reverted r259764 for reason: Causes layout test crashes under GuardMalloc Committed r259915: <https://trac.webkit.org/changeset/259915> (In reply to Ryan Haddad from comment #4) > Reverted r259764 for reason: > > Causes layout test crashes under GuardMalloc > > Committed r259915: <https://trac.webkit.org/changeset/259915> We were seeing crashes on iOS Simulator when running tests under GuardMalloc These are some of the tests that hit crashes: fast/canvas/webgl/no-info-log-for-simple-shaders.html fast/canvas/webgl/tex-image-and-sub-image-2d-with-image.html webgl/1.0.3/conformance/extensions/ext-blend-minmax.html fast/canvas/webgl/no-info-log-for-simple-shaders.html Thanks Ryan and sorry about the crashes. Was worried about that. How can I run my local iOS Simulator build with GuardMalloc? Note - per fix in Bug 210222, if that one lands and sticks, I'll likely close this as WontFix rather than trying to pursue it further. The fix for Bug 210222 seems that it's stable and won't be reverted, so closing this out as WontFix. |