[webkit-reviews] review denied: [Bug 100042] [chromium] Add textureWidth() and textureHeight() to WebVideoFrame : [Attachment 170005] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 22 15:50:26 PDT 2012


James Robinson <jamesr at chromium.org> has denied John Sheu <sheu at chromium.org>'s
request for review:
Bug 100042: [chromium] Add textureWidth() and textureHeight() to WebVideoFrame
https://bugs.webkit.org/show_bug.cgi?id=100042

Attachment 170005: Patch
https://bugs.webkit.org/attachment.cgi?id=170005&action=review

------- Additional Comments from James Robinson <jamesr at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=170005&action=review


> Source/Platform/ChangeLog:8
> +	   Additional information of the change such as approach, rationale.
Please add per-function descriptions below (OOPS!).

You have to fill something out here.  For this, when would
textureWidth/textureHight be different from the frame's width/height?  Why do
we need separate fields?


More information about the webkit-reviews mailing list