Bug 7338

Summary: Incorrect selectorText for multiple subselectors
Product: WebKit Reporter: mitz
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal CC: hyatt
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Testcase
none
Iterate over subselectors
none
A patch that applies darin: review+

mitz
Reported 2006-02-18 08:02:01 PST
Summary: selectorText returns an incorrect string if a descendant selector has more than one subselector. In the attached testcase, "div span.a.b" is serialized as "span.adiv .b".
Attachments
Testcase (297 bytes, text/html)
2006-02-18 08:02 PST, mitz 		no flags
Details
Iterate over subselectors (8.16 KB, patch)
2006-02-20 15:16 PST, mitz 		no flags
DetailsFormatted DiffDiff
A patch that applies (7.70 KB, patch)
2006-02-25 01:28 PST, mitz 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-02-18 08:02:57 PST
Created attachment 6588 [details] Testcase
mitz
Comment 2 2006-02-20 15:16:16 PST
Created attachment 6630 [details] Iterate over subselectors The change is smaller than it might look (most of it is just extra indentation).
Darin Adler
Comment 3 2006-02-20 16:37:00 PST
Comment on attachment 6630 [details] Iterate over subselectors r=me
mitz
Comment 4 2006-02-24 14:55:34 PST
Comment on attachment 6630 [details] Iterate over subselectors This patch no longer applies cleanly. I'm going to post an updated version.
mitz
Comment 5 2006-02-25 01:28:58 PST
Created attachment 6724 [details] A patch that applies Again, the diff is messy but the change is simple.
Darin Adler
Comment 6 2006-02-25 17:36:51 PST
Comment on attachment 6724 [details] A patch that applies r=me again
mitz
Comment 7 2006-03-01 09:34:06 PST
Verified in r13060 nightly
Note You need to log in before you can comment on or make changes to this bug.