RESOLVED FIXED 49155
Remove the remaining editing-style functions from ApplyStyleCommand 
https://bugs.webkit.org/show_bug.cgi?id=49155
Summary Remove the remaining editing-style functions from ApplyStyleCommand
Ryosuke Niwa
Reported 2010-11-07 20:22:44 PST
This is a follow up to the bug 46335, which introduced EditingStyle. In this follow up, we remove removeNonEditingProperties, editingStyleAtPosition, prepareEditingStyleToApplyAt, and removeStylesAddedByNode from ApplyStyleCommand.cpp/h.
Attachments
cleanup (35.29 KB, patch)
2010-11-07 22:04 PST, Ryosuke Niwa 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2010-11-07 22:04:11 PST
Created attachment 73211 [details] cleanup
Darin Adler
Comment 2 2010-11-08 10:43:40 PST
Comment on attachment 73211 [details] cleanup View in context: https://bugs.webkit.org/attachment.cgi?id=73211&action=review Seems good. Refactoring looks like it was done correctly. > WebCore/editing/EditingStyle.h:72 > + void removeStyleAddedByNode(Node* node); > + void removeStyleConflictingWithStyleOfNode(Node* node); No need for argument names here in these function declarations.
Ryosuke Niwa
Comment 3 2010-11-08 12:15:51 PST
Committed r71556: <http://trac.webkit.org/changeset/71556>
Note You need to log in before you can comment on or make changes to this bug.