Pages with accelerated compositing do not redraw unless scrolled

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Chris Marrin]

* STEPS TO REPRODUCE
1. go to http://demos.legendthemes.com/3d/
2. Press F11 for Full Screen
3. Press F11 to leave Full screen

* RESULTS
Page will not repaint correctly unless scrolled.

<rdar://problem/9242845>

[Chris Marrin]

Created attachment 91777 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 91777 [details]
Patch

Why does Windows need this change but not Mac?

[Chris Marrin]

(In reply to comment #2)
> (From update of attachment 91777 [details])
> Why does Windows need this change but not Mac?

I thought about that. I haven't tried debugging this sequence of calls on Mac. But I'm assuming that there is some Mac specific code that does a setNeedsDIsplay() later in time. Or it could be that CA automatically dirties the layer when changing size and CACF doesn't. I don't think doing a setNeedsDisplay here will hurt Mac. But I could always rejigger the code a bit so this is only done on Windows.

[Simon Fraser (smfr)]

Comment on attachment 91777 [details]
Patch

I actually think we need it (see Radar comments).

[Simon Fraser (smfr)]

Comment on attachment 91777 [details]
Patch

Actually, I think you should only repaint if it got larger, and then only the rects that were revealed.

[Chris Marrin]

Created attachment 91956 [details]
Patch

[Chris Marrin]

Landed in http://trac.webkit.org/changeset/85508