commit 94f7511a291821beaa8a3faa80a1c8a9194bd12f
author Tom Sepez <tsepez@chromium.org> Thu Jan 30 20:51:14 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Jan 30 20:51:14 2020 +0000
tree 0601e4e1ec57a63dad47149a75a7a46c7de2c09b
parent 0cd1a39afe9c07a0cbaa766e1355034a6b43d8cc

Remove unused CPDF_TextObject::SetPosition() variant.

This is always called with a point as an argument rather than with
two discreet floats.

-- use const-ref arg in one-arg form while we're at it.

Change-Id: Idb5d00564949f937e4cada96a39839efa7b429c7
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/65835
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

core/fpdfapi/page/cpdf_textobject.cpp

diff --git a/core/fpdfapi/page/cpdf_textobject.cpp b/core/fpdfapi/page/cpdf_textobject.cpp
index afae35b..077e5ed 100644
--- a/core/fpdfapi/page/cpdf_textobject.cpp
+++ b/core/fpdfapi/page/cpdf_textobject.cpp
@@ -343,14 +343,6 @@
   return ret;
 }
 
-void CPDF_TextObject::SetPosition(float x, float y) {
-  float dx = x - m_Pos.x;
-  float dy = y - m_Pos.y;
-  m_Pos.x = x;
-  m_Pos.y = y;
-  m_Rect.Translate(dx, dy);
-}
-
 void CPDF_TextObject::RecalcPositionData() {
   CalcPositionData(1);
 }

core/fpdfapi/page/cpdf_textobject.h

diff --git a/core/fpdfapi/page/cpdf_textobject.h b/core/fpdfapi/page/cpdf_textobject.h
index 1717256..ad9918e 100644
--- a/core/fpdfapi/page/cpdf_textobject.h
+++ b/core/fpdfapi/page/cpdf_textobject.h
@@ -58,8 +58,7 @@
   TextRenderingMode GetTextRenderMode() const;
 
   void SetText(const ByteString& str);
-  void SetPosition(CFX_PointF pos) { m_Pos = pos; }
-  void SetPosition(float x, float y);
+  void SetPosition(const CFX_PointF& pos) { m_Pos = pos; }
 
   void RecalcPositionData();