Bug 52518

Summary:

REGRESSION (r69237): Black border around map elements while using an image map on Mac platform

Product:

WebKit

Reporter:

Darth <priyajeet.hora>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aestes, ap, bdakin, cfleizach, dbates, priyajeet.hora

Priority:

Keywords:

HasReduction, InRadar, Regression

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.6

URL:

https://bug-52518-attachments.webkit.org/attachment.cgi?id=82141

Bug Depends on:

45832

Bug Blocks:

Attachments:

Description	Flags
issue image	none
testcase	none
Patch	oliver: review+

Description Darth 2011-01-15 12:56:15 PST

Created attachment 79073 [details]
issue image

This issue seems to happen on webkit nightly and chrome on a mac (does not happen on windows for either, haven't tried chrome linux).
This issue doesn't happen on Safari 5.0.3 or Firefox on the mac.

Downstream bug http://code.google.com/p/chromium/issues/detail?id=68487

The test url above shows the issue, click on the circles and a black border is seen.
See the attached image.

Comment 1 Darth 2011-01-15 12:58:57 PST

Also, in that test url, if you remove the css border and outline properties, clicking circles shows the normal focus border, which is correct, and no black border is seen.

Comment 2 Alexey Proskuryakov 2011-01-19 17:13:01 PST

<rdar://problem/8889770>

Comment 3 Andy Estes 2011-01-19 17:38:50 PST

Looks like this regression is due to http://trac.webkit.org/changeset/69237.

Comment 4 Andy Estes 2011-01-26 17:52:14 PST

I can also reproduce this issue by tabbing through the areas, although behavior exists in shipping Safari 5.

Comment 5 chris fleizach 2011-01-26 17:55:53 PST

yea it's pretty reproducible

Comment 6 Andy Estes 2011-02-03 16:51:22 PST

Another site affected by this bug is http://www.conduit.com. Click the tabs labeled 'Create apps yourself', 'Promote & optimize', etc. to see the bug.

Comment 7 Darth 2011-02-11 10:03:09 PST

Created attachment 82141 [details]
testcase

Adding a copy of the test case from the url since the old one was modified by it's original author to do a blur() onclick. So the issue wasn't being seen in the url.

Comment 8 Andy Estes 2011-04-02 16:52:36 PDT

Created attachment 87989 [details]
Patch

Comment 9 Andy Estes 2011-04-02 17:01:46 PDT

Committed r82781: <http://trac.webkit.org/changeset/82781>

Comment 10 Brian Weinstein 2011-04-02 17:02:07 PDT

Comment on attachment 87989 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=87989&action=review

> Source/WebCore/rendering/RenderImage.cpp:353
> +    

Might want to get rid of the whitespace added here.

Comment 11 Andy Estes 2011-04-02 17:03:29 PDT

(In reply to comment #10)
> (From update of attachment 87989 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=87989&action=review
> 
> > Source/WebCore/rendering/RenderImage.cpp:353
> > +    
> 
> Might want to get rid of the whitespace added here.

I did before I landed.