Update text markup annotation expected results to use /QuadPoints when appropriate
This CL is from Antonio Gomes.
In [1], PDFium is being updated to use /QuadPoints
coordinates rather than /Rect coordinates for drawing
text markup annotations, in accordance to the presence of /AP.
This CL updates the expected results for some test cases.
Notice that for the test cases updated here, the difference
is in pixel-level, and not able to see by naked eyes.
[1] https://codereview.chromium.org/2289293005/
BUG=pdfium:585
Committed: https://pdfium.googlesource.com/pdfium_tests/+/e86a1bac556194ee572b0cd37d04ea646c7b5320
4 years, 3 months ago
(2016-09-15 20:20:03 UTC)
#2
PTAL.
tonikitoo
On 2016/09/15 20:20:03, dsinclair wrote: > PTAL. lgtm
4 years, 3 months ago
(2016-09-15 20:23:22 UTC)
#3
On 2016/09/15 20:20:03, dsinclair wrote:
> PTAL.
lgtm
dsinclair
Description was changed from ========== Update text markup annotation expected results to use /QuadPoints when ...
4 years, 3 months ago
(2016-09-15 20:23:48 UTC)
#4
Description was changed from
==========
Update text markup annotation expected results to use /QuadPoints when
appropriate
This CL is from Antonio Gomes.
In [1], PDFium is being updated to use /QuadPoints
coordinates rather than /Rect coordinates for drawing
text markup annotations, in accordance to the presence of /AP.
This CL updates the expected results for some test cases.
Notice that for the test cases updated here, the difference
is in pixel-level, and not able to see by naked eyes.
[1] https://codereview.chromium.org/2289293005/
BUG=pdfium:585
==========
to
==========
Update text markup annotation expected results to use /QuadPoints when
appropriate
This CL is from Antonio Gomes.
In [1], PDFium is being updated to use /QuadPoints
coordinates rather than /Rect coordinates for drawing
text markup annotations, in accordance to the presence of /AP.
This CL updates the expected results for some test cases.
Notice that for the test cases updated here, the difference
is in pixel-level, and not able to see by naked eyes.
[1] https://codereview.chromium.org/2289293005/
BUG=pdfium:585
Committed:
https://pdfium.googlesource.com/pdfium_tests/+/e86a1bac556194ee572b0cd37d04ea...
==========
dsinclair
Committed patchset #1 (id:1) manually as e86a1bac556194ee572b0cd37d04ea646c7b5320 (presubmit successful).
4 years, 3 months ago
(2016-09-15 20:23:50 UTC)
#5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
e86a1bac556194ee572b0cd37d04ea646c7b5320 (presubmit successful).
Issue 2340933003: Update text markup annotation expected results to use /QuadPoints when appropriate
(Closed)
Created 4 years, 3 months ago by dsinclair
Modified 4 years, 3 months ago
Reviewers: tonikitoo_gmail.com
Base URL:
Comments: 0