[webkit-reviews] review granted: [Bug 26872] Crash indenting a table cell : [Attachment 32167] Fixes crash.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 2 14:34:07 PDT 2009


Eric Seidel <eric at webkit.org> has granted Ojan Vafai <ojan at chromium.org>'s
request for review:
Bug 26872: Crash indenting a table cell
https://bugs.webkit.org/show_bug.cgi?id=26872

Attachment 32167: Fixes crash.
https://bugs.webkit.org/attachment.cgi?id=32167&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
The fix looks fine to me.  I'm not sure if it's at the right layer or not. 
Justin would know that better.

I assume <td> and root contentEditable nodes are the only non-splitable nodes?

Justin has historically been very slow to respond to Bugzilla bugs, so I'm
going to go ahead and r+ this and we can roll it out or fix it differently if
Justin later comments and feels it should be done at a different level.


More information about the webkit-reviews mailing list