[Webkit-unassigned] [Bug 65084] Build without video but fullscreen API is broken

Sun Jul 24 17:34:18 PDT 2011

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

Darin Adler <darin at apple.com> changed:

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

--- Comment #1 from Darin Adler <darin at apple.com>  2011-07-24 17:34:18 PST ---
(From update of attachment 101833)
View in context: https://bugs.webkit.org/attachment.cgi?id=101833&action=review

> Source/WebCore/dom/Element.h:315
> -#if ENABLE(VIDEO)
> +#if ENABLE(VIDEO) || ENABLE(FULLSCREEN_API)

This is the wrong fix. The right fix is to put the code in EventDispatcher::determineDispatchBehavior inside this:

    #if ENABLE(FULLSCREEN_API) && ENABLE(VIDEO)

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