[webkit-reviews] review granted: [Bug 67763] Crashes in WebCore::InsertNodeBeforeCommand constructor. : [Attachment 106815] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 8 17:40:28 PDT 2011

Ryosuke Niwa <rniwa at webkit.org> has granted Annie Sullivan
<sullivan at chromium.org>'s request for review:
Bug 67763: Crashes in WebCore::InsertNodeBeforeCommand constructor.

Attachment 106815: Patch

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=106815&action=review


> + This test ensures that WebKit does not crash when the selection is outside
of the contenteditable area.

You should hide the meter element so that it won't add space in front of

I'd also like to see this test making sure the DOM is not changed by the
editing command.

> Source/WebCore/ChangeLog:10
> +	   root.

Could you fit "root" in the previous line so that it's easier to read?

More information about the webkit-reviews mailing list