Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 097123389da2454304dbbd9e85129eae13ea307c
      
https://github.com/WebKit/WebKit/commit/097123389da2454304dbbd9e85129eae13ea307c
  Author: Don Olmstead <[email protected]>
  Date:   2022-11-09 (Wed, 09 Nov 2022)

  Changed paths:
    M Source/WebCore/PlatformWin.cmake
    M Source/WebCore/PlatformWinCairo.cmake
    M Source/WebCore/html/canvas/WebGL2RenderingContext.cpp
    M Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp
    M Source/WebCore/platform/graphics/win/FullScreenController.cpp
    M Source/WebCore/platform/graphics/win/FullScreenController.h
    M Source/WebCore/platform/graphics/win/FullScreenControllerClient.h
    A Source/WebCore/platform/graphics/win/FullScreenWindow.cpp
    A Source/WebCore/platform/graphics/win/FullScreenWindow.h
    R 
Source/WebCore/platform/graphics/win/MediaPlayerPrivateFullscreenWindow.cpp
    R Source/WebCore/platform/graphics/win/MediaPlayerPrivateFullscreenWindow.h
    M Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp
    M Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.h
    M Source/WebCore/rendering/RenderReplaced.cpp
    M Source/WebKit/PlatformWin.cmake
    M Source/WebKitLegacy/PlatformWin.cmake
    M Source/WebKitLegacy/win/FullscreenVideoController.cpp
    M Source/WebKitLegacy/win/FullscreenVideoController.h
    M Source/cmake/OptionsWinCairo.cmake

  Log Message:
  -----------
  Cleanup Windows MediaPlayer code
https://bugs.webkit.org/show_bug.cgi?id=247654

Reviewed by Fujii Hironori.

The code in `MediaPlayerPrivateFullscreenWindow` wasn't specific to
`ENABLE(VIDEO)` it was specific to `ENABLE(FULLSCREEN_API)` so modified
it to be more general purpose. Updated references of
`MediaPlayerPrivateFullscreenWindow` to `FullscreenWindow` and
`MediaPlayerPrivateFullscreenClient` to `FullscreenClient` and renamed
the files accordingly.

Fixed the WebCore build of `ENABLE(VIDEO)` set to `OFF`. Modified the
linked libraries to be specific to `ENABLE(VIDEO)` and removed unused
ones. Add guards where appropriate and update style to use
`#pragma once`.

* Source/WebCore/PlatformWin.cmake:
* Source/WebCore/PlatformWinCairo.cmake:
* Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:
* Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp:
* Source/WebCore/platform/graphics/win/FullScreenController.cpp:
* Source/WebCore/platform/graphics/win/FullScreenController.h:
* Source/WebCore/platform/graphics/win/FullScreenControllerClient.h:
* Source/WebCore/platform/graphics/win/FullscreenWindow.cpp: Renamed from 
Source\WebCore\platform\graphics\win\MediaPlayerPrivateFullscreenWindow.cpp.
* Source/WebCore/platform/graphics/win/FullscreenWindow.h: Renamed from 
Source\WebCore\platform\graphics\win\MediaPlayerPrivateFullscreenWindow.h.
* Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:
* Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.h:
* Source/WebCore/rendering/RenderReplaced.cpp:
* Source/WebKitLegacy/PlatformWin.cmake:
* Source/WebKitLegacy/win/FullscreenVideoController.cpp:
* Source/WebKitLegacy/win/FullscreenVideoController.h:
* Source/WebKit/PlatformWin.cmake:
* Source/cmake/OptionsWinCairo.cmake:

Canonical link: https://commits.webkit.org/256492@main


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to