[webkit-reviews] review granted: [Bug 189506] Add missing #if ENABLE(VIDEO) WebProcess/FullScreen/WebFullScreenManager.cpp : [Attachment 349412] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 11 11:00:29 PDT 2018


Tim Horton <thorton at apple.com> has granted Pablo Saavedra
<psaavedra at igalia.com>'s request for review:
Bug 189506: Add missing #if ENABLE(VIDEO)
WebProcess/FullScreen/WebFullScreenManager.cpp
https://bugs.webkit.org/show_bug.cgi?id=189506

Attachment 349412: patch

https://bugs.webkit.org/attachment.cgi?id=349412&action=review




--- Comment #2 from Tim Horton <thorton at apple.com> ---
Comment on attachment 349412
  --> https://bugs.webkit.org/attachment.cgi?id=349412
patch

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

> Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.cpp:102
> +#if ENABLE(VIDEO)

!ENABLE(VIDEO) knows what HTMLVideoElement is, though??


More information about the webkit-reviews mailing list