[webkit-changes] [WebKit/WebKit] fab9c4: REGRESSION(276827 at main): Large animated images may...

Said Abou-Hallawa noreply at github.com
Tue May 21 17:37:37 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: fab9c4ba90cca1121a134d5f62c40ea28f9ff6f0
      https://github.com/WebKit/WebKit/commit/fab9c4ba90cca1121a134d5f62c40ea28f9ff6f0
  Author: Said Abou-Hallawa <said at apple.com>
  Date:   2024-05-21 (Tue, 21 May 2024)

  Changed paths:
    M Source/WebCore/platform/graphics/BitmapImageSource.cpp

  Log Message:
  -----------
  REGRESSION(276827 at main): Large animated images may not animate all the frames
https://bugs.webkit.org/show_bug.cgi?id=274478
rdar://127697287

Reviewed by Simon Fraser.

We used to postpone animating the image till loading the image is complete. The
number of frames can only be known for sure when the loading the encoded data is
complete. If we start animating before we get all the data, we may end up
animating a small set of the image frames.

The fix is to postpone animating the animated image till its encoded data is
completely loaded.

* Source/WebCore/platform/graphics/BitmapImageSource.cpp:
(WebCore::BitmapImageSource::frameAnimator const):

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list