This is a cleanup.
Created attachment 78597 [details] Patch
Comment on attachment 78597 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=78597&action=review > WebCore/editing/ApplyStyleCommand.cpp:571 > + Position rangeStart = firstPositionInNode(scope); Does this cause ref-churn for scope?
Created attachment 78618 [details] cleanup
(In reply to comment #2) > (From update of attachment 78597 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=78597&action=review > > > WebCore/editing/ApplyStyleCommand.cpp:571 > > + Position rangeStart = firstPositionInNode(scope); > > Does this cause ref-churn for scope? Good point. Removed.
Comment on attachment 78618 [details] cleanup On my second thought, I have to be more careful about what kind of Position I'm creating.
Created attachment 78633 [details] don't generate bogus position like [br, 0]
Comment on attachment 78633 [details] don't generate bogus position like [br, 0] Seems sane.
(In reply to comment #7) > (From update of attachment 78633 [details]) > Seems sane. Thanks for the review, Eric. Landing now.
Landed as http://trac.webkit.org/changeset/75672 and http://trac.webkit.org/changeset/75674.