1118504 - Add test for storing <input type="file"> blobs in indexedDB

Status: NEW

(Core :: Storage: IndexedDB, task, P3)

Description

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

Attachment: Test

Bug 1076975 needed a test. Carrying over r+ from khuey.

Comment 1

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/43d7ef422777

Comment 2

[Phil Ringnalda (:philor)]

Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/469dbf49630e - at the very least, you need whatever the actual condition is for "fails-if e10s", dunno if there will wind up being more failures (and you separately broke hell out of Windows debug, so we probably aren't going to know whether or not it passes there).

Comment 3

[Phil Ringnalda (:philor)]

Also, dunno how frequent it will be, but https://treeherder.mozilla.org/logviewer.html#?job_id=5140596&repo=mozilla-inbound is it hanging on b2g debug emulator.

Comment 4

[Phil Ringnalda (:philor)]

And https://treeherder.mozilla.org/logviewer.html#?job_id=5142604&repo=mozilla-inbound is it failing on b2g opt emulator.

Comment 5

[Phil Ringnalda (:philor)]

And https://treeherder.mozilla.org/logviewer.html#?job_id=5144741&repo=mozilla-inbound is it failing on 10.6 debug

Comment 6

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/4af240ffb899

Comment 7

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

(In reply to Phil Ringnalda (:philor) from comment #2)
> Backed out

Yeah, there was a small race in the test.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Still failing on e10s. Backed out.
https://hg.mozilla.org/integration/mozilla-inbound/rev/f34d4207cb44

https://treeherder.mozilla.org/logviewer.html#?job_id=5162783&repo=mozilla-inbound

Comment 9

[Ryan VanderMeulen [:RyanVM]]

And Android:
https://treeherder.mozilla.org/logviewer.html#?job_id=5169380&repo=mozilla-inbound

Comment 10

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

I can't actually land this until we figure out what to do in bug 1068838.

Also, since bug 1076975 landed with a test already there's no reason this needs to be related any more.

Comment 11

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

Attachment: Test

This version uses the temp dir to work around the fact that child processes can't currently access the profile directory. It still doesn't work though because the file picker in the child process hands out a file-backed blob.

Comment 12

[Jens Stutte [:jstutte]]

:sgiesecke, is this still something we would want to test?

Comment 13

[Simon Giesecke [:sg] [he/him]]

Yes, I guess so.