TextIterator/VisibleUnits should use FirstLetterPseudoElement::firstLetterTextLayoutObject instead of traversing layout tree |
||||||||
Issue descriptionTextIterator and VisibleUnits traverses the layout tree to find first-letter objects. They should probably use |FirstLetterPseudoElement::firstLetterTextLayoutObject| instead. https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/dom/FirstLetterPseudoElement.cpp?q=FirstLetterPseudoElement::firstLetterTextLayoutObject&sq=package:chromium&l=97&dr=CSs This is derived from issue 684684 .
,
Aug 24 2017
xiaochengh@, WDYT? It seems we also change AssociatedLayoutObjectOf() to use |FirstLetterPseudoElement::firstLetterTextLayoutObject|. It seems |firstLetterTextLayoutObject()| does match more than |AssociatedLayoutObjectOf()|.
,
Aug 24 2017
To simplify |FirstLetterPseudoElement::firstLetterTextLayoutObject()|, it is better to member variable |another_part_| to |LayoutTextFragment|, which is initialized in FirstLetterPseudoElement::AttachFirstLetterTextLayoutObjects().
,
Sep 6 2017
Assign to yosin@ who is actively working on bugs related to ::first-letter.
,
Sep 15 2017
Mark Available to make someone picks this up.
,
Sep 15 2017
,
Nov 13 2017
,
Nov 14
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 19
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by yoichio@chromium.org
, Jan 27 2017