57484 2011-03-30 11:33:21 -0700 [chromium] Clean up compositor layer dirty state 2013-04-08 11:16:12 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED WONTFIX P2 Normal --- 57113 1 enne enne enne oldest_to_newest 376514 0 enne 2011-03-30 11:33:21 -0700 Here's the state of things: there are currently a number of different ways to track dirtiness on LayerChromium (m_dirtyRect and m_contentsDirty). There's also setNeedsDisplay(FloatRect), setNeedsDisplay(), invalidateRect(FloatRect), and resetNeedsDisplay(). A bunch of layers also skip these functions and touch m_dirtyRect directly, but don't touch m_contentsDirty. On top of that, the tiler that content and image layers will own (a 376517 1 enne 2011-03-30 11:35:05 -0700 (Oops. Clicked on something that sent a half-finished bug-report.) Here's the state of things: there are currently a number of different ways to track dirtiness on LayerChromium (m_dirtyRect and m_contentsDirty). There's also setNeedsDisplay(FloatRect), setNeedsDisplay(), invalidateRect(FloatRect), and resetNeedsDisplay(). A bunch of layers also skip these functions and touch m_dirtyRect directly, but don't touch m_contentsDirty. It also looks like resetNeedsDisplay is never called (but probably should be) and invalidateRect is a no-op virtual that nothing ever overrides. On top of that, the tiler that content and image layers will own (see bug 57113) track their own separate dirty rect. This should get cleaned up and sorted out.