[Webkit-unassigned] [Bug 67767] Crashed in WebCore::AppendNodeCommand::create(WTF::PassRefPtr<WebCore::ContainerNode>, WTF::PassRefPtr<WebCore::Node>)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 8 08:39:21 PDT 2011


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


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #106728|review?                     |review+
               Flag|                            |




--- Comment #2 from Darin Adler <darin at apple.com>  2011-09-08 08:39:21 PST ---
(From update of attachment 106728)
View in context: https://bugs.webkit.org/attachment.cgi?id=106728&action=review

This looks OK. It's a little strange to just do nothing if the node is not an element and the patch would be better if it explained why that is the correct thing to do, not just that it won’t crash.

> LayoutTests/editing/execCommand/ident-crashes-topnode-is-text.html:9
> +    document.writeln("This test ensures WebKit does not crash.<br><br>PASS");

The wording here is wrong. The test does not ensure that WebKit does not crash. The test has no effect on WebKit.

It's good to say that the test has passed if it doesn’t crash, and also probably good to say as clearly as possible why there was a time when the test would crash.

> LayoutTests/editing/execCommand/ident-crashes-topnode-is-text.html:12
> +<meta content="2"/><body onLoad="runTest();">

What is the significance of this "meta" element?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list