RESOLVED FIXED 73984
TypingCommand duplicates code to obtain the last typing command 
https://bugs.webkit.org/show_bug.cgi?id=73984
Summary TypingCommand duplicates code to obtain the last typing command
Ryosuke Niwa
Reported 2011-12-06 23:48:22 PST
Lots of code like: ASSERT(document); Frame* frame = document->frame(); ASSERT(frame); EditCommand* lastEditCommand = frame->editor()->lastEditCommand(); if (isOpenForMoreTypingCommand(lastEditCommand)) { is duplicated throughout TypingCommand.
Attachments
cleanup (11.61 KB, patch)
2011-12-07 00:18 PST, Ryosuke Niwa 		tkent: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2011-12-07 00:18:36 PST
Created attachment 118179 [details] cleanup
Kent Tamura
Comment 2 2011-12-07 00:29:03 PST
Comment on attachment 118179 [details] cleanup Looks good.
Ryosuke Niwa
Comment 3 2011-12-07 00:55:40 PST
Thanks for the review!
Ryosuke Niwa
Comment 4 2011-12-07 01:03:58 PST
Committed r102225: <http://trac.webkit.org/changeset/102225>
Note You need to log in before you can comment on or make changes to this bug.