Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(100)

Issue 1861963002: Fix an infinite loop in agg's stroke_calc_arc(). (Closed)

Created:
4 years, 8 months ago by Lei Zhang
Modified:
4 years, 8 months ago
Reviewers:
Tom Sepez
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

Fix an infinite loop in agg's stroke_calc_arc(). BUG=pdfium:466 Committed: https://pdfium.googlesource.com/pdfium/+/8833342afc225214ce7180db612f104c9880f2c8

Patch Set 1 #

Patch Set 2 : clang-format is confused #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+71 lines, -10 lines) Patch
A third_party/agg23/0000-bug-466.patch View 1 2 1 chunk +45 lines, -0 lines 0 comments Download
A third_party/agg23/README.pdfium View 1 2 1 chunk +14 lines, -0 lines 0 comments Download
M third_party/agg23/agg_math_stroke.h View 1 1 chunk +12 lines, -10 lines 0 comments Download

Messages

Total messages: 10 (3 generated)
Lei Zhang
4 years, 8 months ago (2016-04-06 02:32:48 UTC) #2
Tom Sepez
Change is fine, but what about upstream? Bug filed there? Do we also want a ...
4 years, 8 months ago (2016-04-06 17:07:08 UTC) #3
Lei Zhang
On 2016/04/06 17:07:08, Tom Sepez wrote: > Change is fine, but what about upstream? Bug ...
4 years, 8 months ago (2016-04-07 00:02:17 UTC) #4
Lei Zhang
I added a README.pdfium and a .patch file.
4 years, 8 months ago (2016-04-07 00:08:48 UTC) #5
Tom Sepez
lgtm
4 years, 8 months ago (2016-04-07 04:59:03 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1861963002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1861963002/40001
4 years, 8 months ago (2016-04-07 06:44:32 UTC) #8
commit-bot: I haz the power
4 years, 8 months ago (2016-04-07 06:57:36 UTC) #10
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://pdfium.googlesource.com/pdfium/+/8833342afc225214ce7180db612f104c9880...

Powered by Google App Engine
This is Rietveld 408576698