[webkit-changes] [WebKit/WebKit] 5de5fc: REGRESSION(267188 at main): Calls to videoInlineSize(...

aestes noreply at github.com
Mon Aug 28 16:53:43 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5de5fc19e58abc56edc1b957ebb2df2e8b76ff9a
      https://github.com/WebKit/WebKit/commit/5de5fc19e58abc56edc1b957ebb2df2e8b76ff9a
  Author: Andy Estes <aestes at apple.com>
  Date:   2023-08-28 (Mon, 28 Aug 2023)

  Changed paths:
    M Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm
    M Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm

  Log Message:
  -----------
  REGRESSION(267188 at main): Calls to videoInlineSize() were mistakenly replaced with presentationSize()
https://bugs.webkit.org/show_bug.cgi?id=260826
rdar://114591335

Reviewed by Jer Noble.

In the refactoring done in 267188 at main, two calls to MediaPlayer::videoInlineSize() were mistakenly
replaced with calls to MediaPlayer::presentationSize() rather than MediaPlayer::videoLayerSize().
Fixed the unintentional behavior change.

* Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayerLayer):
* Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::ensureLayer):

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




More information about the webkit-changes mailing list