This was sitting in my tree and I kinda want to get rid of it.
Priority: -- → P3
Summary: Add some more use of AttrValueIs to layout → [FIX]Add some more use of AttrValueIs to layout
Target Milestone: --- → mozilla1.9alpha
Created attachment 208093 [details] [diff] [review] Patch
Comment on attachment 208093 [details] [diff] [review] Patch yum
Fixed. mZ went down by 2900.
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Wasn't there some RFE somewhere for an interface to AttrValueIs on XULElement?
Not sure. AttrValueIs on nsIContent (what I'm using here) is pretty recent (trunk-only).
Well, maybe not that specific interface then... the use case is all the return this.getAttribute("disabled") == "true"; type tests used heavily in XBL.
Oh, a DOM accessible API. Yeah, I guess we could do it. But most of the benefits of the nsIContent AttrValueIs (eg atom compares) would be lost.
You need to log in before you can comment on or make changes to this bug.