Bug 15522

Summary: REGRESSION: Combination of client-side image map and <a> tag is not working properly
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: Beth Dakin <bdakin>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, mitz
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 523.x (Safari 3)   
Hardware: PC   
OS: OS X 10.4   
URL: http://testsuite.nokia-boston.com/content/html_imgmaps/imgmaps_name_anchor.html
Attachments:
Description Flags
Allow <map> to have inline children as well as block hyatt: review+

Description zalan 2007-10-15 12:13:45 PDT 
Links are showing up as plain text when they are linked to image map with the coords attribute.

It works fine on Tiger.
Comment 1 Mark Rowe (bdash) 2007-10-15 12:23:41 PDT 
<rdar://problem/5540855>
Comment 2 David Kilzer (:ddkilzer) 2007-10-15 20:30:52 PDT 
(In reply to comment #0)
> Links are showing up as plain text when they are linked to image map with the
> coords attribute.
> 
> It works fine on Tiger.

This test fails in the oldest WebKit svn nightly:  r11976.
Comment 3 Beth Dakin 2007-11-14 13:09:02 PST 
The Web Inspector actually reports errors claiming that <a> and <br> are not allowed inside <map>! That doesn't seem right...
Comment 4 Beth Dakin 2007-11-14 14:01:37 PST 
I have a patch for this. I am creating a bunch of test cases before I post it, though.
Comment 5 Beth Dakin 2007-11-14 15:24:59 PST 
Created attachment 17283 [details]
Allow <map> to have inline children as well as block
Comment 6 Dave Hyatt 2007-11-14 15:44:32 PST 
Comment on attachment 17283 [details]
Allow <map> to have inline children as well as block

Ditch the "imgTag" part since I assume that's inEithertagList.
Comment 7 Dave Hyatt 2007-11-14 15:44:58 PST 
Comment on attachment 17283 [details]
Allow <map> to have inline children as well as block

r=me if you make that change.
Comment 8 Beth Dakin 2007-11-14 15:49:39 PST 
Fixed it! Thanks, Dave. Fixed with r27800.
Comment 9 zalan 2007-11-14 16:00:09 PST 
wow! Thanks a lot! :) I'll apply it to our trunk based branch to test....