Figure out Cursor.advance arg size

RESOLVED FIXED in mozilla16

Status

()

Core
DOM: IndexedDB
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ben Turner (not reading bugmail, use the needinfo flag!), Assigned: khuey)

Tracking

unspecified
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

The spec says Cursor.advance is supposed to be unsigned long, but then also says that we're supposed to throw on negative numbers. Our code has PRInt32, which means we can't accept PR_UINT32_MAX like we're probably supposed to. Need to figure out what to do here.
This is now written with [EnforceRange], so I think we should punt on this until we have new bindings for IDB.
Created attachment 637182 [details] [diff] [review]
Patch
Assignee: nobody → khuey
Status: NEW → ASSIGNED
Attachment #637182 - Flags: review?(bent.mozilla)
Comment on attachment 637182 [details] [diff] [review]
Patch

Review of attachment 637182 [details] [diff] [review]:
-----------------------------------------------------------------

r+ if you add a test.
Attachment #637182 - Flags: review?(bent.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/333849115ee2
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
You need to log in before you can comment on or make changes to this bug.