Summary: | [Microdata] names.item() must return null for out of range indexes | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Arko Saha <arko> | ||||
Component: | DOM | Assignee: | Arko Saha <arko> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, haraken, ojan, rniwa, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 92986 | ||||||
Attachments: |
|
Description
Arko Saha
2012-09-28 05:20:40 PDT
Created attachment 166236 [details]
Patch
Comment on attachment 166236 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166236&action=review The implementation looks good. I am just concerned about cross-browser compatibility. > Source/WebCore/ChangeLog:8 > + DOMStringList.item() must return null for an invalid index. What is the behavior of other browsers (IE, Firefox, Opera)? (In reply to comment #2) > (From update of attachment 166236 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=166236&action=review > > The implementation looks good. I am just concerned about cross-browser compatibility. > > > Source/WebCore/ChangeLog:8 > > + DOMStringList.item() must return null for an invalid index. > > What is the behavior of other browsers (IE, Firefox, Opera)? I tested in Opera and Firefox. Both return null in case of negative index. Unable to test it in as I don't have access to IE. Comment on attachment 166236 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166236&action=review >>> Source/WebCore/ChangeLog:8 >>> + DOMStringList.item() must return null for an invalid index. >> >> What is the behavior of other browsers (IE, Firefox, Opera)? > > I tested in Opera and Firefox. Both return null in case of negative index. Unable to test it in as I don't have access to IE. Thanks for the clarification. Given that the spec also requires it, the change looks reasonable. Comment on attachment 166236 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166236&action=review >>>> Source/WebCore/ChangeLog:8 >>>> + DOMStringList.item() must return null for an invalid index. >>> >>> What is the behavior of other browsers (IE, Firefox, Opera)? >> >> I tested in Opera and Firefox. Both return null in case of negative index. Unable to test it in as I don't have access to IE. > > Thanks for the clarification. Given that the spec also requires it, the change looks reasonable. It would be better to describe other browsers' behaviors in the ChangeLog. (In reply to comment #5) > (From update of attachment 166236 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=166236&action=review > > >>>> Source/WebCore/ChangeLog:8 > >>>> + DOMStringList.item() must return null for an invalid index. > >>> > >>> What is the behavior of other browsers (IE, Firefox, Opera)? > >> > >> I tested in Opera and Firefox. Both return null in case of negative index. Unable to test it in as I don't have access to IE. > > > > Thanks for the clarification. Given that the spec also requires it, the change looks reasonable. > > It would be better to describe other browsers' behaviors in the ChangeLog. Sure I will update the Change log with other browsers behavior. Thanks for the review. Committed r130026: <http://trac.webkit.org/changeset/130026> |