Bug 9388 – REGRESSION: outlines don't get erased when removed in some cases

• Component: New Bugs
• Status: RESOLVED FIXED

[Hanspeter Schaub]

I use the outline: feature a lot in my web pages, and found last week that the outline appears to stick to the page (doesn't disappear as the mouse moves away) in some cases.  This is not a general behavior. I have been able to create a reduced test web page which shows this happening.  What is peculiar is that to trigger this error, the a:hover {text-shadow xxxx} command must be added.  If this command is not present, then the test page behaves as you expect  With this text-shadow command present, the outlines stick to the page once the mouse activates them.

[Hanspeter Schaub]

Created attachment 8796 [details]
Test HTML page showing how the outline is not removed after the mouse-over event.

Created attachment 8798 [details]
A test case that fails in shipping WebKit as well

This bug occurs when a layout style change happens along with the outline becoming narrower. repaintObjectsBeforeLayout doesn't know about the old outline. The "regression" part of this bug is due to text-shadow changes becoming layout changes (in order to fix bug 8352).

[Hanspeter Schaub]

This issues doesn't appear in the 10.4.6 version of Safari?  I've only seen this outline sticking happening with recent nightly builds of safari.

(In reply to comment #3)
> I've only seen this outline sticking happening with recent nightly builds of safari.

Try attachment 8798 [details] in the version that ships with 10.4.6.

[Hanspeter Schaub]

(In reply to comment #4)
> Try attachment 8798 [details] [edit] in the version that ships with 10.4.6.


You're correct.  This attachment does show the same issue in 10.4.6 Safari version.  

Created attachment 9020 [details]
Always epaint with the old style if the outline is shrinking

[Darin Adler]

Comment on attachment 9020 [details]
Always epaint with the old style if the outline is shrinking

r=me

[David Kilzer (:ddkilzer)]

Committed revision 15038.