While working on the bug 117165, I've converted the following comment in CompositeEditCommand::apply to an assertion: // Only need to call appliedEditing for top-level commands, // and TypingCommands do it on their own (see TypingCommand::typingAddedToOpenCommand). if (!isTypingCommand()) frame->editor().appliedEditing(this); and found a nasty bug where TypingCommand wasn't registering itself to the undo manager in some cases.
Not quite. My assertion missed one corner case.
Created attachment 203640 [details] Adds an assertion
Comment on attachment 203640 [details] Adds an assertion Attachment 203640 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/660560
Comment on attachment 203640 [details] Adds an assertion r=me. I don't see how the EFL failure is related to this change.
Comment on attachment 203640 [details] Adds an assertion Attachment 203640 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/700665 New failing tests: editing/pasteboard/paste-line-endings-002.html editing/pasteboard/paste-4038267-fix.html editing/execCommand/4920488.html editing/pasteboard/paste-line-endings-003.html http/tests/inspector/inspect-element.html editing/pasteboard/paste-line-endings-004.html editing/inserting/insert-empty-html.html editing/style/iframe-onload-crash-win.html editing/execCommand/5573879.html editing/style/iframe-onload-crash-unix.html editing/deleting/delete-ligature-003.html editing/pasteboard/smart-paste-001.html editing/execCommand/19455.html editing/style/push-down-font-styles-win.html editing/deleting/delete-before-block-table.html editing/execCommand/5049671.html editing/deleting/5847330-1.html editing/pasteboard/paste-line-endings-005.html editing/style/push-down-font-styles-mac.html editing/inserting/space-after-removeformat.html editing/execCommand/19653-1.html editing/style/iframe-onload-crash-mac.html editing/deleting/delete-to-select-table.html editing/deleting/delete-before-block-image-1.html editing/undo/orphaned-selection-crash-bug32823-4.html editing/deleting/delete-by-word-002.html editing/execCommand/19403.html
Created attachment 203652 [details] Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-15 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.3
Comment on attachment 203640 [details] Adds an assertion Attachment 203640 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/700746 New failing tests: editing/pasteboard/paste-line-endings-002.html editing/pasteboard/paste-4038267-fix.html editing/execCommand/4920488.html http/tests/security/mixedContent/redirect-https-to-http-iframe-in-main-frame.html editing/undo/orphaned-selection-crash-bug32823-2.html editing/pasteboard/paste-line-endings-003.html http/tests/inspector/inspect-element.html fast/forms/input-text-paste-maxlength.html editing/pasteboard/paste-line-endings-004.html editing/inserting/insert-empty-html.html editing/style/iframe-onload-crash-win.html editing/style/iframe-onload-crash-unix.html editing/deleting/delete-ligature-003.html editing/execCommand/19455.html editing/style/push-down-font-styles-win.html editing/deleting/delete-before-block-table.html editing/execCommand/5049671.html editing/deleting/5847330-1.html editing/pasteboard/paste-line-endings-005.html editing/style/push-down-font-styles-mac.html editing/inserting/space-after-removeformat.html editing/execCommand/19653-1.html editing/style/iframe-onload-crash-mac.html editing/deleting/delete-to-select-table.html editing/deleting/delete-before-block-image-1.html editing/undo/orphaned-selection-crash-bug32823-4.html editing/deleting/delete-by-word-002.html editing/execCommand/19403.html
Created attachment 203680 [details] Archive of layout-test-results from webkit-ews-05 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-05 Port: mac-mountainlion Platform: Mac OS X 10.8.3