Bug 46460 - Text under nested link dropped
Summary: Text under nested link dropped
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-24 06:44 PDT by Michael O'Rourke
Modified: 2010-10-17 23:52 PDT (History)
3 users (show)

See Also:

Attachments
Example SVG file with text and links (28.67 KB, image/svg+xml)
2010-09-24 06:44 PDT, Michael O'Rourke 		no flags Details
Simplified example file (603 bytes, image/svg+xml)
2010-09-24 07:02 PDT, Michael O'Rourke 		no flags Details
First attempt (80.83 KB, patch)
2010-10-16 12:22 PDT, Rob Buis 		krit: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael O'Rourke 2010-09-24 06:44:07 PDT 
The text '5google.com', '6google.com, '7google.com' under a tspan are dropped from the display.

This works fine in both Firefox 3.6.3 and IE 8.0.7600.16385 and the SVG content seems to be legal.

This was seen in Chrome 6.0.472.63.
Comment 1 Michael O'Rourke 2010-09-24 06:44:49 PDT 
Created attachment 68671 [details]
Example SVG file with text and links
Comment 2 Michael O'Rourke 2010-09-24 07:02:10 PDT 
Created attachment 68675 [details]
Simplified example file
Comment 3 Michael O'Rourke 2010-09-24 07:03:49 PDT 
I've attached a simplified example file with only one text span.

If the <a> tag is removed then the text displays fine in webkit browsers.

Note that the existing content still displays fine in IE and Firefox.
Comment 4 Rob Buis 2010-10-16 12:22:47 PDT 
Created attachment 70961 [details]
First attempt
Comment 5 Dirk Schulze 2010-10-16 23:49:03 PDT 
Comment on attachment 70961 [details]
First attempt

r=me, but please move every case in a new line: 
|| child->hasTagName(SVGNames::trefTag)
|| child->hasTagName(SVGNames::tspanTag)
...
Comment 6 Rob Buis 2010-10-17 23:52:07 PDT 
Landed in r69922.