[IndexedDB] array index keys are concatenated across cursor lifetime

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Chris Anderson]

The following test passes in Firefox and Chrome but not Safari (Safari v8 Engine: WebKit v600.1.25 OS: Mac OS X v10.10 and Mobile Safari v8 Engine: WebKit v600.1.4 OS: iOS v10.10)

http://requirebin.com/?gist=fcc75007aa20e504bcd4

multiEntry works fine when the members of the multiEntry array are numbers, but when they are arrays, everything goes sideways. It looks like the arrays are being flattened and concated. My test case only uses one layer of nesting but I discovered it in an application that uses multiple layers of nesting (I'm converting deep objects to nested arrays so I can have a defined collation over objects when used as keys.)

[Chris Anderson]

Oh dear, it's much worse than I thought. Here is the same bug triggered without multiEntry: http://requirebin.com/?gist=4395a44d50f3c6dddcb8

[Chris Anderson]

Good news is this might be a presentation concern, as the results come back in the correct order at least.

[Chris Anderson]

Maybe this helps diagnose or helps others with workarounds:

In my case I know all my keys are 2 element arrays. And the cursor starts with a new correct key on the first row of every transaction.

So if your keys are big, paginate. And know how many elements are in your array keys, and then you can run key.splice(-n) where n is your array key length.

It doesn't flatten it just concats.

[Mark Dixon]

Created attachment 243214 [details]
Patch

[Mark Dixon]

Created attachment 243238 [details]
Patch

[Radar WebKit Bug Importer]

<rdar://problem/19684902>

[Nolan Lawson]

Those RequireBin tests weren't working for me, so I browserified them.

You can open these two links, and if it's green, then they pass, and if it's red, they fail:

* http://bl.ocks.org/nolanlawson/b23876ee55085ebb4696
* http://bl.ocks.org/nolanlawson/fa4f74be1810d726db8a

I'm seeing successes in Chrome 43 and Firefox 38, but failures in Safari 8.0.7 and Webkit nightly r185316 (all on desktop Yosemite).

[Brady Eidson]

Comment on attachment 243238 [details]
Patch

It would be useful to check through any of the currently disabled IDB tests to see if this fixes them - Reenabling them would be preferable to changing this test.

[Brady Eidson]

Since Mark isn't around to respond, I just tried the entire suite of IDB tests and this did not change behavior on any of the skipped tests.

Therefore the included change seems appropriate to me.

r+, will prep for landing.

[Brady Eidson]

http://trac.webkit.org/changeset/185604