[webkit-reviews] review denied: [Bug 133916] Implement DOMPoint/DOMPointReadOnly : [Attachment 233124] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Jun 14 14:53:11 PDT 2014


Benjamin Poulain <benjamin at webkit.org> has denied Dirk Schulze
<krit at webkit.org>'s request for review:
Bug 133916: Implement DOMPoint/DOMPointReadOnly
https://bugs.webkit.org/show_bug.cgi?id=133916

Attachment 233124: Patch
https://bugs.webkit.org/attachment.cgi?id=233124&action=review

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
I am strongly against this. The binding overhead is an order of magnitude
larger than the cost of the operations.

With FTL, the compiler has huge opportunities for inlining and vectorizing this
kind of operations. Denying this would be shooting ourself in the foot with our
biggest gun.


More information about the webkit-reviews mailing list