[webkit-reviews] review requested: [Bug 18512] getPresentationAttribute returns a shared object : [Attachment 20694] First attempt

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Apr 20 00:54:19 PDT 2008


Rob Buis <rwlbuis at gmail.com> has asked	for review:
Bug 18512: getPresentationAttribute returns a shared object
http://bugs.webkit.org/show_bug.cgi?id=18512

Attachment 20694: First attempt
http://bugs.webkit.org/attachment.cgi?id=20694&action=edit

------- Additional Comments from Rob Buis <rwlbuis at gmail.com>
This small patch fixes the problem. However I wonder if there is a nicer way to
do it. I guess there is nothing wrong with sharing the style declaration for
this attribute though, since in the testcase the values are equal and we do not
know up front that we are going to use getPresentationAttribute, which is used
very rarely anyway. So I really can't see another way, though maybe the
unsharing can be done with less statements or some helper method in
StyledElement. I'll do the testcase later, I think the original testcase in the
bug is a good base for an automated test.
Cheers,

Rob.


More information about the webkit-reviews mailing list