[webkit-reviews] review granted: [Bug 201760] Eliminate separate simple line layout path from TextIterator : [Attachment 378794] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Sep 14 05:46:46 PDT 2019


zalan <zalan at apple.com> has granted Antti Koivisto <koivisto at iki.fi>'s request
for review:
Bug 201760: Eliminate separate simple line layout path from TextIterator
https://bugs.webkit.org/show_bug.cgi?id=201760

Attachment 378794: patch

https://bugs.webkit.org/attachment.cgi?id=378794&action=review




--- Comment #3 from zalan <zalan at apple.com> ---
Comment on attachment 378794
  --> https://bugs.webkit.org/attachment.cgi?id=378794
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=378794&action=review

> Source/WebCore/rendering/SimpleLineLayoutTextFragmentIterator.cpp:105
> +//	     bool isSingleTab = !multipleWhitespace &&
m_currentSegment->text[m_currentSegment->toSegmentPosition(startPosition)] ==
'\t';

I guess this is a leftover


More information about the webkit-reviews mailing list