Bug 6473 – REGRESSION: Serious painting problems on new iLife and iWorks pages

• Component: Layout and Rendering
• Status: VERIFIED FIXED

Upon loading http://www.apple.com/ilife/iphoto/ or http://www.apple.com/iwork/pages/ they are 
displayed incorrectly, with the main content offset up and clipped by the top tabs. Doing Select All causes 
the content to draw on top of the tabs. The "Learn More" and "Visit an Apple Store" buttons are also 
broken.

It seems that with the right combination of scrolling and highlighting the entire page can be made to 
repaint correctly, so this is purely a repaint (rather than layout) issue.

This might be a dupe of bug 3509 or bug 6388 or both, or some other regression from the repaint 
optimization from bug 5633.

Created attachment 5609 [details]
Screenshot

[Alice Liu]

<rdar://problem/4405789>

I changed my mind about this being purely a painting issue: those "Learn More" links at the bottom don't 
move until you hover above them.

Created attachment 5643 [details]
Testcase

This testcase demonstrates the bug that makes the iLife pages break: changing
the opacity from <1 to 1 creates layout and rendering problems (supposedly
because of layer positioning).

However, the latter bug is present also in release Safari. The iLife pages work
around it by setting the opacity property to (100/101) at most. The problem is
that they don't use the same workaround for the mozOpacity property, which in
TOT maps to opacity (and in release Safari apparently doesn't).

Created attachment 5646 [details]
Update layer positions when a layer is pulled from the middle

[Dave Hyatt]

Comment on attachment 5646 [details]
Update layer positions when a layer is pulled from the middle

This bug has convinced me that we should go further and also drop support for
the -moz properties.  I think this will just get us in trouble in the future.

Can you go one step further and remove the code that is mapping -moz to -khtml?

Created attachment 5648 [details]
Update layer positions when a layer is pulled from the middle

Revised patch, also undoes the -moz-opacity -> opacity and -moz-border-radius
-> border-radius mappings.

[Dave Hyatt]

Comment on attachment 5648 [details]
Update layer positions when a layer is pulled from the middle

r=me

Looks like Apple changed the iPhoto page since the bug was filed, but I verified the fix using the Pages 
page http://www.apple.com/iwork/pages/ and the testcase.

[Joost de Valk (AlthA)]

Removing keyword(s) since bug is fixed.