Summary: | Implement atan, acos, asin, atan2 calc functions | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikos Mouchtaris <nmouchtaris> | ||||||||||||||||||
Component: | New Bugs | Assignee: | Nikos Mouchtaris <nmouchtaris> | ||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||
Severity: | Normal | CC: | clopez, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, jonlee, macpherson, menard, simon.fraser, webkit-bug-importer, youennf | ||||||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||||||
OS: | Unspecified | ||||||||||||||||||||
Attachments: |
|
Description
Nikos Mouchtaris
2021-09-01 16:06:28 PDT
Created attachment 437076 [details]
Patch
Created attachment 438279 [details]
Patch
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess Created attachment 438298 [details]
Patch
Created attachment 438299 [details]
Patch
Comment on attachment 438299 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438299&action=review > Source/WebCore/ChangeLog:8 > + Reviewed by NOBODY (OOPS!). > + > + Tests: imported/w3c/web-platform-tests/css/css-values/acos-asin-atan-atan2-computed.html Words here please (like a spec link etc). > Source/WebCore/css/calc/CSSCalcOperationNode.cpp:490 > + > Too many blank lines. > Source/WebCore/css/calc/CSSCalcOperationNode.cpp:-441 > - return; Is this removal significant? > Source/WebCore/css/calc/CSSCalcOperationNode.cpp:630 > + auto newChild = CSSCalcPrimitiveValueNode::create(CSSPrimitiveValue::create(resolvedValue, CSSUnitType::CSS_DEG)); Is it CSS_DEG because the spec says that degrees are the canonical angle unit? > Source/WebCore/css/calc/CSSCalcOperationNode.cpp:1133 > + return rad2deg(std::asin(children[0])); Maybe link to the part of the spec that explains why returning angles in degrees is correct. > Source/WebCore/css/calc/CSSCalcOperationNode.cpp:1138 > + return rad2deg(std::asin(children[0])); acts, not asin? Why did tests not catch this. > Source/WebCore/css/calc/CSSCalcOperationNode.cpp:1143 > + return rad2deg(std::asin(children[0])); asin -> atan? Again, where is the test coverage. Created attachment 438772 [details]
asin,acos,atan,atan2
Created attachment 438846 [details]
inverse trig functions
Created attachment 438855 [details]
inverse trig functions
Created attachment 438863 [details]
inverse trig functions
Committed r283013 (242078@main): <https://commits.webkit.org/242078@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 438863 [details]. |