[webkit-reviews] review denied: [Bug 75859] IDBDatabase destructor is never executed : [Attachment 121812] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 10 02:23:20 PST 2012


Tony Gentilcore <tonyg at chromium.org> has denied jochen at chromium.org's request
for review:
Bug 75859: IDBDatabase destructor is never executed
https://bugs.webkit.org/show_bug.cgi?id=75859

Attachment 121812: Patch
https://bugs.webkit.org/attachment.cgi?id=121812&action=review

------- Additional Comments from Tony Gentilcore <tonyg at chromium.org>
It actually looks like this will cause us to lose the fix applied by
http://trac.webkit.org/changeset/78280 . I agree this looks like a real
problem, but I suspect the solution is to find a smarter condition for
hasPendingActivity rather than to remove it entirely. Alternatively, some
explanation or justification for why this is the correct solution should be in
the ChangeLog.


More information about the webkit-reviews mailing list