RESOLVED FIXED 74748
Rename registerCommandFor(Undo|Redo) to register(Undo|Redo)Step 
https://bugs.webkit.org/show_bug.cgi?id=74748
Summary Rename registerCommandFor(Undo|Redo) to register(Undo|Redo)Step
Ryosuke Niwa
Reported 2011-12-16 14:23:06 PST
After http://trac.webkit.org/changeset/103104, the interface for unapply/reapplying an editing action has been changed from EditCommand to UndoStep. We should rename these methods as well.
Attachments
cleanup (25.55 KB, patch)
2011-12-16 14:52 PST, Ryosuke Niwa 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2011-12-16 14:52:36 PST
Created attachment 119678 [details] cleanup
Eric Seidel (no email)
Comment 2 2011-12-16 15:04:13 PST
Comment on attachment 119678 [details] cleanup View in context: https://bugs.webkit.org/attachment.cgi?id=119678&action=review > Source/WebCore/editing/Editor.cpp:892 > - client()->registerCommandForUndo(toCompositeEditCommand(m_lastEditCommand.get())->ensureComposition()); > + client()->registerUndoStep(toCompositeEditCommand(m_lastEditCommand.get())->ensureComposition()); If we do this casting often we should have a lastCommandAsComposite() or similar accessor :)
Ryosuke Niwa
Comment 3 2011-12-16 15:16:08 PST
Comment on attachment 119678 [details] cleanup View in context: https://bugs.webkit.org/attachment.cgi?id=119678&action=review >> Source/WebCore/editing/Editor.cpp:892 >> + client()->registerUndoStep(toCompositeEditCommand(m_lastEditCommand.get())->ensureComposition()); > > If we do this casting often we should have a lastCommandAsComposite() or similar accessor :) Actually, this cast is no longer necessary as m_lastEditCommand's type is now RefPtr<CompositeEditCommand>. Will remove the cast.
Ryosuke Niwa
Comment 4 2011-12-16 15:34:44 PST
Committed r103119: <http://trac.webkit.org/changeset/103119>
Note You need to log in before you can comment on or make changes to this bug.