Add a 'length' property to DOMSVGStringList

RESOLVED FIXED in mozilla12

Status

()

Core
SVG
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

({dev-doc-complete})

Trunk
mozilla12
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Follow-up from the last comment in bug 607854 comment 7. For consistency with the other list classes, we should add a 'length' property to DOMSVGStringList to mirror the 'numberOfItems' property.
(Assignee)

Comment 1

6 years ago
Created attachment 592131 [details] [diff] [review]
patch
Assignee: nobody → jwatt
Status: NEW → ASSIGNED
Attachment #592131 - Flags: review?(dholbert)
Keywords: dev-doc-needed
Comment on attachment 592131 [details] [diff] [review]
patch

Looks good!

Just one thing -- fix end-of-line whitespace after "GetNumberOfItems(aLength);" here:
> NS_IMETHODIMP
>+DOMSVGStringList::GetLength(PRUint32 *aLength)
>+{
>+  return GetNumberOfItems(aLength);  
>+}

r=me with that
Attachment #592131 - Flags: review?(dholbert) → review+
https://hg.mozilla.org/mozilla-central/rev/b939fbd0a46f
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
I've documented this here, flagged as non-standard, since it's not in the spec.

https://developer.mozilla.org/en/DOM/SVGStringList

Also mentioned on Firefox 12 for developers.
I added a note on https://developer.mozilla.org/en/Firefox_12_for_developers
I assumed that SVGStringList is also indexable as an Array. Am I right?
Keywords: dev-doc-needed → dev-doc-complete
(In reply to Jeremie Patonnier from comment #5)
> I assumed that SVGStringList is also indexable as an Array. Am I right?
Sorry for the noise, Array style indexing is bug #722071
You need to log in before you can comment on or make changes to this bug.