Changeset 95017 in webkit
- Timestamp:
- Sep 12, 2011 10:57:30 PM (13 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r95014 r95017 1 2011-09-12 Shinya Kawanaka <shinyak@google.com> 2 3 Crashes in WebCore::InsertListCommand::unlistifyParagraph. 4 https://bugs.webkit.org/show_bug.cgi?id=67918 5 6 Reviewed by Ryosuke Niwa. 7 8 Added tests. 9 10 * editing/execCommand/insert-list-in-noneditable-list-parent-expected.txt: Added. 11 * editing/execCommand/insert-list-in-noneditable-list-parent.html: Added. 12 1 13 2011-09-12 Sheriff Bot <webkit.review.bot@gmail.com> 2 14 -
trunk/Source/WebCore/ChangeLog
r95014 r95017 1 2011-09-12 Shinya Kawanaka <shinyak@google.com> 2 3 Crashes in WebCore::InsertListCommand::unlistifyParagraph. 4 https://bugs.webkit.org/show_bug.cgi?id=67918 5 6 Reviewed by Ryosuke Niwa. 7 8 execCommand("InsertUnorderedList") was crashing if the parent node of the target is 9 a kind of list element and it is not contenteditable. 10 This patch checks the parent node is contenteditable. 11 12 Test: editing/execCommand/insert-list-in-noneditable-list-parent.html 13 14 * editing/htmlediting.cpp: 15 (WebCore::enclosingListChild): Checks the parent node is contenteditable. 16 1 17 2011-09-12 Sheriff Bot <webkit.review.bot@gmail.com> 2 18 -
trunk/Source/WebCore/editing/htmlediting.cpp
r94832 r95017 667 667 // FIXME: This function is inappropriately named if it starts with node instead of node->parentNode() 668 668 for (Node* n = node; n && n->parentNode(); n = n->parentNode()) { 669 if (n->hasTagName(liTag) || isListElement(n->parentNode()))669 if (n->hasTagName(liTag) || (isListElement(n->parentNode()) && n != root)) 670 670 return n; 671 671 if (n == root || isTableCell(n))
Note: See TracChangeset
for help on using the changeset viewer.