DescriptionMake CompositeEditCommand::insertNodeBefore() to update layout for hasEditableStyle()
This patch makes |CompositeEditCommand::insertNodeBefore()| to update layout for
|hasEditableStyle()| to get correct value of |hasEditableStyle()| for sanity
of implementation and a preparation of [1], which may update layout during
construction of |insertNodeBeforeCommand| by computing |VisibleSelection| in
|EditCommand| constructor.
This patch also add regression test cases for
|CompositeEditCommand::insertNodeBefore()| along with new behavior for calling
it on dirty layout tree.
[1] http://crrev.com/2637013002: Make FrameSelection to hold non-canonicalized
positions
BUG=n/a
TEST=run_webkit_unittests --gtest_filter=CompositeEditCommandTest.insertNodeBeforeWithDirtyLayoutTree
Review-Url: https://codereview.chromium.org/2631133002
Cr-Commit-Position: refs/heads/master@{#444329}
Committed: https://chromium.googlesource.com/chromium/src/+/acefebccb68d844f98e746f7877360070abe8349
Patch Set 1 : 2017-01-16T18:23:50 #Patch Set 2 : 2017-01-16T20:09:30 #
Total comments: 2
Messages
Total messages: 23 (13 generated)
|