Bug 15890 – Most of www.aol.com still redraws unnecessarily when headline/photo section changes

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[David Kilzer (:ddkilzer)]

Either the reduction in Bug 15015 was bogus, or AOL changed their web site again.

Quartz Debug shows that everything below the top banner is still redrawn with a local debug build of WebKit r27581 on Tiger with Safari 3 Public Beta.

I cannot reproduce on Leopard with TOT.

(In reply to comment #1)
> I cannot reproduce on Leopard with TOT.
nor with the r27582 nightly build.

I can reproduce on Tiger 10.4.10 with the same nightly build and Safari 2.0.4.

(In reply to comment #2)
> (In reply to comment #1)
> > I cannot reproduce on Leopard with TOT.
> nor with the r27582 nightly build.
not even when spoofing as Safari 2.0.4, so I don't think the page is different between Leopard and Tiger.

[David Kilzer (:ddkilzer)]

(In reply to comment #2)
> (In reply to comment #1)
> > I cannot reproduce on Leopard with TOT.
> nor with the r27582 nightly build.
> 
> I can reproduce on Tiger 10.4.10 with the same nightly build and Safari 2.0.4.

Could this be a CG issue?

(In reply to comment #4)
> Could this be a CG issue?

Could be that or AppKit, but I should probably set up a Tiger dev environment just to make sure that WebKit invalidates the same regions as it does in Leopard.

[David Kilzer (:ddkilzer)]

(In reply to comment #1)
> I cannot reproduce on Leopard with TOT.

I still see most of the page flash with Quartz Debug (red identical updates) with r27585 on Leopard 10.5 (9A581).  I'm on a PowerPC Quad G5 with an NVIDIA GeForce 6600 and an Apple 23" Cinema Display , though.

[David Kilzer (:ddkilzer)]

(In reply to comment #6)
> (In reply to comment #1)
> > I cannot reproduce on Leopard with TOT.
> I still see most of the page flash with Quartz Debug (red identical updates)
> with r27585 on Leopard 10.5 (9A581).  I'm on a PowerPC Quad G5 with an NVIDIA
> GeForce 6600 and an Apple 23" Cinema Display , though.

Most of the page only flashes when the first 5 news items initially load.  After they're done loading, much less of the page repaints after that.

Not sure if there is anything that can be done when the first 5 news items are displayed; may just close this as WONTFIX.

(In reply to comment #7)
> Most of the page only flashes when the first 5 news items initially load. 
> After they're done loading, much less of the page repaints after that.

Looking for things that trigger repainting outside the rotating story box, I found a div (id="itrk") with visibility:hidden; position: absolute; bottom: 0 into which 1x1 images are loaded. Using Drosera I set a breakpoint in at main.js:246 (in imp()), and indeed one such image is created for each story when it is loaded. This is consistent with the excessive repainting only during the first cycle.
It is kind of silly to repaint invisible objects, but we do it because they may have visible descendants. We keep track of whether that is the case only at the layer level.

Created attachment 17210 [details]
Don't repaint invisible objects sometimes

This is a ad-hoc patch that happens to solve the problem with AOL. Not sure the extra checks are useful on any other page.

Comment on attachment 17210 [details]
Don't repaint invisible objects sometimes

Oops, missing a file.

Created attachment 17211 [details]
Don't repaint invisible objects sometimes

This is an ad-hoc patch that happens to solve the problem with AOL. Not sure the extra checks are useful on any other page.

[Darin Adler]

Comment on attachment 17211 [details]
Don't repaint invisible objects sometimes

r=me

Landed in <http://trac.webkit.org/projects/webkit/changeset/27743>.