Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f0d8991bbfe39c094e6e8662db8f7da51cc27bc7
      
https://github.com/WebKit/WebKit/commit/f0d8991bbfe39c094e6e8662db8f7da51cc27bc7
  Author: Thomas Devoogdt <[email protected]>
  Date:   2025-09-18 (Thu, 18 Sep 2025)

  Changed paths:
    M Source/WebCore/dom/DocumentFullscreen.cpp
    M Source/WebCore/inspector/agents/InspectorDOMAgent.cpp
    M Source/WebCore/page/Page.cpp
    M Source/WebCore/platform/MediaStrategy.h
    M Source/WebCore/rendering/RenderReplaced.cpp

  Log Message:
  -----------
  WebCore: fix compilation if ENABLE(VIDEO) is not set

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

Reviewed by Michael Catanzaro.

Source/WebCore/dom/DocumentFullscreen.cpp: In member function ‘void 
WebCore::DocumentFullscreen::requestFullscreen(WTF::Ref<WebCore::Element>&&, 
FullscreenCheckType, 
WTF::CompletionHandler<void(WebCore::ExceptionOr<void>)>&&, 
WebCore::MediaPlayerEnums::VideoFullscreenMode)’:
Source/WebCore/dom/DocumentFullscreen.cpp:156:40: error: invalid use of 
incomplete type ‘class WebCore::HTMLVideoElement’
  156 |             CheckedPtr renderer = video->renderer();
      |                                        ^~

Source/WebCore/inspector/agents/InspectorDOMAgent.cpp:3171:90: error: 
‘VideoProjectionMetadataKind’ was not declared in this scope; did you mean 
‘Inspector::Protocol::DOM::VideoProjectionMetadataKind’?
 3171 | static Inspector::Protocol::DOM::VideoProjectionMetadataKind 
videoProjectionMetadataKind(VideoProjectionMetadataKind kind)
      |                                                                         
                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                                                         
                 Inspector::Protocol::DOM::VideoProjectionMetadataKind

Source/WebCore/platform/MediaStrategy.h:73:13: error: no declaration matches 
‘void WebCore::MediaStrategy::nativeImageFromVideoFrame(const 
WebCore::VideoFrame&, 
WTF::CompletionHandler<void(std::optional<WTF::RefPtr<WebCore::NativeImage> 
>&&)>&&)’
   73 | inline void MediaStrategy::nativeImageFromVideoFrame(const VideoFrame&, 
CompletionHandler<void(std::optional<RefPtr<NativeImage>>&&)>&& 
completionHandler)
      |             ^~~~~~~~~~~~~

Source/WebCore/page/Page.cpp:3105:18: error: ‘class WebCore::Document’ has no 
member named ‘shouldSuppressHDRDidChange’
 3105 |         document.shouldSuppressHDRDidChange();
      |         ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~

Source/WebCore/rendering/RenderReplaced.cpp: In function ‘bool 
WebCore::shouldRepaintOnSizeChange(RenderReplaced&)’:
Source/WebCore/rendering/RenderReplaced.cpp:123:88: error: ‘RenderMedia’ was 
not declared in this scope; did you mean ‘RenderMeter’?
  123 |     if (auto* renderImage = dynamicDowncast<RenderImage>(renderer); 
renderImage && !is<RenderMedia>(*renderImage) && 
!renderImage->isShowingMissingOrImageError())
      |                                                                         
               ^~~~~~~~~~~
      |                                                                         
               RenderMeter

Signed-off-by: Thomas Devoogdt <[email protected]>
Canonical link: https://commits.webkit.org/300185@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to