Skip to content
Permalink
Browse files
Align BitmapImage::LargeAnimationCutoff to a megabyte value
https://bugs.webkit.org/show_bug.cgi?id=177924

Reviewed by Carlos Garcia Campos.

* platform/graphics/BitmapImage.h: Fix the 1014 multiplier used to calculate
the LargeAnimationCutoff value. It was most likely a typo.


Canonical link: https://commits.webkit.org/194181@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@222906 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
zdobersek committed Oct 5, 2017
1 parent 747b0da commit f8c9546fdce431161f6f19b4fb95d699a2eeba9a
Showing with 11 additions and 1 deletion.
  1. +10 −0 Source/WebCore/ChangeLog
  2. +1 −1 Source/WebCore/platform/graphics/BitmapImage.h
@@ -1,3 +1,13 @@
2017-10-05 Zan Dobersek <zdobersek@igalia.com>

Align BitmapImage::LargeAnimationCutoff to a megabyte value
https://bugs.webkit.org/show_bug.cgi?id=177924

Reviewed by Carlos Garcia Campos.

* platform/graphics/BitmapImage.h: Fix the 1014 multiplier used to calculate
the LargeAnimationCutoff value. It was most likely a typo.

2017-10-05 Wenson Hsieh <wenson_hsieh@apple.com>

Address post-review feedback following r222885
@@ -202,7 +202,7 @@ class BitmapImage final : public Image {
void dump(WTF::TextStream&) const override;

// Animated images over a certain size are considered large enough that we'll only hang on to one frame at a time.
static const unsigned LargeAnimationCutoff = 30 * 1014 * 1024;
static const unsigned LargeAnimationCutoff = 30 * 1024 * 1024;

mutable ImageSource m_source;

0 comments on commit f8c9546

Please sign in to comment.