Hi Team, Just going through Blink Commits, came across following commit: Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=174294 and Webkit is still using wrong one, I think matching would be fine rather than updating comment to reflect current value. https://github.com/WebKit/WebKit/blob/06829a72816b3507674af7be8a740a4964ae3518/Source/WebCore/platform/Decimal.cpp#L51 I am just creating this bug to do PR and merge above commit. Let me know if someone want me to just update comment. Thanks!
ch.dumez@samsung.com! Sadly, no test modified in the commit.
(In reply to Alexey Proskuryakov from comment #1) > ch.dumez@samsung.com! Sadly, no test modified in the commit. https://github.com/WebKit/WebKit/pull/6160
(In reply to Ahmad Saleem from comment #2) > (In reply to Alexey Proskuryakov from comment #1) > > ch.dumez@samsung.com! Sadly, no test modified in the commit. > > https://github.com/WebKit/WebKit/pull/6160 Latest position is that whether there is std:decimal to replace our use case or whether we just add 'DecmialTest.cpp' API for testing similar to Chrome.
(In reply to Alexey Proskuryakov from comment #1) > ch.dumez@samsung.com! Sadly, no test modified in the commit. There were tests modified in MY commit :)
<rdar://problem/102261978>
Will fix via https://github.com/WebKit/WebKit/pull/7908 in one go with test-case as well.
*** This bug has been marked as a duplicate of bug 248784 ***