From 798e18f5e5cfb672c7f3186f6358b84c5ff7785b Mon Sep 17 00:00:00 2001
From: dsinclair <dsinclair@chromium.org>
Date: Wed, 12 Oct 2016 13:05:38 -0700
Subject: Convert from int to float values.

The CPDF_Page::GetDisplayMatrix expects to set float values into the
|display_matrix| but all of the input values are currently int. It is possible
to overflow the int values, so this CL changes the variables to be int which
closer reflects what they're being used for.

BUG=chromium:652038

Review-Url: https://codereview.chromium.org/2412983002
---
 core/fpdfapi/page/cpdf_page.cpp | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

(limited to 'core')

diff --git a/core/fpdfapi/page/cpdf_page.cpp b/core/fpdfapi/page/cpdf_page.cpp
index 89d0b35f3e..2143ddb015 100644
--- a/core/fpdfapi/page/cpdf_page.cpp
+++ b/core/fpdfapi/page/cpdf_page.cpp
@@ -126,12 +126,12 @@ void CPDF_Page::GetDisplayMatrix(CFX_Matrix& matrix,
     return;
   }
   CFX_Matrix display_matrix;
-  int x0 = 0;
-  int y0 = 0;
-  int x1 = 0;
-  int y1 = 0;
-  int x2 = 0;
-  int y2 = 0;
+  float x0 = 0;
+  float y0 = 0;
+  float x1 = 0;
+  float y1 = 0;
+  float x2 = 0;
+  float y2 = 0;
   iRotate %= 4;
   switch (iRotate) {
     case 0:
@@ -167,10 +167,9 @@ void CPDF_Page::GetDisplayMatrix(CFX_Matrix& matrix,
       y2 = yPos;
       break;
   }
-  display_matrix.Set(
-      ((FX_FLOAT)(x2 - x0)) / m_PageWidth, ((FX_FLOAT)(y2 - y0)) / m_PageWidth,
-      ((FX_FLOAT)(x1 - x0)) / m_PageHeight,
-      ((FX_FLOAT)(y1 - y0)) / m_PageHeight, (FX_FLOAT)x0, (FX_FLOAT)y0);
+  display_matrix.Set((x2 - x0) / m_PageWidth, (y2 - y0) / m_PageWidth,
+                     (x1 - x0) / m_PageHeight, (y1 - y0) / m_PageHeight, x0,
+                     y0);
   matrix = m_PageMatrix;
   matrix.Concat(display_matrix);
 }
-- 
cgit v1.2.3