DescriptionRevert of RenderFlexibleBox should be treated as a candidate for visible position. (patchset #1 id:1 of https://codereview.chromium.org/897633002/)
Reason for revert:
Revert for now until the fix is ready. Will also be reverted on release branch.
Original issue's description:
> RenderFlexibleBox should be treated as a candidate for visible position.
>
> Visible selection in an editable area is recalculated whenever an inner value of div is changed.
> If the inner value is empty, the visible selection recalculated with DeleteSelectionCommand::m_endingPostion
> is not correct. Because RenderBlockFlow is only considered but not RenderFlexibleBox in Position::isCandidate
> so that a calculated ending VisiblePosition for an editable div having ‘display : flex’ goes weird
> and VisibleSelection is empty accordingly.
>
> BUG=450617
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189482
TBR=sigbjornf@opera.com,rego@igalia.com,shivamidow@gmail.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450617
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=193212
Patch Set 1 #Patch Set 2 : Manual revert #
Messages
Total messages: 9 (3 generated)
|