[Webkit-unassigned] [Bug 78406] Move CSSOM wrapper pointer out of StylePropertySet
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Feb 10 23:33:16 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=78406
Andreas Kling <kling at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #126620|review? |review+, commit-queue-
Flag| |
--- Comment #2 from Andreas Kling <kling at webkit.org> 2012-02-10 23:33:15 PST ---
(From update of attachment 126620)
View in context: https://bugs.webkit.org/attachment.cgi?id=126620&action=review
r=me with one fix:
> Source/WebCore/css/StylePropertySet.cpp:49
> +static PropertySetCSSOMWrapperMap* propertySetCSSOMWrapperMap;
This pointer needs to be explicitly initialized.
Also, a common pattern in WebKit is to write these like so:
static PropertySetCSSOMWrapperMap& propertySetCSSOMWrapperMap()
{
DEFINE_STATIC_LOCAL(PropertySetCSSOMWrapperMap, derp, ());
return derp;
}
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list