[Webkit-unassigned] [Bug 66597] [skia] BitmapImageSingleFrameSkia returns true to isBitmapImage() but does not inherit from BitmapImage

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from James Robinson <jamesr at chromium.org>  2011-08-19 17:30:52 PST ---
The original motivation for BitmapImageSingleFrameSkia is to avoid having all of the animation overhead for simple 1-frame images, which seems like a useful optimization for all ports.  I think the option 3 from https://bugs.webkit.org/show_bug.cgi?id=65063#c18 is the best way to go here - we should turn BitmapImage into a simple class that doesn't handle animation concerns, and use an AnimatedBitmapImage subclass to handle the animation case.  This should have the nice benefit of reducing the memory use for non-animated images for all platforms and clean up the class hierarchy a lot for the skia case.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.