WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
265149
Use 'rotateRadians' in CanvasRenderingContext2DBase.cpp
https://bugs.webkit.org/show_bug.cgi?id=265149
Summary
Use 'rotateRadians' in CanvasRenderingContext2DBase.cpp
Ahmad Saleem
Reported
2023-11-20 09:53:12 PST
Hi Team, We merged Blink's patch to have 'rotateRadians' rather than multiple conversions, we can use it here as well:
https://searchfox.org/wubkat/source/Source/WebCore/html/canvas/CanvasRenderingContext2DBase.cpp#784
^ From 'newTransform.rotate(angleInRadians / piDouble * 180.0);' to 'newTransform.rotateRadians(angleInRadians);'
https://searchfox.org/wubkat/source/Source/WebCore/html/canvas/CanvasRenderingContext2DBase.cpp#792
^ From 'm_path.transform(AffineTransform().rotate(-angleInRadians / piDouble * 180.0));' to 'm_path.transform(AffineTransform().rotateRadians(-angleInRadians));' It is just to use the function more and more. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
EWS
Comment 1
2023-11-20 17:53:53 PST
Committed
270994@main
(1df0b1914071): <
https://commits.webkit.org/270994@main
> Reviewed commits have been landed. Closing PR #20754 and removing active labels.
Radar WebKit Bug Importer
Comment 2
2023-11-20 17:54:14 PST
<
rdar://problem/118666924
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug