[Webkit-unassigned] [Bug 211245] Enable NSZombieEnabled=YES on bots to catch use-after-release bugs sooner

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=211245

David Kilzer (:ddkilzer) <ddkilzer at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|EWS bots should run with    |Enable NSZombieEnabled=YES
                   |NSZombieEnabled=YES         |on bots to catch
                   |                            |use-after-release bugs
                   |                            |sooner

--- Comment #2 from David Kilzer (:ddkilzer) <ddkilzer at webkit.org> ---
(In reply to Alexey Proskuryakov from comment #1)
> I don't think that EWS should be getting more different from the normal way
> of running tests. If tests run well with NSZombieEnabled, then we should
> have that in local testing and post-commit too. EWS-only failures are the
> worst.

EWS finding failures early is also the best.  Lots of ways to implement this:  Enable NSZombieEnabled=YES everywhere; add separate bots just to run in NSZombieEnabled=YES mode; make NSZombieEnabled=YES part of a "catch flakey tests before they're committed" effort; etc.

I probably made the title too specific.  Tried to update the title.

> I don't know if NSZombieEnabled comes for free though.

It's not 100% free, but it may be worth the cost of catching bugs before they're committed.  Might be hard to gather metrics, though.

Part of the reason I filed this was simply to track issues that might have been caught sooner had this been implemented (to provide a case for it later), like Bug 211386.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20200504/dc34b3e8/attachment.htm>