[webkit-reviews] review granted: [Bug 97898] [Microdata] names.item() must return null for out of range indexes : [Attachment 166236] Patch

[bugzilla-daemon at webkit.org]

Kentaro Hara <haraken at chromium.org> has granted Arko Saha <arko at motorola.com>'s
request for review:
Bug 97898: [Microdata] names.item() must return null for out of range indexes
https://bugs.webkit.org/show_bug.cgi?id=97898

Attachment 166236: Patch
https://bugs.webkit.org/attachment.cgi?id=166236&action=review

------- Additional Comments from Kentaro Hara <haraken at chromium.org>
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.