[Webkit-unassigned] [Bug 53390] Files missing ENABLE_VIDEO #ifdef

[bugzilla-daemon at webkit.org]

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


David Levin <levin at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #81738|review?                     |review-
               Flag|                            |




--- Comment #4 from David Levin <levin at chromium.org>  2011-02-10 17:55:40 PST ---
(From update of attachment 81738)
View in context: https://bugs.webkit.org/attachment.cgi?id=81738&action=review

> Source/WebCore/html/canvas/WebGLRenderingContext.cpp:2988
>  PassRefPtr<Image> WebGLRenderingContext::videoFrameToImage(HTMLVideoElement* video)

Can't you put the #if ENABLE(VIDEO) on the outside of this method? (In the header too.)

Ditto for the other instances in this file.

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