[webkit-reviews] review requested: [Bug 87510] [chromium] create WebTransformOperation interface for chromium platform : [Attachment 144355] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 28 07:19:00 PDT 2012
vollick at chromium.org has asked for review:
Bug 87510: [chromium] create WebTransformOperation interface for chromium
platform
https://bugs.webkit.org/show_bug.cgi?id=87510
Attachment 144355: Patch
https://bugs.webkit.org/attachment.cgi?id=144355&action=review
------- Additional Comments from vollick at chromium.org
(In reply to comment #5)
> (From update of attachment 144123 [details])
> View in context:
https://bugs.webkit.org/attachment.cgi?id=144123&action=review
>
> Needs a touch more indirection but the API looks quite nice.
>
> > Source/Platform/chromium/public/WebTransformOperations.h:30
> > +#include <wtf/Vector.h>
>
> this has to be hidden - users in the chromium repo don't have WTF/ on their
include path so this won't work.
Done.
>
> One option is to have a WebTransformOperationsPrivate that is forward
declared here and held via a WebPrivateOwnPtr and have that class' impl hold a
WTF::Vector
>
> > Source/Platform/chromium/public/WebTransformOperations.h:51
> > + WebTransformationMatrix apply() const;
>
> please add WEBKIT_EXPORT for functions declared here but defined in the .cpp
Done.
More information about the webkit-reviews
mailing list