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+

Description mitz 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".
Comment 1 mitz 2006-02-18 08:02:57 PST 
Created attachment 6588 [details]
Testcase
Comment 2 mitz 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).
Comment 3 Darin Adler 2006-02-20 16:37:00 PST 
Comment on attachment 6630 [details]
Iterate over subselectors

r=me
Comment 4 mitz 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.
Comment 5 mitz 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.
Comment 6 Darin Adler 2006-02-25 17:36:51 PST 
Comment on attachment 6724 [details]
A patch that applies

r=me again
Comment 7 mitz 2006-03-01 09:34:06 PST 
Verified in r13060 nightly