[webkit-reviews] review granted: [Bug 16163] SVG crash in Node::setChanged() on Debug builds only (trashed parent) : [Attachment 17806] First attempt

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Dec 9 16:06:10 PST 2007


Nikolas Zimmermann <zimmermann at kde.org> has granted Rob Buis
<rwlbuis at gmail.com>'s request for review:
Bug 16163: SVG crash in Node::setChanged() on Debug builds only (trashed
parent)
http://bugs.webkit.org/show_bug.cgi?id=16163

Attachment 17806: First attempt
http://bugs.webkit.org/attachment.cgi?id=17806&action=edit

------- Additional Comments from Nikolas Zimmermann <zimmermann at kde.org>
Good spot! r=me.

Is there a specific reason to not use a for-loop, but the setTimeout solution?
Is it a race condition?

If not necessary I'd suggest to change that, and maybe use some more
descriptive variable/element names ;-)


More information about the webkit-reviews mailing list