Closed
Bug 1187210
Opened 11 years ago
Closed 11 years ago
Path2D.arc hang
Categories
(Core :: Graphics: Canvas2D, defect)
Core
Graphics: Canvas2D
Tracking
()
VERIFIED
FIXED
mozilla42
| Tracking | Status | |
|---|---|---|
| firefox42 | --- | verified |
People
(Reporter: jruderman, Assigned: lsalzman)
References
Details
(Keywords: hang, testcase)
Attachments
(2 files, 1 obsolete file)
|
188 bytes,
text/html
|
Details | |
|
4.86 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
Might be related to bug 943587.
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
| Assignee | ||
Comment 1•11 years ago
|
||
Due to floating point inaccuracies, large angles would cause the angle correction step to modify aStartAngle and aEndAngle such that arcSweepLeft still resulted in a huge value, much larger than 2*PI, thus causing ArcToBezier to iterate far more than the expected number of times.
Attachment #8638755 -
Flags: review?(jmuizelaar)
| Assignee | ||
Updated•11 years ago
|
OS: Unspecified → All
Hardware: Unspecified → All
Updated•11 years ago
|
Attachment #8638755 -
Flags: review?(jmuizelaar) → review+
| Assignee | ||
Comment 3•11 years ago
|
||
Fix some mochitest issues with reverse sweeps and Skia canvas precision issues.
Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=47d0b14e7330
Attachment #8638755 -
Attachment is obsolete: true
Attachment #8638885 -
Flags: review+
| Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Comment 5•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Updated•10 years ago
|
Flags: qe-verify+
Comment 7•10 years ago
|
||
Reproduced the initial issue on Win 10 x32, Ubuntu 14.04 x32 and Mac OS X 10.11 using Firefox 40.0.2 (20150812163655)build using the attached testcase. Verified that the issues does not reproduce on Firefox 42.0 Beta 4 (20151005144425) on the same environment.
Status: RESOLVED → VERIFIED
Updated•10 years ago
|
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•