TextControlElement::indexForVisiblePosition() should use EphemeralRange |
||||
Issue descriptionEphemralRange is ennough for TextControlElement::indexForVisiblePosition().
,
Feb 12 2017
,
Mar 2 2017
I take this.
,
Mar 2 2017
In review: http://crrev.com/2727923002
,
Mar 3 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/99ed48423a76851451298d54a5260cc30503fa1d commit 99ed48423a76851451298d54a5260cc30503fa1d Author: yosin <yosin@chromium.org> Date: Fri Mar 03 01:51:48 2017 Make TextControlElement::indexForVisiblePosition() not to use temporary Range This patch makes |TextControlElement::indexForVisiblePosition()| to call |TextIterator::rangeLength()| without using temporary |Range| object to avoid registering temporary |Range| object into register update list to reduce memory usage an relocation cost for improving code halth. BUG= 691200 TEST=n/a; no behavior change Review-Url: https://codereview.chromium.org/2727923002 Cr-Commit-Position: refs/heads/master@{#454461} [modify] https://crrev.com/99ed48423a76851451298d54a5260cc30503fa1d/third_party/WebKit/Source/core/html/TextControlElement.cpp
,
Mar 6 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by yosin@chromium.org
, Feb 11 2017