Bug 65084 – Build without video but fullscreen API is broken

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Denis Oliver Kropp]

Created attachment 101833 [details]
Patch fixing missing isMediaElement() when building without video but fullscreen API

Building without video support, but with fullscreen API is currently broken.

The virtual method isMediaElement() is not defined in that case.

Attached patch fixes this.

[Darin Adler]

Comment on attachment 101833 [details]
Patch fixing missing isMediaElement() when building without video but fullscreen API

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)

[Evgeny Burmentyev]

I build nightlies with --disable-video --disable-video-track, and I get the same error:

Source/WebCore/dom/EventDispatcher.cpp:387:22: error: ‘class WebCore::Element’ has no member named ‘isMediaElement’

...thus confirming the bug.

[Denis Oliver Kropp]

Created attachment 102132 [details]
Proper fix for the bug.

[Philippe Normand]

Comment on attachment 102132 [details]
Proper fix for the bug.

This looks good but lacks a ChangeLog entry. Please see http://www.webkit.org/coding/contributing.html :)

[Denis Oliver Kropp]

Created attachment 102133 [details]
Proper fix with ChangeLog entry.

[Philippe Normand]

Comment on attachment 102133 [details]
Proper fix with ChangeLog entry.

View in context: https://bugs.webkit.org/attachment.cgi?id=102133&action=review

Almost there, thanks again for the patch!

> Source/WebCore/ChangeLog:3
> +        This fixes build without video enabled. Bug 65084.

You need to have 2 lines, one for the bug description and the second line containing the link to the bug.
prepare-ChangeLog --bug 65084 can be used too :)

> Source/WebCore/dom/EventDispatcher.cpp:9
> + * Copyright (C) 2011 Denis Oliver Kropp ([e-mail])

According to our contributor policy only substantive code changes allow a copyright update. This patch is quite small, so please remove this line.

[Denis Oliver Kropp]

Created attachment 102245 [details]
Final bug fix

[Philippe Normand]

Comment on attachment 102245 [details]
Final bug fix

Thanks!

[WebKit Review Bot]

Comment on attachment 102245 [details]
Final bug fix

Clearing flags on attachment: 102245

Committed r91923: <http://trac.webkit.org/changeset/91923>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.