No need to truncate text after calling InlineTextBox::text()
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Oct 2017 21:42:19 +0000 (21:42 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Oct 2017 21:42:19 +0000 (21:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=177748

Reviewed by Zalan Bujtas.

Following r222670 it is no longer necessary to explicitly truncate the text run length
as InlineTextBox::text() returns a text run with respect to the truncation of the text box.

No functionality changed. So, no new tests.

* rendering/InlineTextBox.cpp:
(WebCore::InlineTextBox::paint):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@222745 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/InlineTextBox.cpp

index f6b658d..f3ec4c4 100644 (file)
@@ -1,3 +1,18 @@
+2017-10-02  Daniel Bates  <dabates@apple.com>
+
+        No need to truncate text after calling InlineTextBox::text()
+        https://bugs.webkit.org/show_bug.cgi?id=177748
+
+        Reviewed by Zalan Bujtas.
+
+        Following r222670 it is no longer necessary to explicitly truncate the text run length
+        as InlineTextBox::text() returns a text run with respect to the truncation of the text box.
+
+        No functionality changed. So, no new tests.
+
+        * rendering/InlineTextBox.cpp:
+        (WebCore::InlineTextBox::paint):
+
 2017-10-02  Antti Koivisto  <antti@apple.com>
 
         Add makeWeakPtr variant that takes pointer
index 16905fd..13673fc 100644 (file)
@@ -514,7 +514,6 @@ void InlineTextBox::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset,
     if (m_truncation != cNoTruncation) {
         selectionStart = std::min(selectionStart, static_cast<unsigned>(m_truncation));
         selectionEnd = std::min(selectionEnd, static_cast<unsigned>(m_truncation));
-        length = m_truncation;
     }
 
     float emphasisMarkOffset = 0;