32422 2009-12-11 04:24:09 -0800 InsertUnorderedList falls into an infinite loop because startOfCurrentParagraph stays on the same node 2011-08-01 09:41:26 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) PC Windows Vista RESOLVED FIXED http://skypher.com/SkyLined/Repro/WebKit/Bug%2032422%20-%20execCommand%20100%25%20CPU%20usage/repro.html P1 Normal --- 42436 36430 41403 1 skylined webkit-unassigned enrica eric rniwa tony oldest_to_newest 170686 0 44669 skylined 2009-12-11 04:24:09 -0800 Created attachment 44669 Repro <BODY></BODY> <SCRIPT> document.execCommand("SelectAll",false,""); document.designMode="on"; document.execCommand("InsertParagraph",false,1); document.execCommand("Indent",6); document.execCommand("insertimage",false,"ftp:"); document.execCommand("insertimage",false); document.execCommand("SelectAll",3); document.execCommand("subscript",""); document.execCommand("insertunorderedlist",false,3); </SCRIPT> 170688 1 skylined 2009-12-11 04:27:14 -0800 Does not affect Safari, only Chrome. 202691 2 rniwa 2010-03-22 12:31:55 -0700 I suspect this is a duplicate of the bug 33668. 202757 3 rniwa 2010-03-22 14:30:22 -0700 This bug is caused by startOfCurrentParagraph not moving forward on http://trac.webkit.org/browser/trunk/WebCore/editing/InsertListCommand.cpp#L96. It stays on the same node while we add more LI elements to the document. We need to cleanup InsertListCommmand first so that removing/creating list element does not rely on selection. 444558 4 skylined 2011-08-01 00:54:05 -0700 @Ryosuke: the repro loads fast and without crashing for me in Chromium 15.0.840.0 with WebKit 535.1 (@91997). I'm assuming this got fixed together with some of your other changes to code in this area. 444724 5 rniwa 2011-08-01 09:41:26 -0700 As I said on some other bugs, there's an inherent design issue in IndentOutdentCommand and InsertListCommand, and it's simply not worth my time to fix any inifinite loop or crash involving these two classes until we come up with a more comprehensive design fix. 44669 2009-12-11 04:24:09 -0800 2009-12-11 04:24:09 -0800 Repro repro.html text/html 424 skylined PEJPRFk+PC9CT0RZPgo8U0NSSVBUPgogIGRvY3VtZW50LmV4ZWNDb21tYW5kKCJTZWxlY3RBbGwi LGZhbHNlLCIiKTsKICBkb2N1bWVudC5kZXNpZ25Nb2RlPSJvbiI7CiAgZG9jdW1lbnQuZXhlY0Nv bW1hbmQoIkluc2VydFBhcmFncmFwaCIsZmFsc2UsMSk7CiAgZG9jdW1lbnQuZXhlY0NvbW1hbmQo IkluZGVudCIsNik7CiAgZG9jdW1lbnQuZXhlY0NvbW1hbmQoImluc2VydGltYWdlIixmYWxzZSwi ZnRwOiIpOwogIGRvY3VtZW50LmV4ZWNDb21tYW5kKCJpbnNlcnRpbWFnZSIsZmFsc2UpOwogIGRv Y3VtZW50LmV4ZWNDb21tYW5kKCJTZWxlY3RBbGwiLDMpOwogIGRvY3VtZW50LmV4ZWNDb21tYW5k KCJzdWJzY3JpcHQiLCIiKTsKICBkb2N1bWVudC5leGVjQ29tbWFuZCgiaW5zZXJ0dW5vcmRlcmVk bGlzdCIsZmFsc2UsMyk7CjwvU0NSSVBUPg==