[Webkit-unassigned] [Bug 60439] [Texmap][Qt] Upstream texture-mapper changes from Qt's WebKit2 branch

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=60439





--- Comment #33 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2011-05-08 09:21:13 PST ---
(From update of attachment 92715)
View in context: https://bugs.webkit.org/attachment.cgi?id=92715&action=review

> Source/WebCore/platform/graphics/texmap/TextureMapperNode.h:133
> +    void setOpacity(float o) { m_opacity = o; }

I prefer value over o

> Source/WebCore/platform/graphics/texmap/TextureMapperNode.h:184
> +        TransformationMatrix target, replica, forDescendants, local, base, perspective;

We normally would declare these on separate lines

> Source/WebCore/platform/graphics/texmap/TextureMapperNode.h:321
> +inline void deleteOwnedPtr(TextureMapperNode* ptr)
> +{
> +    delete ptr;
> +}

Huh? why is this necessary?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.