618134 - IndexedDB: Rename cursor types to comply with spec changes

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, defect)

Description

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Spec changed, openCursor is now openKeyCursor, openObjectCursor is now OpenCursor.

Comment 1

[Johnny Stenback (:jst)]

This is part of the last set of bugs we've deemed necessary in order to ship
2.0, blocking beta9.

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Attachment: Patch, v1

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 497046 [details] [diff] [review]
Patch, v1

For future reference, you don't need to actually change the iid on interfaces when just renaming functions.

Comment 4

[Shawn Wilsher :sdwilsh]

(In reply to comment #3)
> For future reference, you don't need to actually change the iid on interfaces
> when just renaming functions.
Why not?  Isn't the iid for binary compat, which renaming functions would break?

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Renaming functions doesn't break binary compat

Comment 6

[Ben Turner (not reading bugmail, use the needinfo flag!)]

http://hg.mozilla.org/mozilla-central/rev/5ce86536adbb

Comment 7

[LegNeato]

As per today's meeting, beta 9 will be a time-based release. Marking these all betaN+. Please move it back to beta9+ if  you believe it MUST be in the next beta (ie: trunk is in an unshippable state without this)