[webkit-reviews] review granted: [Bug 88528] Reduce Node object size from 72 byte to 64 byte : [Attachment 146455] Fixed mac build failure

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 7 22:32:51 PDT 2012


Ryosuke Niwa <rniwa at webkit.org> has granted Kentaro Hara
<haraken at chromium.org>'s request for review:
Bug 88528: Reduce Node object size from 72 byte to 64 byte
https://bugs.webkit.org/show_bug.cgi?id=88528

Attachment 146455: Fixed mac build failure
https://bugs.webkit.org/attachment.cgi?id=146455&action=review

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
We had a length discussion with andersca & hareken about this but we concluded
that it's probably okay to land the patch as is. We can always do a follow up
cleanup when someone more familiar with C++ (andersca, darin, eseidel,
othermaciej, etc...) comes up with a better solution. For now, let us enjoy
4/8-byte memory savings on all DOM nodes.


More information about the webkit-reviews mailing list