IndexedDB could use the tool from bug 709569 for dictionaries

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

8 years ago
No description provided.
Assignee

Updated

8 years ago
Depends on: 709569
Assignee

Comment 1

8 years ago
Posted patch patch (obsolete) — Splinter Review
This passes IndexedDB tests.

I believe !JSVAL_IS_NULL(aOptions) checks are wrong in the ifs, but I decided to
not change them, so the behavior should stay the same.

I did this change to figure out whether the tool in bug 709569 works
with this case.
Attachment #581413 - Flags: review?(jonas)
Assignee

Comment 2

8 years ago
Comment on attachment 581413 [details] [diff] [review]
patch

This doesn't apply cleanly anymore
Attachment #581413 - Flags: review?(jonas)
Assignee

Comment 3

8 years ago
Posted patch up-to-dateSplinter Review
Attachment #581413 - Attachment is obsolete: true
Assignee

Updated

8 years ago
Attachment #584332 - Flags: review?(khuey)
Comment on attachment 584332 [details] [diff] [review]
up-to-date

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

Beautiful.
Attachment #584332 - Flags: review?(khuey) → review+
Assignee

Comment 6

8 years ago
https://hg.mozilla.org/mozilla-central/rev/a8be91348c31
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.